Author: aconway
Date: Wed Aug 22 12:18:17 2007
New Revision: 568732

URL: http://svn.apache.org/viewvc?rev=568732&view=rev
Log:

* specs/amqp-transitional.0-10.xml: Removed ping and pong methods.
  They have been removed from the spec.

* cpp/src/tests/Cluster.cpp, Cluster_child.cpp:
  Removed use of SessionPing/PongBody.

Modified:
    incubator/qpid/trunk/qpid/cpp/src/tests/Cluster.cpp
    incubator/qpid/trunk/qpid/cpp/src/tests/Cluster_child.cpp
    incubator/qpid/trunk/qpid/specs/amqp-transitional.0-10.xml

Modified: incubator/qpid/trunk/qpid/cpp/src/tests/Cluster.cpp
URL: 
http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/cpp/src/tests/Cluster.cpp?rev=568732&r1=568731&r2=568732&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/cpp/src/tests/Cluster.cpp (original)
+++ incubator/qpid/trunk/qpid/cpp/src/tests/Cluster.cpp Wed Aug 22 12:18:17 2007
@@ -19,8 +19,8 @@
 #include "Cluster.h"
 #include "test_tools.h"
 
-#include "qpid/framing/SessionPingBody.h"
-#include "qpid/framing/SessionPongBody.h"
+#include "qpid/framing/SessionOpenBody.h"
+#include "qpid/framing/SessionAttachedBody.h"
 #include "qpid/framing/all_method_bodies.h"
 #include "qpid/cluster/ClassifierHandler.h"
 
@@ -34,11 +34,11 @@
 /** Verify membership in a cluster with one member. */
 BOOST_AUTO_TEST_CASE(testClusterOne) {
     TestCluster cluster("clusterOne", "amqp:one:1");
-    AMQFrame send(VER, 1, SessionPingBody(VER));
+    AMQFrame send(VER, 1, SessionOpenBody(VER));
     cluster.handle(send);
     AMQFrame received;
     BOOST_REQUIRE(cluster.received.waitPop(received));
-    BOOST_CHECK_TYPEID_EQUAL(SessionPingBody, *received.getBody());
+    BOOST_CHECK_TYPEID_EQUAL(SessionOpenBody, *received.getBody());
     BOOST_CHECK_EQUAL(1u, cluster.size());
     Cluster::MemberList members = cluster.getMembers();
     BOOST_CHECK_EQUAL(1u, members.size());
@@ -60,14 +60,14 @@
         BOOST_REQUIRE(cluster.waitFor(2)); // Myself and child.
 
         // Exchange frames with child.
-        AMQFrame send(VER, 1, SessionPingBody(VER));
+        AMQFrame send(VER, 1, SessionOpenBody(VER));
         cluster.handle(send);
         AMQFrame received;
         BOOST_REQUIRE(cluster.received.waitPop(received));
-        BOOST_CHECK_TYPEID_EQUAL(SessionPingBody, *received.getBody());
+        BOOST_CHECK_TYPEID_EQUAL(SessionOpenBody, *received.getBody());
         
         BOOST_REQUIRE(cluster.received.waitPop(received));
-        BOOST_CHECK_TYPEID_EQUAL(SessionPongBody, *received.getBody());
+        BOOST_CHECK_TYPEID_EQUAL(SessionAttachedBody, *received.getBody());
 
         if (!nofork) {
             // Wait for child to exit.

Modified: incubator/qpid/trunk/qpid/cpp/src/tests/Cluster_child.cpp
URL: 
http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/cpp/src/tests/Cluster_child.cpp?rev=568732&r1=568731&r2=568732&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/cpp/src/tests/Cluster_child.cpp (original)
+++ incubator/qpid/trunk/qpid/cpp/src/tests/Cluster_child.cpp Wed Aug 22 
12:18:17 2007
@@ -20,8 +20,8 @@
 
 #include "Cluster.h"
 #include "test_tools.h"
-#include "qpid/framing/SessionPingBody.h"
-#include "qpid/framing/SessionPongBody.h"
+#include "qpid/framing/SessionOpenBody.h"
+#include "qpid/framing/SessionAttachedBody.h"
 
 using namespace std;
 using namespace qpid;
@@ -37,13 +37,13 @@
     TestCluster cluster("clusterTwo", "amqp:child:2");
     AMQFrame frame;
     BOOST_REQUIRE(cluster.received.waitPop(frame)); // Frame from parent.
-    BOOST_CHECK_TYPEID_EQUAL(SessionPingBody, *frame.getBody());
+    BOOST_CHECK_TYPEID_EQUAL(SessionOpenBody, *frame.getBody());
     BOOST_CHECK_EQUAL(2u, cluster.size()); // Me and parent
 
-    AMQFrame send(VER, 1, SessionPongBody(VER));
+    AMQFrame send(VER, 1, SessionAttachedBody(VER));
     cluster.handle(send);
     BOOST_REQUIRE(cluster.received.waitPop(frame));
-    BOOST_CHECK_TYPEID_EQUAL(SessionPongBody, *frame.getBody());
+    BOOST_CHECK_TYPEID_EQUAL(SessionAttachedBody, *frame.getBody());
 } 
 
 int test_main(int, char**) {

Modified: incubator/qpid/trunk/qpid/specs/amqp-transitional.0-10.xml
URL: 
http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/specs/amqp-transitional.0-10.xml?rev=568732&r1=568731&r2=568732&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/specs/amqp-transitional.0-10.xml (original)
+++ incubator/qpid/trunk/qpid/specs/amqp-transitional.0-10.xml Wed Aug 22 
12:18:17 2007
@@ -1863,10 +1863,6 @@
                     / C:RESUME S:ATTACHED
       use-session   = C:FLOW S:FLOW-OK
                     / S:FLOW C:FLOW-OK
-                    / S:PING
-                    / C:PONG
-                    / C:PING
-                    / S:PONG
       close-session = C:SUSPEND S:DETACHED
                     / C:CLOSE S:CLOSED
                     / S:CLOSED
@@ -2225,19 +2221,6 @@
       <chassis name="server" implement="MUST" />
       <chassis name="client" implement="MUST" />
     </method>
-
-    <!-- ping and pong are still used by cluster test -->
-    <method name = "ping" index = "140" label = "ping">
-      <chassis name = "server" implement = "MUST" />
-      <chassis name = "client" implement = "MUST" />
-    </method>
-
-    <method name = "pong" index = "150" label = "pong">
-      <chassis name = "server" implement = "MUST" />
-      <chassis name = "client" implement = "MUST" />
-    </method>
-
-
   </class>
 
   <!-- == Class: access 
======================================================================== -->


Reply via email to