diff options
author | Alan Conway <aconway@apache.org> | 2012-07-26 20:08:29 +0000 |
---|---|---|
committer | Alan Conway <aconway@apache.org> | 2012-07-26 20:08:29 +0000 |
commit | 507cef570e873171aa0a1096cbca2acc76e282d4 (patch) | |
tree | b79d97bd50440d9ad7210f178002f2c0e5d5cc0c /cpp/src/tests | |
parent | 4dab66463b394fa8052ceb89ce185fd4e76ce632 (diff) | |
download | qpid-python-507cef570e873171aa0a1096cbca2acc76e282d4.tar.gz |
QPID-4159: HA missing messages in failover test.
Fix test_failover_send_receive showing missing messages. With this fix,
ran with -DDURATION=2 overnight with no failures.
- Primary, RemoteBackup: Only report "ready" once per remote backup.
- HaBroker: Put membership updates under mutex.
- ReplicatingSubscription: Check for backup missing messages at the front.
- ha_tests.py: Added assertion to test_priority_ring, verify primary queue as expected.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1366179 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/tests')
-rwxr-xr-x | cpp/src/tests/ha_tests.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/cpp/src/tests/ha_tests.py b/cpp/src/tests/ha_tests.py index df54ea8ec3..d25281eed5 100755 --- a/cpp/src/tests/ha_tests.py +++ b/cpp/src/tests/ha_tests.py @@ -588,8 +588,10 @@ class ReplicationTests(BrokerTest): # correct result, the uncommented one is for the actualy buggy # result. See https://issues.apache.org/jira/browse/QPID-3866 # - # backup.assert_browse_backup("q", sorted(priorities,reverse=True)[0:5], transform=lambda m: m.priority) - backup.assert_browse_backup("q", [9,9,9,9,2], transform=lambda m: m.priority) + # expect = sorted(priorities,reverse=True)[0:5] + expect = [9,9,9,9,2] + primary.assert_browse("q", expect, transform=lambda m: m.priority) + backup.assert_browse_backup("q", expect, transform=lambda m: m.priority) def test_backup_acquired(self): """Verify that acquired messages are backed up, for all queue types.""" |