Re: git commit: Reorder some of hte calls to get the createSequence stuff occuring immediately to fix the tests

Wed, 07 May 2014 13:11:01 -0700

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  <dk...@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 <dk...@apache.org>
Authored: Fri May 2 14:14:55 2014 -0400
Committer: Daniel Kulp <dk...@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

Reply via email to