Skip to content

Commit 8f8fb9d

Browse files
committed
[yugabyte#5122] Fix TestDeleteEmptyTable
Summary: After Disabling the 'ignore deleted tables' feature in the D9297 hotfix, it broke TestDeleteEmptyTable. This is a false positive. The error message slightly changes based on whether the table information is brought into RAM on load or not but it is functionally equivalent. Fixed the unit test to be less detailed on error message expectations. Test Plan: TestDeleteEmptyTable Reviewers: rao, bogdan, alex Reviewed By: alex Subscribers: ybase Differential Revision: https://phabricator.dev.yugabyte.com/D9306
1 parent 6934936 commit 8f8fb9d

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/yb/integration-tests/delete_table-test.cc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -335,8 +335,7 @@ TEST_F(DeleteTableTest, TestDeleteEmptyTable) {
335335
ASSERT_OK(cluster_->master_proxy()->GetTabletLocations(req, &resp, &rpc));
336336
SCOPED_TRACE(resp.DebugString());
337337
ASSERT_EQ(1, resp.errors_size());
338-
ASSERT_STR_CONTAINS(resp.errors(0).ShortDebugString(),
339-
"code: NOT_FOUND message: \"Unknown tablet");
338+
ASSERT_STR_CONTAINS(resp.errors(0).ShortDebugString(), "code: NOT_FOUND");
340339
}
341340

342341
// 4) The master 'dump-entities' page should not list the deleted table or tablets.

0 commit comments

Comments
 (0)