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

Unified Diff: chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc

Issue 2379433002: [Sync] Refactoring of sync integration test checkers to remove boilerplate await methods. (Closed)
Patch Set: Rebase 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
Index: chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc b/chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc
index f6a5a61f2212920a0c3380a5d8bf8632598b3e84..b6a32dc5676eea7f0752950228054bd2dd492a68 100644
--- a/chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc
+++ b/chrome/browser/sync/test/integration/single_client_wifi_credentials_sync_test.cc
@@ -6,15 +6,14 @@
#include "base/command_line.h"
#include "base/macros.h"
-#include "chrome/browser/sync/test/integration/sync_integration_test_util.h"
#include "chrome/browser/sync/test/integration/sync_test.h"
+#include "chrome/browser/sync/test/integration/updated_progress_marker_checker.h"
#include "chrome/browser/sync/test/integration/wifi_credentials_helper.h"
#include "chrome/common/chrome_switches.h"
#include "components/browser_sync/browser_sync_switches.h"
#include "components/wifi_sync/wifi_credential.h"
#include "components/wifi_sync/wifi_security_class.h"
-using sync_integration_test_util::AwaitCommitActivityCompletion;
using wifi_sync::WifiCredential;
using WifiCredentialSet = wifi_sync::WifiCredential::CredentialSet;
@@ -71,6 +70,7 @@ IN_PROC_BROWSER_TEST_F(SingleClientWifiCredentialsSyncTest, SingleCredential) {
EXPECT_EQ(WifiCredential::MakeSsidBytesForTest(ssid),
verifier_credentials.begin()->ssid());
- ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(profile_index)));
+ ASSERT_TRUE(
+ UpdatedProgressMarkerChecker(GetSyncService(profile_index)).Wait());
EXPECT_TRUE(wifi_credentials_helper::ProfileMatchesVerifier(profile_index));
}

Powered by Google App Engine
This is Rietveld 408576698