Index: chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc |
diff --git a/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc b/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc |
index 7fb554c8ec06630e1fe065c064f99bf1f74a6d84..762240bfe1e8f72670675fee9e1a1c845f04bad5 100644 |
--- a/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc |
+++ b/chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc |
@@ -10,6 +10,8 @@ |
#include "chrome/browser/sync/test/integration/sync_integration_test_util.h" |
#include "chrome/browser/sync/test/integration/sync_test.h" |
#include "sync/internal_api/public/sessions/sync_session_snapshot.h" |
+#include "sync/test/fake_server/bookmark_entity_builder.h" |
+#include "sync/test/fake_server/entity_builder_factory.h" |
#include "ui/base/layout.h" |
using bookmarks_helper::AddFolder; |
@@ -39,10 +41,12 @@ using bookmarks_helper::SetURL; |
using bookmarks_helper::SortChildren; |
using passwords_helper::SetDecryptionPassphrase; |
using passwords_helper::SetEncryptionPassphrase; |
-using sync_integration_test_util::AwaitCommitActivityCompletion; |
using sync_integration_test_util::AwaitPassphraseAccepted; |
using sync_integration_test_util::AwaitPassphraseRequired; |
+using bookmarks_helper::AwaitAllModelsMatch; |
+using bookmarks_helper::AwaitAllModelsMatchVerifier; |
+ |
const std::string kGenericURL = "http://www.host.ext:1234/path/filename"; |
const std::string kGenericURLTitle = "URL Title"; |
const std::string kGenericFolderName = "Folder Name"; |
@@ -55,56 +59,44 @@ class TwoClientBookmarksSyncTest : public SyncTest { |
TwoClientBookmarksSyncTest() : SyncTest(TWO_CLIENT) {} |
virtual ~TwoClientBookmarksSyncTest() {} |
- private: |
- DISALLOW_COPY_AND_ASSIGN(TwoClientBookmarksSyncTest); |
-}; |
- |
-class LegacyTwoClientBookmarksSyncTest : public SyncTest { |
- public: |
- LegacyTwoClientBookmarksSyncTest() : SyncTest(TWO_CLIENT_LEGACY) {} |
- virtual ~LegacyTwoClientBookmarksSyncTest() {} |
+ virtual bool TestUsesSelfNotifications() OVERRIDE { return false; } |
private: |
- DISALLOW_COPY_AND_ASSIGN(LegacyTwoClientBookmarksSyncTest); |
+ DISALLOW_COPY_AND_ASSIGN(TwoClientBookmarksSyncTest); |
}; |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, Sanity) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL google_url("http://www.google.com"); |
ASSERT_TRUE(AddURL(0, "Google", google_url) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(1, "Yahoo", GURL("http://www.yahoo.com")) != NULL); |
- ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* new_folder = AddFolder(0, 2, "New Folder"); |
Move(0, GetUniqueNodeByURL(0, google_url), new_folder, 0); |
SetTitle(0, GetBookmarkBarNode(0)->GetChild(0), "Yahoo!!"); |
ASSERT_TRUE(AddURL(0, GetBookmarkBarNode(0), 1, "CNN", |
GURL("http://www.cnn.com")) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(1, "Facebook", GURL("http://www.facebook.com")) != NULL); |
- ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
SortChildren(1, GetBookmarkBarNode(1)); |
- ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
DisableVerifier(); |
SetTitle(0, GetUniqueNodeByURL(0, google_url), "Google++"); |
SetTitle(1, GetUniqueNodeByURL(1, google_url), "Google--"); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SimultaneousURLChanges) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL initial_url("http://www.google.com"); |
GURL second_url("http://www.google.com/abc"); |
@@ -112,46 +104,42 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SimultaneousURLChanges) { |
std::string title = "Google"; |
ASSERT_TRUE(AddURL(0, title, initial_url) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
DisableVerifier(); |
ASSERT_TRUE(SetURL( |
0, GetUniqueNodeByURL(0, initial_url), second_url) != NULL); |
ASSERT_TRUE(SetURL( |
1, GetUniqueNodeByURL(1, initial_url), third_url) != NULL); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
SetTitle(0, GetBookmarkBarNode(0)->GetChild(0), "Google1"); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
} |
// Test Scribe ID - 370558. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_AddFirstFolder) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddFolder(0, kGenericFolderName) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370559. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_AddFirstBMWithoutFavicon) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370489. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_AddFirstBMWithFavicon) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const GURL page_url(kGenericURL); |
const GURL icon_url("http://www.google.com/favicon.ico"); |
@@ -159,11 +147,10 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_AddFirstBMWithFavicon) { |
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, page_url); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
SetFavicon(0, bookmark, icon_url, CreateFavicon(SK_ColorWHITE), |
bookmarks_helper::FROM_UI); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test that the history service logic for not losing the hidpi versions of |
@@ -184,60 +171,55 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SetFaviconHiDPI) { |
const GURL icon_url1("http://www.google.com/favicon1.ico"); |
const GURL icon_url2("http://www.google.com/favicon2.ico"); |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* bookmark0 = AddURL(0, kGenericURLTitle, page_url); |
ASSERT_TRUE(bookmark0 != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
SetFavicon(0, bookmark0, icon_url1, CreateFavicon(SK_ColorWHITE), |
bookmarks_helper::FROM_UI); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* bookmark1 = GetUniqueNodeByURL(1, page_url); |
SetFavicon(1, bookmark1, icon_url1, CreateFavicon(SK_ColorBLUE), |
bookmarks_helper::FROM_UI); |
- ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
SetFavicon(0, bookmark0, icon_url2, CreateFavicon(SK_ColorGREEN), |
bookmarks_helper::FROM_UI); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370560. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_AddNonHTTPBMs) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL( |
0, "FTP UR", GURL("ftp://user:password@host:1234/path")) != NULL); |
ASSERT_TRUE(AddURL(0, "File UR", GURL("file://host/path")) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370561. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_AddFirstBMUnderFolder) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
ASSERT_TRUE(AddURL( |
0, folder, 0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370562. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_AddSeveralBMsUnderBMBarAndOtherBM) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 20; ++i) { |
std::string title = IndexedURLTitle(i); |
@@ -246,15 +228,14 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL( |
0, GetOtherNode(0), i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370563. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_AddSeveralBMsAndFolders) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 15; ++i) { |
if (base::RandDouble() > 0.6) { |
@@ -280,156 +261,139 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, GetOtherNode(0), i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370641. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DuplicateBMWithDifferentURLSameName) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url0 = GURL(IndexedURL(0)); |
GURL url1 = GURL(IndexedURL(1)); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, url0) != NULL); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, url1) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 370639 - Add bookmarks with different name and same URL. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DuplicateBookmarksWithSameURL) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title0 = IndexedURLTitle(0); |
std::string title1 = IndexedURLTitle(1); |
ASSERT_TRUE(AddURL(0, title0, GURL(kGenericURL)) != NULL); |
ASSERT_TRUE(AddURL(0, title1, GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371817. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_RenameBMName) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title = IndexedURLTitle(1); |
const BookmarkNode* bookmark = AddURL(0, title, GURL(kGenericURL)); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedURLTitle(2); |
SetTitle(0, bookmark, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371822. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_RenameBMURL) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url = GURL(IndexedURL(1)); |
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, url); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL new_url = GURL(IndexedURL(2)); |
ASSERT_TRUE(SetURL(0, bookmark, new_url) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371818 - Renaming the same bookmark name twice. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_TwiceRenamingBookmarkName) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title = IndexedURLTitle(1); |
const BookmarkNode* bookmark = AddURL(0, title, GURL(kGenericURL)); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedURLTitle(2); |
SetTitle(0, bookmark, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
SetTitle(0, bookmark, title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371823 - Renaming the same bookmark URL twice. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_TwiceRenamingBookmarkURL) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url = GURL(IndexedURL(1)); |
const BookmarkNode* bookmark = AddURL(0, kGenericURLTitle, url); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL new_url = GURL(IndexedURL(2)); |
ASSERT_TRUE(SetURL(0, bookmark, new_url) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(SetURL(0, bookmark, url) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371824. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_RenameBMFolder) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title = IndexedFolderName(1); |
const BookmarkNode* folder = AddFolder(0, title); |
ASSERT_TRUE(AddURL( |
0, folder, 0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedFolderName(2); |
SetTitle(0, folder, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371825. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_RenameEmptyBMFolder) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title = IndexedFolderName(1); |
const BookmarkNode* folder = AddFolder(0, title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedFolderName(2); |
SetTitle(0, folder, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371826. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_RenameBMFolderWithLongHierarchy) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string title = IndexedFolderName(1); |
const BookmarkNode* folder = AddFolder(0, title); |
@@ -444,20 +408,18 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddFolder(0, folder, i, title) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedFolderName(2); |
SetTitle(0, folder, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371827. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_RenameBMFolderThatHasParentAndChildren) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -478,76 +440,68 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddFolder(0, subfolder, i, title) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
std::string new_title = IndexedSubfolderName(2); |
SetTitle(0, subfolder, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371828. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_RenameBMNameAndURL) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url = GURL(IndexedURL(1)); |
std::string title = IndexedURLTitle(1); |
const BookmarkNode* bookmark = AddURL(0, title, url); |
ASSERT_TRUE(bookmark != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL new_url = GURL(IndexedURL(2)); |
std::string new_title = IndexedURLTitle(2); |
bookmark = SetURL(0, bookmark, new_url); |
ASSERT_TRUE(bookmark != NULL); |
SetTitle(0, bookmark, new_title); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371832. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DeleteBMEmptyAccountAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL( |
0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371833. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelBMNonEmptyAccountAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 20; ++i) { |
std::string title = IndexedURLTitle(i); |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371835. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelFirstBMUnderBMFoldNonEmptyFoldAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -556,19 +510,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, folder, 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371836. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelLastBMUnderBMFoldNonEmptyFoldAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -577,19 +529,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, folder, folder->child_count() - 1); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371856. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelMiddleBMUnderBMFoldNonEmptyFoldAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -598,19 +548,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, folder, 4); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371857. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelBMsUnderBMFoldEmptyFolderAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -619,37 +567,33 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
int child_count = folder->child_count(); |
for (int i = 0; i < child_count; ++i) { |
Remove(0, folder, 0); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371858. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelEmptyBMFoldEmptyAccountAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddFolder(0, kGenericFolderName) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371869. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelEmptyBMFoldNonEmptyAccountAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddFolder(0, kGenericFolderName) != NULL); |
for (int i = 1; i < 15; ++i) { |
@@ -662,19 +606,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddFolder(0, i, title) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371879. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelBMFoldWithBMsNonEmptyAccountAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
const BookmarkNode* folder = AddFolder(0, 1, kGenericFolderName); |
@@ -694,19 +636,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 1); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371880. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelBMFoldWithBMsAndBMFoldsNonEmptyACAfterwards) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
const BookmarkNode* folder = AddFolder(0, 1, kGenericFolderName); |
@@ -747,19 +687,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
} |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, GetBookmarkBarNode(0), 1); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371882. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_DelBMFoldWithParentAndChildrenBMsAndBMFolds) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -781,19 +719,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddFolder(0, subfolder, i, title) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Remove(0, folder, 0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371931. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_ReverseTheOrderOfTwoBMs) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url0 = GURL(IndexedURL(0)); |
GURL url1 = GURL(IndexedURL(1)); |
@@ -803,37 +739,33 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
const BookmarkNode* bookmark1 = AddURL(0, 1, title1, url1); |
ASSERT_TRUE(bookmark0 != NULL); |
ASSERT_TRUE(bookmark1 != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Move(0, bookmark0, GetBookmarkBarNode(0), 2); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371933. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_ReverseTheOrderOf10BMs) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 10; ++i) { |
std::string title = IndexedURLTitle(i); |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ReverseChildOrder(0, GetBookmarkBarNode(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371954. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_MovingBMsFromBMBarToBMFolder) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
const BookmarkNode* folder = AddFolder(0, 1, kGenericFolderName); |
@@ -843,24 +775,22 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
int num_bookmarks_to_move = |
GetBookmarkBarNode(0)->child_count() - 2; |
for (int i = 0; i < num_bookmarks_to_move; ++i) { |
Move( |
0, GetBookmarkBarNode(0)->GetChild(2), folder, i); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
} |
// Test Scribe ID - 371957. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_MovingBMsFromBMFoldToBMBar) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, kGenericURLTitle, GURL(kGenericURL)) != NULL); |
const BookmarkNode* folder = AddFolder(0, 1, kGenericFolderName); |
@@ -870,22 +800,20 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
int num_bookmarks_to_move = folder->child_count() - 2; |
for (int i = 0; i < num_bookmarks_to_move; ++i) { |
Move(0, folder->GetChild(0), GetBookmarkBarNode(0), i); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
} |
// Test Scribe ID - 371961. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_MovingBMsFromParentBMFoldToChildBMFold) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -902,22 +830,20 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i + 3)); |
ASSERT_TRUE(AddURL(0, subfolder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 3; ++i) { |
GURL url = GURL(IndexedURL(i)); |
Move(0, GetUniqueNodeByURL(0, url), subfolder, i + 10); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371964. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_MovingBMsFromChildBMFoldToParentBMFold) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
@@ -934,21 +860,19 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i + 3)); |
ASSERT_TRUE(AddURL(0, subfolder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 3; ++i) { |
GURL url = GURL(IndexedURL(i + 3)); |
Move(0, GetUniqueNodeByURL(0, url), folder, i + 4); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371967. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_HoistBMs10LevelUp) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = GetBookmarkBarNode(0); |
const BookmarkNode* folder_L0 = NULL; |
@@ -971,8 +895,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_HoistBMs10LevelUp) { |
GURL url = GURL(IndexedURL(i + 10)); |
ASSERT_TRUE(AddURL(0, folder_L10, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url10 = GURL(IndexedURL(10)); |
Move(0, GetUniqueNodeByURL( |
@@ -981,15 +904,14 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_HoistBMs10LevelUp) { |
Move(0, GetUniqueNodeByURL(0, url11), folder_L0, 0); |
GURL url12 = GURL(IndexedURL(12)); |
Move(0, GetUniqueNodeByURL(0, url12), folder_L0, 1); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371968. |
// Flaky. http://crbug.com/107744. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SinkBMs10LevelDown) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = GetBookmarkBarNode(0); |
const BookmarkNode* folder_L0 = NULL; |
@@ -1012,8 +934,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SinkBMs10LevelDown) { |
GURL url = GURL(IndexedURL(i + 10)); |
ASSERT_TRUE(AddURL(0, folder_L0, 0, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
GURL url10 = GURL(IndexedURL(10)); |
Move(0, GetUniqueNodeByURL(0, url10), folder_L10, folder_L10->child_count()); |
@@ -1021,15 +942,14 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SinkBMs10LevelDown) { |
Move(0, GetUniqueNodeByURL(0, url11), folder_L10, 0); |
GURL url12 = GURL(IndexedURL(12)); |
Move(0, GetUniqueNodeByURL(0, url12), folder_L10, 1); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371980. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_SinkEmptyBMFold5LevelsDown) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = GetBookmarkBarNode(0); |
const BookmarkNode* folder_L5 = NULL; |
@@ -1049,19 +969,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
folder = AddFolder( |
0, GetBookmarkBarNode(0)->child_count(), kGenericFolderName); |
ASSERT_TRUE(folder != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Move(0, folder, folder_L5, folder_L5->child_count()); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 371997. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_SinkNonEmptyBMFold5LevelsDown) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = GetBookmarkBarNode(0); |
const BookmarkNode* folder_L5 = NULL; |
@@ -1085,19 +1003,16 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Move(0, folder, folder_L5, folder_L5->child_count()); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 372006. |
-IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
- SC_HoistFolder5LevelsUp) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_HoistFolder5LevelsUp) { |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
const BookmarkNode* folder = GetBookmarkBarNode(0); |
const BookmarkNode* folder_L5 = NULL; |
@@ -1122,19 +1037,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
Move(0, folder, GetBookmarkBarNode(0), GetBookmarkBarNode(0)->child_count()); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 372026. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_ReverseTheOrderOfTwoBMFolders) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 2; ++i) { |
std::string title = IndexedFolderName(i); |
@@ -1146,19 +1059,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(0, folder, j, title, url) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ReverseChildOrder(0, GetBookmarkBarNode(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 372028. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SC_ReverseTheOrderOfTenBMFolders) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
for (int i = 0; i < 10; ++i) { |
std::string title = IndexedFolderName(i); |
@@ -1170,19 +1081,17 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(0, folder, j, title, url) != NULL); |
} |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ReverseChildOrder(0, GetBookmarkBarNode(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 373379. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_BiDirectionalPushAddingBM) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
DisableVerifier(); |
for (int i = 0; i < 2; ++i) { |
@@ -1193,16 +1102,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url1 = GURL(IndexedURL(2*i+1)); |
ASSERT_TRUE(AddURL(1, title1, url1) != NULL); |
} |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 373503. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_BiDirectionalPush_AddingSameBMs) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
// Note: When a racy commit is done with identical bookmarks, it is possible |
// for duplicates to exist after sync completes. See http://crbug.com/19769. |
@@ -1213,22 +1121,20 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(0, title, url) != NULL); |
ASSERT_TRUE(AddURL(1, title, url) != NULL); |
} |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
} |
// Test Scribe ID - 373506. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_BootStrapEmptyStateEverywhere) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
// Test Scribe ID - 373505. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_Merge_CaseInsensitivity_InNames) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folder0 = AddFolder(0, "Folder"); |
@@ -1243,16 +1149,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, folder1, 1, "BooKMarK 1", GURL(kGenericURL)) != NULL); |
ASSERT_TRUE(AddURL(1, folder1, 2, "bOOKMARK 2", GURL(kGenericURL)) != NULL); |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 373508. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_SimpleMergeOfDifferentBMModels) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 3; ++i) { |
@@ -1271,16 +1176,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, i, title1, url1) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 386586. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeSimpleBMHierarchyUnderBMBar) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 3; ++i) { |
@@ -1296,16 +1200,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, i, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 386589. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeSimpleBMHierarchyEqualSetsUnderBMBar) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 3; ++i) { |
@@ -1315,16 +1218,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, i, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 373504 - Merge bookmark folders with different bookmarks. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeBMFoldersWithDifferentBMs) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folder0 = AddFolder(0, kGenericFolderName); |
@@ -1339,16 +1241,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url1 = GURL(IndexedURL(2*i+1)); |
ASSERT_TRUE(AddURL(1, folder1, i, title1, url1) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// Test Scribe ID - 373509 - Merge moderately complex bookmark models. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeDifferentBMModelsModeratelyComplex) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 25; ++i) { |
@@ -1381,16 +1282,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(0, title, url) != NULL); |
ASSERT_TRUE(AddURL(1, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// TCM ID - 3675271 - Merge simple bookmark subset under bookmark folder. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeSimpleBMHierarchySubsetUnderBMFolder) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 2; ++i) { |
@@ -1407,16 +1307,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
} |
} |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// TCM ID - 3727284 - Merge subsets of bookmark under bookmark bar. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_MergeSimpleBMHierarchySubsetUnderBookmarkBar) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 4; ++i) { |
@@ -1431,9 +1330,8 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, j, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
ASSERT_FALSE(ContainsDuplicateBookmarks(1)); |
} |
@@ -1441,7 +1339,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// TCM ID - 3659294 - Merge simple bookmark hierarchy under bookmark folder. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_Merge_SimpleBMHierarchy_Under_BMFolder) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folder0 = AddFolder(0, 0, kGenericFolderName); |
@@ -1463,9 +1361,8 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL( |
1, folder1, 4, IndexedURLTitle(1), GURL(IndexedURL(1))) != NULL); |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
@@ -1473,7 +1370,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// folder. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_Merge_SimpleBMHierarchy_DisjointSets_Under_BMFolder) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folder0 = |
@@ -1495,9 +1392,8 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL( |
1, folder1, 3, IndexedURLTitle(8), GURL(IndexedURL(8))) != NULL); |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
@@ -1505,7 +1401,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// bar. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_Merge_SimpleBMHierarchy_DisjointSets_Under_BookmarkBar) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
for (int i = 0; i < 3; ++i) { |
@@ -1520,16 +1416,15 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(0, j, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// TCM ID - 3616282 - Merge sets of duplicate bookmarks under bookmark bar. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_Merge_SimpleBMHierarchy_DuplicateBMs_Under_BMBar) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
// Let's add duplicate set of bookmark {1,2,2,3,3,3,4,4,4,4} to client0. |
@@ -1549,52 +1444,18 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AddURL(1, i, title, url) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
for (int i = 1; i < 5 ; ++i) { |
ASSERT_TRUE(CountBookmarksWithTitlesMatching(1, IndexedURLTitle(i)) == i); |
} |
} |
-// TCM ID - 6593872. |
-IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, DisableBookmarks) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
- |
- ASSERT_TRUE(GetClient(1)->DisableSyncForDatatype(syncer::BOOKMARKS)); |
- ASSERT_TRUE(AddFolder(1, kGenericFolderName) != NULL); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_FALSE(AllModelsMatch()); |
- |
- ASSERT_TRUE(GetClient(1)->EnableSyncForDatatype(syncer::BOOKMARKS)); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
-} |
- |
-// TCM ID - 7343544. |
-IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, DisableSync) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
- |
- ASSERT_TRUE(GetClient(1)->DisableSyncForAllDatatypes()); |
- ASSERT_TRUE(AddFolder(0, IndexedFolderName(0)) != NULL); |
- ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); |
- ASSERT_FALSE(AllModelsMatch()); |
- |
- ASSERT_TRUE(AddFolder(1, IndexedFolderName(1)) != NULL); |
- ASSERT_FALSE(AllModelsMatch()); |
- |
- ASSERT_TRUE(GetClient(1)->EnableSyncForAllDatatypes()); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
-} |
- |
// TCM ID - 3662298 - Test adding duplicate folder - Both with different BMs |
// underneath. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, MC_DuplicateFolders) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folder0 = AddFolder(0, kGenericFolderName); |
@@ -1610,15 +1471,14 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, MC_DuplicateFolders) { |
ASSERT_TRUE(AddURL(1, folder1, i, title1, url1) != NULL); |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
// This test fails when run with FakeServer and FakeServerInvalidationService. |
-IN_PROC_BROWSER_TEST_F(LegacyTwoClientBookmarksSyncTest, MC_DeleteBookmark) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
+IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, MC_DeleteBookmark) { |
+ ASSERT_TRUE(SetupSync()); |
ASSERT_TRUE(GetClient(1)->DisableSyncForDatatype(syncer::BOOKMARKS)); |
const GURL bar_url("http://example.com/bar"); |
@@ -1627,21 +1487,18 @@ IN_PROC_BROWSER_TEST_F(LegacyTwoClientBookmarksSyncTest, MC_DeleteBookmark) { |
ASSERT_TRUE(AddURL(0, GetBookmarkBarNode(0), 0, "bar", bar_url) != NULL); |
ASSERT_TRUE(AddURL(0, GetOtherNode(0), 0, "other", other_url) != NULL); |
- ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); |
- |
ASSERT_TRUE(HasNodeWithURL(0, bar_url)); |
ASSERT_TRUE(HasNodeWithURL(0, other_url)); |
ASSERT_FALSE(HasNodeWithURL(1, bar_url)); |
ASSERT_FALSE(HasNodeWithURL(1, other_url)); |
Remove(0, GetBookmarkBarNode(0), 0); |
- ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); |
ASSERT_FALSE(HasNodeWithURL(0, bar_url)); |
ASSERT_TRUE(HasNodeWithURL(0, other_url)); |
ASSERT_TRUE(GetClient(1)->EnableSyncForDatatype(syncer::BOOKMARKS)); |
- ASSERT_TRUE(AwaitQuiescence()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_FALSE(HasNodeWithURL(0, bar_url)); |
ASSERT_TRUE(HasNodeWithURL(0, other_url)); |
@@ -1653,7 +1510,7 @@ IN_PROC_BROWSER_TEST_F(LegacyTwoClientBookmarksSyncTest, MC_DeleteBookmark) { |
// from two clients at the same time. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_BookmarkNameChangeConflict) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
+ ASSERT_TRUE(SetupSync()); |
const BookmarkNode* folder0 = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder0 != NULL); |
@@ -1662,8 +1519,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder0, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
DisableVerifier(); |
@@ -1671,8 +1527,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SetTitle(0, GetUniqueNodeByURL(0, url), "Title++"); |
SetTitle(1, GetUniqueNodeByURL(1, url), "Title--"); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
@@ -1680,7 +1535,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// from two clients at the same time. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_BookmarkURLChangeConflict) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
+ ASSERT_TRUE(SetupSync()); |
const BookmarkNode* folder0 = AddFolder(0, kGenericFolderName); |
ASSERT_TRUE(folder0 != NULL); |
@@ -1689,8 +1544,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
GURL url = GURL(IndexedURL(i)); |
ASSERT_TRUE(AddURL(0, folder0, i, title, url) != NULL); |
} |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
DisableVerifier(); |
@@ -1700,8 +1554,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(SetURL( |
1, GetUniqueNodeByURL(1, url), GURL("http://www.google.com/11"))); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
@@ -1709,7 +1562,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// clients at the same time. |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
MC_FolderNameChangeConflict) { |
- ASSERT_TRUE(SetupClients()) << "SetupClients() failed."; |
+ ASSERT_TRUE(SetupClients()); |
DisableVerifier(); |
const BookmarkNode* folderA[2]; |
@@ -1749,127 +1602,110 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
} |
} |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
// Simultaneously rename folder A on both clients. |
SetTitle(0, folderA[0], "Folder A++"); |
SetTitle(1, folderA[1], "Folder A--"); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
// Simultaneously rename folder B on both clients. |
SetTitle(0, folderB[0], "Folder B++"); |
SetTitle(1, folderB[1], "Folder B--"); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
// Simultaneously rename folder C on both clients. |
SetTitle(0, folderC[0], "Folder C++"); |
SetTitle(1, folderC[1], "Folder C--"); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_FALSE(ContainsDuplicateBookmarks(0)); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SingleClientEnabledEncryption) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
- |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(EnableEncryption(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(IsEncryptionComplete(0)); |
- ASSERT_TRUE(IsEncryptionComplete(1)); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitEncryptionComplete(1)); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SingleClientEnabledEncryptionAndChanged) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(EnableEncryption(0)); |
ASSERT_TRUE(AddURL(0, IndexedURLTitle(0), GURL(IndexedURL(0))) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(IsEncryptionComplete(0)); |
- ASSERT_TRUE(IsEncryptionComplete(1)); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ |
+ ASSERT_TRUE(AwaitEncryptionComplete(1)); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
BothClientsEnabledEncryption) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(EnableEncryption(0)); |
ASSERT_TRUE(EnableEncryption(1)); |
- ASSERT_TRUE(AwaitQuiescence()); |
ASSERT_TRUE(IsEncryptionComplete(0)); |
ASSERT_TRUE(IsEncryptionComplete(1)); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SingleClientEnabledEncryptionBothChanged) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(EnableEncryption(0)); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(IsEncryptionComplete(0)); |
- ASSERT_TRUE(IsEncryptionComplete(1)); |
+ ASSERT_TRUE(AwaitEncryptionComplete(1)); |
+ |
ASSERT_TRUE(AddURL(0, IndexedURLTitle(0), GURL(IndexedURL(0))) != NULL); |
ASSERT_TRUE(AddURL(0, IndexedURLTitle(1), GURL(IndexedURL(1))) != NULL); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(IsEncryptionComplete(0)); |
ASSERT_TRUE(IsEncryptionComplete(1)); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
SingleClientEnabledEncryptionAndChangedMultipleTimes) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, IndexedURLTitle(0), GURL(IndexedURL(0))) != NULL); |
ASSERT_TRUE(EnableEncryption(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(IsEncryptionComplete(0)); |
- ASSERT_TRUE(IsEncryptionComplete(1)); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitEncryptionComplete(1)); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
ASSERT_TRUE(AddURL(0, IndexedURLTitle(1), GURL(IndexedURL(1))) != NULL); |
ASSERT_TRUE(AddFolder(0, IndexedFolderName(0)) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
FirstClientEnablesEncryptionWithPassSecondChanges) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
// Add initial bookmarks. |
ASSERT_TRUE(AddURL(0, 0, IndexedURLTitle(0), GURL(IndexedURL(0))) != NULL); |
ASSERT_TRUE(AddURL(0, 1, IndexedURLTitle(1), GURL(IndexedURL(1))) != NULL); |
ASSERT_TRUE(AddURL(0, 2, IndexedURLTitle(2), GURL(IndexedURL(2))) != NULL); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
// Set a passphrase and enable encryption on Client 0. Client 1 will not |
// understand the bookmark updates. |
SetEncryptionPassphrase(0, kValidPassphrase, ProfileSyncService::EXPLICIT); |
ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService((0)))); |
ASSERT_TRUE(EnableEncryption(0)); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(IsEncryptionComplete(0)); |
- ASSERT_TRUE(IsEncryptionComplete(1)); |
- ASSERT_TRUE(GetSyncService((1))->IsPassphraseRequired()); |
+ ASSERT_TRUE(AwaitEncryptionComplete(1)); |
// Client 1 adds bookmarks between the first two and between the second two. |
ASSERT_TRUE(AddURL(0, 1, IndexedURLTitle(3), GURL(IndexedURL(3))) != NULL); |
@@ -1881,15 +1717,13 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE(AwaitPassphraseRequired(GetSyncService((1)))); |
ASSERT_TRUE(SetDecryptionPassphrase(1, kValidPassphrase)); |
ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService((1)))); |
- ASSERT_TRUE(AwaitQuiescence()); |
- EXPECT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
ASSERT_EQ(0, |
GetClient(1)->GetLastSessionSnapshot().num_encryption_conflicts()); |
// Ensure everything is syncing normally by appending a final bookmark. |
ASSERT_TRUE(AddURL(1, 5, IndexedURLTitle(5), GURL(IndexedURL(5))) != NULL); |
- ASSERT_TRUE(GetClient(1)->AwaitMutualSyncCycleCompletion(GetClient(0))); |
- EXPECT_TRUE(AllModelsMatch()); |
+ EXPECT_TRUE(AwaitAllModelsMatch()); |
ASSERT_EQ(0, |
GetClient(1)->GetLastSessionSnapshot().num_encryption_conflicts()); |
} |
@@ -1898,7 +1732,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
// code results in a consistent view across machines (no matter what the final |
// order is). |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, RacyPositionChanges) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
+ ASSERT_TRUE(SetupSync()); |
ASSERT_TRUE(AllModelsMatchVerifier()); |
// Add initial bookmarks. |
@@ -1908,8 +1742,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, RacyPositionChanges) { |
} |
// Once we make diverging changes the verifer is helpless. |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
DisableVerifier(); |
// Make changes on client 0. |
@@ -1930,8 +1763,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, RacyPositionChanges) { |
Move(1, node, node->parent(), rand_pos); |
} |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
// Now make changes to client 1 first. |
for (size_t i = 0; i < num_bookmarks; ++i) { |
@@ -1951,41 +1783,33 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, RacyPositionChanges) { |
Move(0, node, node->parent(), rand_pos); |
} |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
} |
// Trigger the server side creation of Synced Bookmarks. Ensure both clients |
// remain syncing afterwards. Add bookmarks to the synced bookmarks folder |
// and ensure both clients receive the boomkmark. |
-IN_PROC_BROWSER_TEST_F(LegacyTwoClientBookmarksSyncTest, |
- CreateSyncedBookmarks) { |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, CreateSyncedBookmarks) { |
+ std::string title = "Montreal Canadiens"; |
+ fake_server::EntityBuilderFactory entity_builder_factory; |
+ scoped_ptr<fake_server::FakeServerEntity> entity = |
+ entity_builder_factory.NewBookmarkEntityBuilder( |
+ title, GURL("http://canadiens.nhl.com")) |
+ .Build(); |
+ fake_server_->InjectEntity(entity.Pass()); |
- TriggerCreateSyncedBookmarks(); |
- |
- // Add a bookmark on Client 0 and ensure it syncs over. This will also trigger |
- // both clients downloading the new Synced Bookmarks folder. |
- ASSERT_TRUE(AddURL(0, "Google", GURL("http://www.google.com"))); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ DisableVerifier(); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
- // Now add a bookmark within the Synced Bookmarks folder and ensure it syncs |
- // over. |
- const BookmarkNode* synced_bookmarks = GetSyncedBookmarksNode(0); |
- ASSERT_TRUE(synced_bookmarks); |
- ASSERT_TRUE(AddURL(0, synced_bookmarks, 0, "Google2", |
- GURL("http://www.google2.com"))); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_EQ(1, CountBookmarksWithTitlesMatching(0, title)); |
+ ASSERT_EQ(1, CountBookmarksWithTitlesMatching(1, title)); |
} |
IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
BookmarkAllNodesRemovedEvent) { |
- |
- ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; |
- ASSERT_TRUE(AllModelsMatchVerifier()); |
+ ASSERT_TRUE(SetupSync()); |
+ ASSERT_TRUE(AwaitAllModelsMatchVerifier()); |
// Starting state: |
// other_node |
@@ -2018,15 +1842,14 @@ IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, |
ASSERT_TRUE( |
AddURL(0, GetBookmarkBarNode(0), 2, "Gmai", GURL("http://gmail.com"))); |
- ASSERT_TRUE(AwaitQuiescence()); |
- ASSERT_TRUE(AllModelsMatch()); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
// Remove all |
RemoveAll(0); |
- ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); |
+ ASSERT_TRUE(AwaitAllModelsMatch()); |
+ |
// Verify other node has no children now. |
EXPECT_EQ(0, GetOtherNode(0)->child_count()); |
EXPECT_EQ(0, GetBookmarkBarNode(0)->child_count()); |
- ASSERT_TRUE(AllModelsMatch()); |
} |