cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dennis Sosnoski <...@sosnoski.com>
Subject Re: git commit: Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests
Date Thu, 08 May 2014 22:04:35 GMT
I reverted the change yesterday, so let me know if you're still seeing 
any problems. I don't know why I didn't see the issues with the 
integration tests, which led to Dan changing the order, before I checked 
in the original change. Normally I run the RM integration tests before 
every commit. Sorry for the slip-up.

Please copy me directly on any issues and emails for now - I'm not 
receiving emails from the CXF lists at present.

   - Dennis

On 05/07/2014 10:47 PM, Aki Yoshida wrote:
> Hi,
> Alessio identified the cause of this problem to be CXF-5726 and I just
> commented on that ticket about this issue.
> regards, aki
>
> 2014-05-06 20:01 GMT+02:00 Aki Yoshida <elakito@gmail.com>:
>> Hi Dennis,
>> I was wondering about the code change that caused the test error that
>> lead to this test code change.
>>
>> I am seeing the following exception in one of my local old scenarios
>> that invokes oneway and request-responser operations in that order.
>> This has been working for 2.7.x and also working with
>> 3.0.0-milerstonres2.
>>
>> However, I noticed that it is failing with 3.0.0-SNAPSHOT with the
>> following exception. And this seems to be the same error.
>>
>> org.apache.cxf.interceptor.Fault: It is not possible to send a create
>> sequence request to the anonymous address
>> http://www.w3.org/2005/08/addressing/anonymous
>> at org.apache.cxf.ws.rm.AbstractRMInterceptor.handleMessage(AbstractRMInterceptor.java:104)
>> at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:307)
>> at org.apache.cxf.interceptor.OutgoingChainInterceptor.handleMessage(OutgoingChainInterceptor.java:81)
>> at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:307)
>> at org.apache.cxf.transport.ChainInitiationObserver.onMessage(ChainInitiationObserver.java:121)
>> at org.apache.cxf.transport.http.AbstractHTTPDestination.invoke(AbstractHTTPDestination.java:243)
>> at org.apache.cxf.transport.http_jetty.JettyHTTPDestination.doService(JettyHTTPDestination.java:259)
>> at org.apache.cxf.transport.http_jetty.JettyHTTPHandler.handle(JettyHTTPHandler.java:65)
>> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1088)
>> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1024)
>> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)
>> at org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:255)
>> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)
>> at org.eclipse.jetty.server.Server.handle(Server.java:370)
>> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494)
>> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982)
>> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043)
>> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)
>> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)
>> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)
>> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696)
>> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53)
>> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)
>> at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)
>> at java.lang.Thread.run(Thread.java:695)
>> Caused by: org.apache.cxf.ws.rm.RMException: It is not possible to
>> send a create sequence request to the anonymous address
>> http://www.w3.org/2005/08/addressing/anonymous
>> at org.apache.cxf.ws.rm.RMManager.getSequence(RMManager.java:458)
>> at org.apache.cxf.ws.rm.RMCaptureOutInterceptor.handle(RMCaptureOutInterceptor.java:156)
>> at org.apache.cxf.ws.rm.AbstractRMInterceptor.handleMessage(AbstractRMInterceptor.java:83)
>> ... 24 more
>>
>> I think there is some regression issue in the code.
>>
>>
>> regards, aki
>>
>> 2014-05-02 20:29 GMT+02:00  <dkulp@apache.org>:
>>> Repository: cxf
>>> Updated Branches:
>>>    refs/heads/master 126775733 -> a4c85510a
>>>
>>>
>>> Reorder some of hte calls to get the createSequence stuff occuring immediately
to fix the tests
>>>
>>>
>>> Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
>>> Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/a4c85510
>>> Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/a4c85510
>>> Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/a4c85510
>>>
>>> Branch: refs/heads/master
>>> Commit: a4c85510a14854349d877510d3688c47457e6b3b
>>> Parents: 1267757
>>> Author: Daniel Kulp <dkulp@apache.org>
>>> Authored: Fri May 2 14:14:55 2014 -0400
>>> Committer: Daniel Kulp <dkulp@apache.org>
>>> Committed: Fri May 2 14:14:55 2014 -0400
>>>
>>> ----------------------------------------------------------------------
>>>   .../cxf/systest/ws/policy/RM10PolicyWsdlTest.java      | 11 ++++++-----
>>>   .../cxf/systest/ws/policy/RM12PolicyWsdlTest.java      | 13 +++++++------
>>>   .../org/apache/cxf/systest/ws/policy/RMPolicyTest.java | 13 +++++++------
>>>   3 files changed, 20 insertions(+), 17 deletions(-)
>>> ----------------------------------------------------------------------
>>>
>>>
>>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java
>>> ----------------------------------------------------------------------
>>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java
b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java
>>> index 3bb79e8..b19536f 100644
>>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java
>>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM10PolicyWsdlTest.java
>>> @@ -82,13 +82,14 @@ public class RM10PolicyWsdlTest extends RMPolicyWsdlTestBase
{
>>>
>>>           ConnectionHelper.setKeepAliveConnection(greeter, true);
>>>
>>> -        // oneway
>>> -
>>> -        greeter.greetMeOneWay("CXF");
>>>
>>>           // two-way
>>>
>>>           assertEquals("CXF", greeter.greetMe("cxf"));
>>> +
>>> +        // oneway
>>> +
>>> +        greeter.greetMeOneWay("CXF");
>>>
>>>           // exception
>>>
>>> @@ -118,14 +119,14 @@ public class RM10PolicyWsdlTest extends RMPolicyWsdlTestBase
{
>>>
>>>           mf.verifyMessages(5, true);
>>>           String[] expectedActions = new String[] {RM10Constants.INSTANCE.getCreateSequenceAction(),
>>> -                                                 GREETMEONEWAY_ACTION,
>>>                                                    GREETME_ACTION,
>>> +                                                 GREETMEONEWAY_ACTION,
>>>                                                    PINGME_ACTION,
>>>                                                    PINGME_ACTION};
>>>           mf.verifyActions(expectedActions, true);
>>>           mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true);
>>>           mf.verifyLastMessage(new boolean[] {false, false, false, false, false},
true);
>>> -        mf.verifyAcknowledgements(new boolean[] {false, false, false, true,
true}, true);
>>> +        mf.verifyAcknowledgements(new boolean[] {false, false, true, false,
true}, true);
>>>
>>>           mf.verifyMessages(4, false);
>>>   //        mf.verifyMessages(9, false);
>>>
>>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java
>>> ----------------------------------------------------------------------
>>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java
b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java
>>> index e31d86f..e1b9191 100644
>>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java
>>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RM12PolicyWsdlTest.java
>>> @@ -81,14 +81,15 @@ public class RM12PolicyWsdlTest extends RMPolicyWsdlTestBase
{
>>>           ConnectionHelper.setKeepAliveConnection(greeter, true);
>>>           LOG.fine("Created greeter client.");
>>>
>>> -        // oneway
>>> -
>>> -        greeter.greetMeOneWay("CXF");
>>>
>>>           // two-way
>>>
>>>           assertEquals("CXF", greeter.greetMe("cxf"));
>>> -
>>> +
>>> +        // oneway
>>> +
>>> +        greeter.greetMeOneWay("CXF");
>>> +
>>>           // exception
>>>
>>>           try {
>>> @@ -117,14 +118,14 @@ public class RM12PolicyWsdlTest extends RMPolicyWsdlTestBase
{
>>>
>>>           mf.verifyMessages(5, true);
>>>           String[] expectedActions = new String[] {RM11Constants.INSTANCE.getCreateSequenceAction(),
>>> -                                                 GREETMEONEWAY_ACTION,
>>>                                                    GREETME_ACTION,
>>> +                                                 GREETMEONEWAY_ACTION,
>>>                                                    PINGME_ACTION,
>>>                                                    PINGME_ACTION};
>>>           mf.verifyActions(expectedActions, true);
>>>           mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true);
>>>           mf.verifyLastMessage(new boolean[] {false, false, false, false, false},
true);
>>> -        mf.verifyAcknowledgements(new boolean[] {false, false, false, true,
true}, true);
>>> +        mf.verifyAcknowledgements(new boolean[] {false, false, true, false,
true}, true);
>>>
>>>           mf.verifyMessages(4, false);
>>>   //        mf.verifyMessages(9, false);
>>>
>>> http://git-wip-us.apache.org/repos/asf/cxf/blob/a4c85510/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java
>>> ----------------------------------------------------------------------
>>> diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java
b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java
>>> index 4507d96..0f18399 100644
>>> --- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java
>>> +++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/policy/RMPolicyTest.java
>>> @@ -145,14 +145,15 @@ public class RMPolicyTest extends AbstractBusClientServerTestBase
{
>>>           LOG.fine("Created greeter client.");
>>>
>>>           ConnectionHelper.setKeepAliveConnection(greeter, true);
>>> -        // oneway
>>> -
>>> -        greeter.greetMeOneWay("CXF");
>>>
>>>           // two-way
>>>
>>>           assertEquals("CXF", greeter.greetMe("cxf"));
>>> -
>>> +
>>> +        // oneway
>>> +
>>> +        greeter.greetMeOneWay("CXF");
>>> +
>>>           // exception
>>>
>>>           try {
>>> @@ -178,14 +179,14 @@ public class RMPolicyTest extends AbstractBusClientServerTestBase
{
>>>
>>>           mf.verifyMessages(5, true);
>>>           String[] expectedActions = new String[] {RM10Constants.INSTANCE.getCreateSequenceAction(),
>>> -                                                 GREETMEONEWAY_ACTION,
>>>                                                    GREETME_ACTION,
>>> +                                                 GREETMEONEWAY_ACTION,
>>>                                                    PINGME_ACTION,
>>>                                                    PINGME_ACTION};
>>>           mf.verifyActions(expectedActions, true);
>>>           mf.verifyMessageNumbers(new String[] {null, "1", "2", "3", "4"}, true);
>>>           mf.verifyLastMessage(new boolean[] {false, false, false, false, false},
true);
>>> -        mf.verifyAcknowledgements(new boolean[] {false, false, false, true,
true}, true);
>>> +        mf.verifyAcknowledgements(new boolean[] {false, false, true, false,
true}, true);
>>>
>>>           mf.verifyMessages(4, false);
>>>   //        mf.verifyMessages(9, false);
>>>


Mime
View raw message