Commit 833dec2d authored by Boris Sazonov's avatar Boris Sazonov Committed by Commit Bot

[Signin][Android] Add waitForIdleSync and asserts to ManageSyncSettingsTest

This test has been flaky, so this CL adds waitForIdleSync to wait for
UI to settle. It also adds an assert to verify that "Sync everything"
toggle is turned off.

Bug: 1059116
Change-Id: I3955eaa187c5ce9f57ac715fc416e9b88a5ac461
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108555Reviewed-by: default avatarAlice Wang <aliceywang@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756296}
parent 36c8ca99
......@@ -111,8 +111,12 @@ public class ManageSyncSettingsTest {
ChromeSwitchPreference syncEverything = getSyncEverything(fragment);
Map<Integer, CheckBoxPreference> dataTypes = getDataTypes(fragment);
InstrumentationRegistry.getInstrumentation().waitForIdleSync();
assertSyncOnState(fragment);
mSyncTestRule.togglePreference(syncEverything);
InstrumentationRegistry.getInstrumentation().waitForIdleSync();
Assert.assertFalse(syncEverything.isChecked());
for (CheckBoxPreference dataType : dataTypes.values()) {
Assert.assertTrue(dataType.isChecked());
Assert.assertTrue(dataType.isEnabled());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment