Commit 9bb4073b authored by Stephen McGruer's avatar Stephen McGruer Committed by Commit Bot

[Sheriff] Disable BookmarksCommandManagerTest.All on Linux Tests (dbg)

Because this file has multiple tests with the same name ('All') and an
existing MAYBE_All macro, rename the test to 'AllBCM' to avoid macro
collision.

TBR=tommycli@chromium.org

Bug: 1010381
Change-Id: I475153dc2ec05fb737fe59a7568c619ebfdc68da
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1878530Reviewed-by: default avatarStephen McGruer <smcgruer@chromium.org>
Reviewed-by: default avatarJeremy Roman <jbroman@chromium.org>
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709052}
parent cf75117d
...@@ -72,7 +72,14 @@ BookmarksCommandManagerTest.prototype = { ...@@ -72,7 +72,14 @@ BookmarksCommandManagerTest.prototype = {
]), ]),
}; };
TEST_F('BookmarksCommandManagerTest', 'All', function() { // https://crbug.com/1010381: Flaky on Linux Tests (dbg)
GEN('#if defined(OS_LINUX) && !defined(NDEBUG)');
GEN('#define MAYBE_AllBCM DISABLED_AllBCM');
GEN('#else');
GEN('#define MAYBE_AllBCM AllBCM');
GEN('#endif');
TEST_F('BookmarksCommandManagerTest', 'MAYBE_AllBCM', function() {
mocha.run(); mocha.run();
}); });
......
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