cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alessio Soldano <asold...@redhat.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 07:20:04 GMT
For the records, this is basically the same I asked yesterday on the 
user list: 
http://cxf.547215.n5.nabble.com/Regression-related-to-CXF-5726-td5743759.html

On 06/05/14 20:01, Aki Yoshida wrote:
> 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);
>>


-- 
Alessio Soldano
Web Service Lead, JBoss


Mime
View raw message