camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jb@nanthrax.net" ...@nanthrax.net>
Subject Re: svn commit: r1408670 - in /camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf: ./ mtom/
Date Wed, 14 Nov 2012 13:51:41 GMT
Hi guys,
I have access to my AIX box from here. I will update my local working copy to run the tests
tonight (east coast time).

Regards
JB
--
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://wwx.talend.com

----- Reply message -----
From: "Daniel Kulp" <dkulp@apache.org>
To: <dev@camel.apache.org>, "Claus Ibsen" <claus.ibsen@gmail.com>
Subject: svn commit: r1408670 - in /camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf:
./ mtom/
Date: Wed, Nov 14, 2012 7:42 am



On Nov 14, 2012, at 2:07 AM, Claus Ibsen <claus.ibsen@gmail.com> wrote:

> On Tue, Nov 13, 2012 at 1:25 PM, Daniel Kulp <dkulp@apache.org> wrote:
>> 
>> 
>> Can I ask why they cannot run on AIX?   Other than the images which may not work
headless, the tests should be able to run fine.    I've afraid this is addressing a symptom,
not the actual problem and I prefer to get the problem looked at.
>> 
> 
> Yeah there was nothing obvious I could see from the junit error that
> indicated why it didn't work on AIX.
> I needed to disable failing tests to have the build and tests be able
> to do a full test suite of the project to also uncover issues further
> down the road.'

If that's the case, I'm -1 on all the "disable on AIX" commits you did.   Disabling a test
just because you don't know why its failing is not OK.  There aren't even descriptions of
what's failing, stack traces, whatever, that would allow others on the list help diagnose
it.  I'm also concerned because we did have these running OK on AIX not too long ago.  Thus
it may be something with your specific AIX instance.   When JB gets back to France, I'll see
if we can retry on his AIX box.


> Though its suspicious that these tests are using images and detect for
> headless mode. Wonder if we can use non images for the test?

Well, it's specifically there to test the images.   JAX-WS requires mapping certain wsdl constructs
to Images as part of the wsdl2java process (although it's a relatively annoying part of the
spec, I'd personally prefer DataSource or similar).   The test is there to specifically test
that part of the spec.  Unfortunately, it won't work with headless.   (actually, that may
need to be re-checked.   That was true with Java5, not sure with 6 or 7.  I'll double check
that today as that could allow re-enabling some test in CXF on trunk that no longer supports
5)


Dan



> 
>> Dan
>> 
>> 
>> 
>> On Nov 13, 2012, at 5:35 AM, davsclaus@apache.org wrote:
>> 
>>> Author: davsclaus
>>> Date: Tue Nov 13 10:35:20 2012
>>> New Revision: 1408670
>>> 
>>> URL: http://svn.apache.org/viewvc?rev=1408670&view=rev
>>> Log:
>>> Skip test that cannot run on aix
>>> 
>>> Added:
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
  (with props)
>>> Modified:
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfJavaMtomProducerPayloadTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerPayloadModeTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomDisabledProducerPayloadModeTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomPOJOProducerTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomProducerPayloadModeTest.java
>>>   camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomRouterPayloadModeTest.java
>>> 
>>> Added: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java?rev=1408670&view=auto
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
(added)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
Tue Nov 13 10:35:20 2012
>>> @@ -0,0 +1,41 @@
>>> +/**
>>> + * Licensed to the Apache Software Foundation (ASF) under one or more
>>> + * contributor license agreements.  See the NOTICE file distributed with
>>> + * this work for additional information regarding copyright ownership.
>>> + * The ASF licenses this file to You under the Apache License, Version 2.0
>>> + * (the "License"); you may not use this file except in compliance with
>>> + * the License.  You may obtain a copy of the License at
>>> + *
>>> + *      http://www.apache.org/licenses/LICENSE-2.0
>>> + *
>>> + * Unless required by applicable law or agreed to in writing, software
>>> + * distributed under the License is distributed on an "AS IS" BASIS,
>>> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
>>> + * See the License for the specific language governing permissions and
>>> + * limitations under the License.
>>> + */
>>> +package org.apache.camel.component.cxf;
>>> +
>>> +import java.util.Locale;
>>> +
>>> +/**
>>> + *
>>> + */
>>> +public final class TestHelper {
>>> +
>>> +    private TestHelper() {
>>> +    }
>>> +
>>> +    /**
>>> +     * Is this OS the given platform.
>>> +     * <p/>
>>> +     * Uses <tt>os.name</tt> from the system properties to determine
the OS.
>>> +     *
>>> +     * @param platform such as Windows
>>> +     * @return <tt>true</tt> if its that platform.
>>> +     */
>>> +    public static boolean isPlatform(String platform) {
>>> +        String osName = System.getProperty("os.name").toLowerCase(Locale.US);
>>> +        return osName.indexOf(platform.toLowerCase(Locale.US)) > -1;
>>> +    }
>>> +}
>>> 
>>> Propchange: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
>>> ------------------------------------------------------------------------------
>>>   svn:eol-style = native
>>> 
>>> Propchange: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/TestHelper.java
>>> ------------------------------------------------------------------------------
>>>   svn:keywords = Rev Date
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfJavaMtomProducerPayloadTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfJavaMtomProducerPayloadTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfJavaMtomProducerPayloadTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfJavaMtomProducerPayloadTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -22,6 +22,7 @@ import javax.xml.ws.Holder;
>>> 
>>> import org.apache.camel.Exchange;
>>> import org.apache.camel.Processor;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.junit.Test;
>>> 
>>> public class CxfJavaMtomProducerPayloadTest extends CxfMtomConsumerTest {
>>> @@ -35,6 +36,11 @@ public class CxfJavaMtomProducerPayloadT
>>>            return;
>>>        }
>>> 
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        final Holder<byte[]> photo = new Holder<byte[]>("RequestFromCXF".getBytes("UTF-8"));
>>>        final Holder<Image> image = new Holder<Image>(getImage("/java.jpg"));
>>> 
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerPayloadModeTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerPayloadModeTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerPayloadModeTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerPayloadModeTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -38,6 +38,7 @@ import org.apache.camel.ExchangePattern;
>>> import org.apache.camel.Processor;
>>> import org.apache.camel.component.cxf.CXFTestSupport;
>>> import org.apache.camel.component.cxf.CxfPayload;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.apache.camel.converter.jaxp.XmlConverter;
>>> import org.apache.cxf.binding.soap.SoapHeader;
>>> import org.apache.cxf.helpers.DOMUtils;
>>> @@ -65,6 +66,15 @@ public class CxfMtomConsumerPayloadModeT
>>> 
>>>    @Test
>>>    public void testConsumer() throws Exception {
>>> +        if (MtomTestHelper.isAwtHeadless(logger, null)) {
>>> +            return;
>>> +        }
>>> +
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        context.createProducerTemplate().send("cxf:bean:consumerEndpoint", new
Processor() {
>>> 
>>>            public void process(Exchange exchange) throws Exception {
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomConsumerTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -31,6 +31,7 @@ import org.apache.camel.Message;
>>> import org.apache.camel.Processor;
>>> import org.apache.camel.builder.RouteBuilder;
>>> import org.apache.camel.component.cxf.CXFTestSupport;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.apache.camel.cxf.mtom_feature.Hello;
>>> import org.apache.camel.cxf.mtom_feature.HelloService;
>>> import org.apache.camel.test.junit4.CamelTestSupport;
>>> @@ -99,6 +100,11 @@ public class CxfMtomConsumerTest extends
>>>            return;
>>>        }
>>> 
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        Holder<byte[]> photo = new Holder<byte[]>("RequestFromCXF".getBytes("UTF-8"));
>>>        Holder<Image> image = new Holder<Image>(getImage("/java.jpg"));
>>> 
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomDisabledProducerPayloadModeTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomDisabledProducerPayloadModeTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomDisabledProducerPayloadModeTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomDisabledProducerPayloadModeTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -40,6 +40,7 @@ import org.apache.camel.Exchange;
>>> import org.apache.camel.ExchangePattern;
>>> import org.apache.camel.Processor;
>>> import org.apache.camel.component.cxf.CxfPayload;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.apache.camel.cxf.mtom_feature.Hello;
>>> import org.apache.cxf.attachment.AttachmentDataSource;
>>> import org.apache.cxf.binding.soap.SoapHeader;
>>> @@ -77,6 +78,11 @@ public class CxfMtomDisabledProducerPayl
>>>            return;
>>>        }
>>> 
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        Exchange exchange = context.createProducerTemplate().send("direct:testEndpoint",
new Processor() {
>>> 
>>>            public void process(Exchange exchange) throws Exception {
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomPOJOProducerTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomPOJOProducerTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomPOJOProducerTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomPOJOProducerTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -16,7 +16,6 @@
>>> */
>>> package org.apache.camel.component.cxf.mtom;
>>> 
>>> -
>>> import java.awt.Image;
>>> import java.awt.image.BufferedImage;
>>> 
>>> @@ -32,6 +31,7 @@ import org.apache.camel.Exchange;
>>> import org.apache.camel.Processor;
>>> import org.apache.camel.component.cxf.CXFTestSupport;
>>> 
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.junit.After;
>>> import org.junit.Before;
>>> import org.junit.Test;
>>> @@ -40,6 +40,7 @@ import org.springframework.test.context.
>>> import org.springframework.test.context.junit4.AbstractJUnit4SpringContextTests;
>>> 
>>> import static org.junit.Assert.assertEquals;
>>> +
>>> /**
>>> * Unit test for exercising MTOM enabled end-to-end router in PAYLOAD mode
>>> *
>>> @@ -75,6 +76,11 @@ public class CxfMtomPOJOProducerTest ext
>>>            return;
>>>        }
>>> 
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        final Holder<byte[]> photo = new Holder<byte[]>(MtomTestHelper.REQ_PHOTO_DATA);
>>>        final Holder<Image> image = new Holder<Image>(getImage("/java.jpg"));
>>> 
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomProducerPayloadModeTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomProducerPayloadModeTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomProducerPayloadModeTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomProducerPayloadModeTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -40,6 +40,7 @@ import org.apache.camel.ExchangePattern;
>>> import org.apache.camel.Processor;
>>> import org.apache.camel.component.cxf.CXFTestSupport;
>>> import org.apache.camel.component.cxf.CxfPayload;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.apache.camel.converter.jaxp.XmlConverter;
>>> import org.apache.cxf.binding.soap.SoapHeader;
>>> import org.apache.cxf.helpers.DOMUtils;
>>> @@ -89,6 +90,11 @@ public class CxfMtomProducerPayloadModeT
>>>            return;
>>>        }
>>> 
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> +
>>>        // START SNIPPET: producer
>>> 
>>>        Exchange exchange = context.createProducerTemplate().send("direct:testEndpoint",
new Processor() {
>>> 
>>> Modified: camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomRouterPayloadModeTest.java
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomRouterPayloadModeTest.java?rev=1408670&r1=1408669&r2=1408670&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomRouterPayloadModeTest.java
(original)
>>> +++ camel/trunk/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/mtom/CxfMtomRouterPayloadModeTest.java
Tue Nov 13 10:35:20 2012
>>> @@ -16,7 +16,6 @@
>>> */
>>> package org.apache.camel.component.cxf.mtom;
>>> 
>>> -
>>> import java.awt.Image;
>>> import java.awt.image.BufferedImage;
>>> import java.net.URL;
>>> @@ -31,6 +30,7 @@ import junit.framework.Assert;
>>> 
>>> import org.apache.camel.CamelContext;
>>> import org.apache.camel.component.cxf.CXFTestSupport;
>>> +import org.apache.camel.component.cxf.TestHelper;
>>> import org.apache.camel.cxf.mtom_feature.Hello;
>>> import org.apache.camel.cxf.mtom_feature.HelloService;
>>> import org.junit.After;
>>> @@ -41,6 +41,7 @@ import org.springframework.test.context.
>>> import org.springframework.test.context.junit4.AbstractJUnit4SpringContextTests;
>>> 
>>> import static org.junit.Assert.assertNotNull;
>>> +
>>> /**
>>> * Unit test for exercising MTOM enabled end-to-end router in PAYLOAD mode
>>> *
>>> @@ -76,6 +77,11 @@ public class CxfMtomRouterPayloadModeTes
>>>        if (MtomTestHelper.isAwtHeadless(logger, null)) {
>>>            return;
>>>        }
>>> +
>>> +        // skip test on aix
>>> +        if (TestHelper.isPlatform("aix")) {
>>> +            return;
>>> +        }
>>> 
>>>        Holder<byte[]> photo = new Holder<byte[]>(MtomTestHelper.REQ_PHOTO_DATA);
>>>        Holder<Image> image = new Holder<Image>(getImage("/java.jpg"));
>>> 
>>> 
>> 
>> --
>> Daniel Kulp
>> dkulp@apache.org - http://dankulp.com/blog
>> Talend Community Coder - http://coders.talend.com
>> 
> 
> 
> 
> -- 
> Claus Ibsen
> -----------------
> Red Hat, Inc.
> FuseSource is now part of Red Hat
> Email: cibsen@redhat.com
> Web: http://fusesource.com
> Twitter: davsclaus
> Blog: http://davsclaus.com
> Author of Camel in Action: http://www.manning.com/ibsen

-- 
Daniel Kulp
dkulp@apache.org - http://dankulp.com/blog
Talend Community Coder - http://coders.talend.com

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message