diff options
| author | Robert Gemmell <robbie@apache.org> | 2010-05-31 13:11:55 +0000 |
|---|---|---|
| committer | Robert Gemmell <robbie@apache.org> | 2010-05-31 13:11:55 +0000 |
| commit | c586912d2d95595ec9ccec0579fa65c0e7748340 (patch) | |
| tree | 263b0b0698f05e85379923939d3e2bea79cf288e | |
| parent | 9e522de1ececed29fa9c0a02caa535c4f235b486 (diff) | |
| download | qpid-python-c586912d2d95595ec9ccec0579fa65c0e7748340.tar.gz | |
Make PersistentStoreTest.testForcibleStartStopMidTransaction() do a syncchronous op before killing the broker to ensure all the additional messages reach the broker before it is killed. Update message numbering to be based from 1 to aid log clarity.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@949735 13f79535-47bb-0310-9956-ffa450edef68
| -rw-r--r-- | qpid/java/systests/src/main/java/org/apache/qpid/server/store/PersistentStoreTest.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/qpid/java/systests/src/main/java/org/apache/qpid/server/store/PersistentStoreTest.java b/qpid/java/systests/src/main/java/org/apache/qpid/server/store/PersistentStoreTest.java index 65127e50ec..e1941d4629 100644 --- a/qpid/java/systests/src/main/java/org/apache/qpid/server/store/PersistentStoreTest.java +++ b/qpid/java/systests/src/main/java/org/apache/qpid/server/store/PersistentStoreTest.java @@ -21,6 +21,7 @@ package org.apache.qpid.server.store; +import org.apache.qpid.client.AMQSession; import org.apache.qpid.test.utils.QpidTestCase; import javax.jms.Connection; @@ -64,7 +65,7 @@ public class PersistentStoreTest extends QpidTestCase _session = _con.createSession(false, Session.AUTO_ACKNOWLEDGE); _con.start(); _consumer = _session.createConsumer(_destination); - for (int i = 0; i < NUM_MESSAGES; i++) + for (int i = 1; i <= NUM_MESSAGES; i++) { Message msg = _consumer.receive(RECEIVE_TIMEOUT); assertNotNull("Message " + i + " not received", msg); @@ -118,6 +119,7 @@ public class PersistentStoreTest extends QpidTestCase public void testForcibleStartStopMidTransaction() throws Exception { sendMessage(_session, _destination, 5); + ((AMQSession) _session).sync(); restartBroker(); checkMessages(); } @@ -168,7 +170,7 @@ public class PersistentStoreTest extends QpidTestCase MessageProducer producer = session.createProducer(destination); - for (int i = 0;i < (count); i++) + for (int i = 1;i <= (count); i++) { Message next = createNextMessage(session, i); |