diff options
author | Tor Egge <Tor.Egge@online.no> | 2021-10-14 11:41:13 +0200 |
---|---|---|
committer | Tor Egge <Tor.Egge@online.no> | 2021-10-14 11:41:13 +0200 |
commit | f78d629d643c24a20a1f4622c12a236bf008f1e6 (patch) | |
tree | 1635dfe1390e55273a5b334ef0eeaebf2109499e /storage/src/tests/distributor/mergeoperationtest.cpp | |
parent | 1ff64ef8a4a9051558a809d30b838045e9fe0540 (diff) |
Use ASSERT_NO_FATAL_FAILURE() to propagate fatal failures.
Diffstat (limited to 'storage/src/tests/distributor/mergeoperationtest.cpp')
-rw-r--r-- | storage/src/tests/distributor/mergeoperationtest.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/storage/src/tests/distributor/mergeoperationtest.cpp b/storage/src/tests/distributor/mergeoperationtest.cpp index ff8f8101670..65ee5254193 100644 --- a/storage/src/tests/distributor/mergeoperationtest.cpp +++ b/storage/src/tests/distributor/mergeoperationtest.cpp @@ -95,9 +95,9 @@ MergeOperationTest::get_merge_metrics() TEST_F(MergeOperationTest, simple) { auto op = setup_simple_merge_op(); - assert_simple_merge_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_merge_bucket_command()); sendReply(*op); - assert_simple_delete_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_delete_bucket_command()); EXPECT_EQ(0, get_merge_metrics().ok.getValue()); sendReply(*op); EXPECT_EQ(1, get_merge_metrics().ok.getValue()); @@ -105,7 +105,7 @@ TEST_F(MergeOperationTest, simple) { TEST_F(MergeOperationTest, fail_if_source_only_copies_changed) { auto op = setup_simple_merge_op(); - assert_simple_merge_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_merge_bucket_command()); { auto& cmd = dynamic_cast<api::MergeBucketCommand&>(*_sender.command(0)); EXPECT_EQ(0, cmd.getSourceIndex()); @@ -118,7 +118,7 @@ TEST_F(MergeOperationTest, fail_if_source_only_copies_changed) { "2=10/1/1/t"); sendReply(*op); // Should not be a remove here! - assert_simple_merge_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_merge_bucket_command()); EXPECT_FALSE(op->ok()); EXPECT_EQ(1, get_merge_metrics().failed.getValue()); EXPECT_EQ(1, get_merge_metrics().source_only_copy_changed.getValue()); @@ -126,9 +126,9 @@ TEST_F(MergeOperationTest, fail_if_source_only_copies_changed) { TEST_F(MergeOperationTest, fail_if_delete_bucket_fails) { auto op = setup_simple_merge_op(); - assert_simple_merge_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_merge_bucket_command()); sendReply(*op); - assert_simple_delete_bucket_command(); + ASSERT_NO_FATAL_FAILURE(assert_simple_delete_bucket_command()); sendReply(*op, -1, api::ReturnCode::ABORTED); EXPECT_EQ(1, get_merge_metrics().failed.getValue()); EXPECT_EQ(1, get_merge_metrics().source_only_copy_delete_failed.getValue()); |