fault_injection_test: bring back 3 iteration runs
Summary: 3 iterations were disabled by mistake by one recent commit, causing CLANG build error. Fix it Test Plan: USE_CLANG=1 make fault_injection_test and run the test Reviewers: igor Reviewed By: igor Subscribers: leveldb, dhruba Differential Revision: https://reviews.facebook.net/D32109
This commit is contained in:
parent
c2e8e8c1c0
commit
43ec4e68ba
@ -620,39 +620,41 @@ TEST(FaultInjectionTest, FaultTest) {
|
|||||||
Random rnd(301);
|
Random rnd(301);
|
||||||
ASSERT_OK(SetUp());
|
ASSERT_OK(SetUp());
|
||||||
|
|
||||||
int num_pre_sync = rnd.Uniform(kMaxNumValues);
|
for (size_t idx = 0; idx < kNumIterations; idx++) {
|
||||||
int num_post_sync = rnd.Uniform(kMaxNumValues);
|
int num_pre_sync = rnd.Uniform(kMaxNumValues);
|
||||||
|
int num_post_sync = rnd.Uniform(kMaxNumValues);
|
||||||
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
|
||||||
PartialCompactTestReopenWithFault(kResetDropUnsyncedData, num_pre_sync,
|
|
||||||
num_post_sync);
|
|
||||||
NoWriteTestPreFault();
|
|
||||||
NoWriteTestReopenWithFault(kResetDropUnsyncedData);
|
|
||||||
|
|
||||||
// TODO(t6070540) Need to sync WAL Dir and other DB paths too.
|
|
||||||
|
|
||||||
// Setting a separate data path won't pass the test as we don't sync
|
|
||||||
// it after creating new files,
|
|
||||||
if (option_config_ != kDifferentDataDir) {
|
|
||||||
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
|
||||||
// Since we don't sync WAL Dir, this test dosn't pass.
|
|
||||||
if (option_config_ != kWalDirSyncWal) {
|
|
||||||
PartialCompactTestReopenWithFault(kResetDropAndDeleteUnsynced,
|
|
||||||
num_pre_sync, num_post_sync);
|
|
||||||
}
|
|
||||||
NoWriteTestPreFault();
|
|
||||||
NoWriteTestReopenWithFault(kResetDropAndDeleteUnsynced);
|
|
||||||
|
|
||||||
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
||||||
// No new files created so we expect all values since no files will be
|
PartialCompactTestReopenWithFault(kResetDropUnsyncedData, num_pre_sync,
|
||||||
// dropped.
|
num_post_sync);
|
||||||
// WAL Dir is not synced for now.
|
|
||||||
if (option_config_ != kWalDir && option_config_ != kWalDirSyncWal) {
|
|
||||||
PartialCompactTestReopenWithFault(kResetDeleteUnsyncedFiles,
|
|
||||||
num_pre_sync + num_post_sync, 0);
|
|
||||||
}
|
|
||||||
NoWriteTestPreFault();
|
NoWriteTestPreFault();
|
||||||
NoWriteTestReopenWithFault(kResetDeleteUnsyncedFiles);
|
NoWriteTestReopenWithFault(kResetDropUnsyncedData);
|
||||||
|
|
||||||
|
// TODO(t6070540) Need to sync WAL Dir and other DB paths too.
|
||||||
|
|
||||||
|
// Setting a separate data path won't pass the test as we don't sync
|
||||||
|
// it after creating new files,
|
||||||
|
if (option_config_ != kDifferentDataDir) {
|
||||||
|
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
||||||
|
// Since we don't sync WAL Dir, this test dosn't pass.
|
||||||
|
if (option_config_ != kWalDirSyncWal) {
|
||||||
|
PartialCompactTestReopenWithFault(kResetDropAndDeleteUnsynced,
|
||||||
|
num_pre_sync, num_post_sync);
|
||||||
|
}
|
||||||
|
NoWriteTestPreFault();
|
||||||
|
NoWriteTestReopenWithFault(kResetDropAndDeleteUnsynced);
|
||||||
|
|
||||||
|
PartialCompactTestPreFault(num_pre_sync, num_post_sync);
|
||||||
|
// No new files created so we expect all values since no files will be
|
||||||
|
// dropped.
|
||||||
|
// WAL Dir is not synced for now.
|
||||||
|
if (option_config_ != kWalDir && option_config_ != kWalDirSyncWal) {
|
||||||
|
PartialCompactTestReopenWithFault(kResetDeleteUnsyncedFiles,
|
||||||
|
num_pre_sync + num_post_sync, 0);
|
||||||
|
}
|
||||||
|
NoWriteTestPreFault();
|
||||||
|
NoWriteTestReopenWithFault(kResetDeleteUnsyncedFiles);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
} while (ChangeOptions());
|
} while (ChangeOptions());
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user