Commit f0eb7e08 authored by Noel Gordon's avatar Noel Gordon Committed by Commit Bot

Revert "Disable more CopyBetweenWindows tests."

This reverts commit 5a12eea7.

Reason for revert: The fixture is a timeout/pass in DEBUG only, but 
this CL disabled test on all bots: RELEASE/MSAN/ASAN.  So can we
please not do that?  Thank you.

Original change's description:
> Disable more CopyBetweenWindows tests.
> 
> Similar to copyBetweenWindowsDriveToUsb, copyBetweenWindowsLocalToUsb
> and other related tests are also flaky. Disable them all.
> 
> TBR=austinct@chromium.org
> 
> Bug: 1008909
> Change-Id: Ib87e1403a653fa599111faa38b30947ca0fbf216
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1857650
> Reviewed-by: Lei Zhang <thestig@chromium.org>
> Commit-Queue: Lei Zhang <thestig@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#705367}

TBR=thestig@chromium.org,austinct@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1008909
Change-Id: Iabbc58b3c23781b51d1b1fd3bc41df5d635c9f4a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1859414Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Reviewed-by: default avatarAustin Tankiang <austinct@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#705473}
parent ecfdd411
...@@ -680,11 +680,16 @@ WRAPPED_INSTANTIATE_TEST_SUITE_P( ...@@ -680,11 +680,16 @@ WRAPPED_INSTANTIATE_TEST_SUITE_P(
WRAPPED_INSTANTIATE_TEST_SUITE_P( WRAPPED_INSTANTIATE_TEST_SUITE_P(
DISABLED_CopyBetweenWindows, /* copy_between_windows.js */ DISABLED_CopyBetweenWindows, /* copy_between_windows.js */
FilesAppBrowserTest, FilesAppBrowserTest,
::testing::Values(TestCase("copyBetweenWindowsDriveToUsb")));
WRAPPED_INSTANTIATE_TEST_SUITE_P(
CopyBetweenWindows, /* copy_between_windows.js */
FilesAppBrowserTest,
::testing::Values(TestCase("copyBetweenWindowsLocalToDrive"), ::testing::Values(TestCase("copyBetweenWindowsLocalToDrive"),
TestCase("copyBetweenWindowsLocalToUsb"), TestCase("copyBetweenWindowsLocalToUsb"),
TestCase("copyBetweenWindowsUsbToDrive"), TestCase("copyBetweenWindowsUsbToDrive"),
TestCase("copyBetweenWindowsDriveToLocal"), TestCase("copyBetweenWindowsDriveToLocal"),
TestCase("copyBetweenWindowsDriveToUsb"), // TestCase("copyBetweenWindowsDriveToUsb"),
TestCase("copyBetweenWindowsUsbToLocal"))); TestCase("copyBetweenWindowsUsbToLocal")));
WRAPPED_INSTANTIATE_TEST_SUITE_P( WRAPPED_INSTANTIATE_TEST_SUITE_P(
......
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