Chromium Code Reviews
DescriptionRevert of Disable flaky MigrationSingleClientTest tests. (patchset #1 id:1 of https://codereview.chromium.org/2353953002/ )
Reason for revert:
No need to disable these tests as the flakyness was fixed by https://codereview.chromium.org/2338213009
Original issue's description:
> Disable flaky MigrationSingleClientTest tests.
>
> These are reported as flaky. Though locally they fail every time?
>
> Disabling:
> - MigrationSingleClientTest.AllTypesAtOnce
> - MigrationSingleClientTest.AllTypesAtOnceTriggerNotification
> - MigrationSingleClientTest.AllTypesWithNigoriAtOnce
> - MigrationSingleClientTest.BookmarksPrefsBoth
> - MigrationSingleClientTest.BookmarksPrefsIndividually
> - MigrationSingleClientTest.NigoriOnly
> - MigrationSingleClientTest.PrefsNigoriBoth
> - MigrationSingleClientTest.PrefsOnlyModifyBookmark
> - MigrationSingleClientTest.PrefsOnlyModifyPref
> - MigrationSingleClientTest.PrefsOnlyTriggerNotification
>
> With this, *ALL* MigrationSingleClientTests that were not already
> disabled are disabled. This bug needs to be investigated and the tests
> re-enabled.
>
> BUG=648480
> NOTRY=True
> TBR=lgcheng,stevenjb,pavely
>
> Committed: https://crrev.com/76ad59ea95d48f29f5b8658d467d6323789f1b51
> Cr-Commit-Position: refs/heads/master@{#419757}
TBR=lgcheng@google.com,pavely@chromium.org,stevenjb@chromium.org,hbos@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648480
Committed: https://crrev.com/f58e5fea14e496cc7bf2b4af814e1c4726b07f87
Cr-Commit-Position: refs/heads/master@{#419766}
Patch Set 1 #
Messages
Total messages: 6 (2 generated)
|
|||||||||||||||||||