Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(17)

Unified Diff: components/sync/test/directory_backing_store_corruption_testing.cc

Issue 2424533002: Modify sync unit test to use standard sql::test:: helper. (Closed)
Patch Set: BUILD.gn merge Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/sync/test/directory_backing_store_corruption_testing.h ('k') | sql/test/test_helpers.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/test/directory_backing_store_corruption_testing.cc
diff --git a/components/sync/test/directory_backing_store_corruption_testing.cc b/components/sync/test/directory_backing_store_corruption_testing.cc
deleted file mode 100644
index 5f7fdb8c2f0a4a739d818962e8368244c592f537..0000000000000000000000000000000000000000
--- a/components/sync/test/directory_backing_store_corruption_testing.cc
+++ /dev/null
@@ -1,39 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "components/sync/test/directory_backing_store_corruption_testing.h"
-
-#include <string>
-
-#include "base/files/file_util.h"
-#include "base/files/scoped_file.h"
-
-namespace syncer {
-namespace syncable {
-namespace corruption_testing {
-
-// This value needs to be large enough to force the underlying DB to be read
-// from disk before writing, but small enough so that tests don't take too long
-// and timeout. The value depend on the underlying DB page size as well as the
-// DB's cache_size PRAGMA. If test fails, you either increase
-// kNumEntriesRequiredForCorruption, or increase the size of each entry.
-const int kNumEntriesRequiredForCorruption = 2000;
-
-bool CorruptDatabase(const base::FilePath& backing_file_path) {
- // Corrupt the DB by write a bunch of zeros at the beginning.
- //
- // Because the file may already open for writing, it's important that we open
- // it in a sharing-compatible way for platforms that have the concept of
- // shared/exclusive file access (e.g. Windows).
- base::ScopedFILE db_file(base::OpenFile(backing_file_path, "wb"));
- if (!db_file.get())
- return false;
- const std::string zeros(4096, '\0');
- const int num_written = fwrite(zeros.data(), zeros.size(), 1, db_file.get());
- return num_written == 1U;
-}
-
-} // namespace corruption_testing
-} // namespace syncable
-} // namespace syncer
« no previous file with comments | « components/sync/test/directory_backing_store_corruption_testing.h ('k') | sql/test/test_helpers.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698