Only needs to work on USS.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/148e5e3c9eb4a821b39913b2673b6842adc4b7bc commit 148e5e3c9eb4a821b39913b2673b6842adc4b7bc Author: skym <skym@chromium.org> Date: Thu Jun 29 20:13:27 2017 [Sync] Sanity test for UserEvents. BUG= 728753 Review-Url: https://codereview.chromium.org/2948303002 Cr-Commit-Position: refs/heads/master@{#483460} [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/chrome/browser/sync/test/integration/migration_test.cc [add] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/chrome/browser/sync/test/integration/single_client_user_events_sync_test.cc [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/chrome/test/BUILD.gn [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/components/sync/test/fake_server/fake_server.cc [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/components/sync/test/fake_server/unique_client_entity.cc [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/components/sync/test/fake_server/unique_client_entity.h [modify] https://crrev.com/148e5e3c9eb4a821b39913b2673b6842adc4b7bc/testing/variations/fieldtrial_testing_config.json
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0ff047cc8c4e9b8bcadf0ab951781f253afc3a12 commit 0ff047cc8c4e9b8bcadf0ab951781f253afc3a12 Author: skym <skym@chromium.org> Date: Fri Jul 07 15:46:47 2017 [Sync] Added integ tests to verify UserEvents retry on TRANSIENT_ERROR. BUG= 728753 Review-Url: https://codereview.chromium.org/2973833002 Cr-Commit-Position: refs/heads/master@{#484939} [modify] https://crrev.com/0ff047cc8c4e9b8bcadf0ab951781f253afc3a12/chrome/browser/sync/test/integration/single_client_user_events_sync_test.cc [modify] https://crrev.com/0ff047cc8c4e9b8bcadf0ab951781f253afc3a12/components/sync/test/fake_server/fake_server.cc [modify] https://crrev.com/0ff047cc8c4e9b8bcadf0ab951781f253afc3a12/components/sync/test/fake_server/fake_server.h
Need to add tests that verify we can return TRANSIENT_ERROR for USER_EVENTS, but other types are still committing successfully while USER_EVENTS is in backoff. See issue 750142 .
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/dec5aed76404ee950893480c4ad3d27a1cd6f4e1 commit dec5aed76404ee950893480c4ad3d27a1cd6f4e1 Author: Sky Malice <skym@chromium.org> Date: Thu Aug 10 20:23:06 2017 [Sync] Add unit/integ tests for suppressed UserEvents. This is in part a follow up CL to add tests for https://chromium-review.googlesource.com/c/610343/ Bug: 728753 Change-Id: Iaba4225389e3005f0df9b9ab37b9005b392d4328 Reviewed-on: https://chromium-review.googlesource.com/610620 Reviewed-by: Patrick Noland <pnoland@google.com> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#493507} [modify] https://crrev.com/dec5aed76404ee950893480c4ad3d27a1cd6f4e1/chrome/browser/sync/test/integration/single_client_user_events_sync_test.cc [modify] https://crrev.com/dec5aed76404ee950893480c4ad3d27a1cd6f4e1/components/sync/user_events/user_event_service_impl.cc [modify] https://crrev.com/dec5aed76404ee950893480c4ad3d27a1cd6f4e1/components/sync/user_events/user_event_service_impl_unittest.cc
Integrat
Comment 1 by bugdroid1@chromium.org
, Jun 29 2017