cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aki Yoshida <elak...@gmail.com>
Subject Re: git commit: Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests
Date Wed, 07 May 2014 10:47:50 GMT
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