Missing Status check for kPurgeAllFiles restore mode in test

facebook · Jan 18, 2025 · b155720 · b155720
1 parent 4bd616e
commit b155720
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/utilities/backup/backup_engine_test.cc b/utilities/backup/backup_engine_test.cc
@@ -1309,11 +1309,11 @@ TEST_F(BackupEngineTest, IncrementalRestore) {
     Status s = db_->VerifyChecksum();
 
     // Check DB contents.
-    if (mode == RestoreOptions::Mode::kVerifyChecksum) {
+    if (mode == RestoreOptions::Mode::kKeepLatestDbSessionIdFiles) {
+      ASSERT_TRUE(s.IsCorruption());
+    } else {
       EXPECT_OK(s);
       AssertExists(db_.get(), 0, keys_iteration * 2);
-    } else if (mode == RestoreOptions::Mode::kKeepLatestDbSessionIdFiles) {
-      ASSERT_TRUE(s.IsCorruption());
     }
 
     db_.reset();  // Close DB.