You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gs...@apache.org on 2009/01/21 15:22:55 UTC

svn commit: r736317 - /qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp

Author: gsim
Date: Wed Jan 21 06:22:55 2009
New Revision: 736317

URL: http://svn.apache.org/viewvc?rev=736317&view=rev
Log:
Fix for race condition in test identified by Ted.


Modified:
    qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp

Modified: qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp?rev=736317&r1=736316&r2=736317&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp (original)
+++ qpid/trunk/qpid/cpp/src/tests/QueueEvents.cpp Wed Jan 21 06:22:55 2009
@@ -79,14 +79,15 @@
     QueuedMessage event1(&queue, MessageUtils::createMessage(), id);
     QueuedMessage event2(&queue, MessageUtils::createMessage(), ++id);
 
-    events.enqueued(event1);
-    events.enqueued(event2);
-    events.dequeued(event1);
     //define events expected by listener:
     listener.expect(QueueEvents::Event(QueueEvents::ENQUEUE, event1));
     listener.expect(QueueEvents::Event(QueueEvents::ENQUEUE, event2));
     listener.expect(QueueEvents::Event(QueueEvents::DEQUEUE, event1));
 
+    events.enqueued(event1);
+    events.enqueued(event2);
+    events.dequeued(event1);
+
     dispatchThread.join();
     events.shutdown();
     events.unregisterListener("dummy");



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org