Commit bfdb88aa authored by Rushan Suleymanov's avatar Rushan Suleymanov Committed by Commit Bot

[Sync] Disable an integration test for the SharingMessage.

The test should check that the sharing message will be sent to the
server again if there was AUTH_ERROR in the last request. However the
SharingMessage data type doesn't always generate data for the next
commit requests.

This CL disables the test until it gets fixed because it might fail in
other CLs.

Bug: 1097054
Change-Id: I6d0b952026e1cdab9f6e8ff51f9b4c86724ec1e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460736
Commit-Queue: Marc Treib <treib@chromium.org>
Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Auto-Submit: Rushan Suleymanov <rushans@google.com>
Cr-Commit-Position: refs/heads/master@{#815183}
parent 2941eac4
...@@ -279,9 +279,12 @@ IN_PROC_BROWSER_TEST_F( ...@@ -279,9 +279,12 @@ IN_PROC_BROWSER_TEST_F(
EXPECT_TRUE(DisabledSharingMessageChecker(GetSyncService(0)).Wait()); EXPECT_TRUE(DisabledSharingMessageChecker(GetSyncService(0)).Wait());
} }
// TODO(crbug.com/1097054): enable the test once the issue is fixed, without
// that it may be flaky because SharingMessage data type might not generate
// retries.
IN_PROC_BROWSER_TEST_F( IN_PROC_BROWSER_TEST_F(
SingleClientSharingMessageSyncTest, SingleClientSharingMessageSyncTest,
ShouldRetrySendingSharingMessageDataTypeOnTransientAuthError) { DISABLED_ShouldRetrySendingSharingMessageDataTypeOnTransientAuthError) {
ASSERT_TRUE(SetupSync()); ASSERT_TRUE(SetupSync());
GetFakeServer()->SetHttpError(net::HTTP_UNAUTHORIZED); GetFakeServer()->SetHttpError(net::HTTP_UNAUTHORIZED);
SetOAuth2TokenResponse(kEmptyOAuth2Token, net::HTTP_INTERNAL_SERVER_ERROR, SetOAuth2TokenResponse(kEmptyOAuth2Token, net::HTTP_INTERNAL_SERVER_ERROR,
......
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