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 Tue, 06 May 2014 18:01:01 GMT
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