From 7c5d55bd383dc63f708486b94dd494f78adc73c2 Mon Sep 17 00:00:00 2001 From: Yingxin Cheng Date: Sat, 12 Oct 2019 16:18:00 +0800 Subject: [PATCH] test/crimson: send message only after getting the connection It is possible for stateless/stateful server to lose the pending message due to the remote markdown, causing unexpected test failure. Signed-off-by: Yingxin Cheng --- src/test/crimson/test_messenger.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/test/crimson/test_messenger.cc b/src/test/crimson/test_messenger.cc index 3046c8090ff..3d6aeb6744d 100644 --- a/src/test/crimson/test_messenger.cc +++ b/src/test/crimson/test_messenger.cc @@ -2945,8 +2945,8 @@ test_v2_lossy_client(FailoverTest& test) { return suite.markdown(); }).then([&suite] { return suite.connect_peer(); - }).then([&test] { - return test.send_bidirectional(); + }).then([&suite] { + return suite.send_peer(); }).then([&suite] { return suite.wait_results(2); }).then([] (ConnResults& results) { @@ -2995,7 +2995,7 @@ test_v2_stateless_server(FailoverTest& test) { }).then([&test] { return test.peer_connect_me(); }).then([&test] { - return test.send_bidirectional(); + return test.peer_send_me(); }).then([&suite] { return suite.wait_results(2); }).then([] (ConnResults& results) { -- 2.47.3