From de2654dfdf37abc35237ab97f5fd7c2141710282 Mon Sep 17 00:00:00 2001 From: Francis McCabe Date: Thu, 29 Aug 2019 14:44:58 +0000 Subject: [PATCH] Revert "[wasm-c-api][test] Run Wasm C API tests on bots" This reverts commit a5d279da00738fe648e1341d81c116f771ec0dd7. Reason for revert: Causing failures in UBSAN Linux See https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20UBSan/7672 Original change's description: > [wasm-c-api][test] Run Wasm C API tests on bots > > by including them in the "bot_default" and "default" test sets. > The build targets are already up to date, only the test runner > needs to be updated. > > Change-Id: I06a4a35a8d00c25ab56874d8eb365418841a02ac > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1768370 > Commit-Queue: Tamer Tas > Reviewed-by: Tamer Tas > Cr-Commit-Position: refs/heads/master@{#63436} TBR=jkummerow@chromium.org,tmrts@chromium.org Change-Id: I5b78e45eeae11ce460d2cdecba4e3cf8eacdb4f8 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1775927 Reviewed-by: Francis McCabe Commit-Queue: Francis McCabe Cr-Commit-Position: refs/heads/master@{#63454} --- tools/testrunner/base_runner.py | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/tools/testrunner/base_runner.py b/tools/testrunner/base_runner.py index 5a0fecc307..7f9b43435f 100644 --- a/tools/testrunner/base_runner.py +++ b/tools/testrunner/base_runner.py @@ -46,10 +46,8 @@ DEFAULT_OUT_GN = 'out.gn' # Map of test name synonyms to lists of test suites. Should be ordered by # expected runtimes (suites with slow test cases first). These groups are # invoked in separate steps on the bots. -# The mapping from names used here to GN targets (which must stay in sync) -# is defined in infra/mb/gn_isolate_map.pyl. TEST_MAP = { - # This needs to stay in sync with group("v8_bot_default") in test/BUILD.gn. + # This needs to stay in sync with test/bot_default.isolate. "bot_default": [ "debugger", "mjsunit", @@ -64,9 +62,8 @@ TEST_MAP = { "preparser", "intl", "unittests", - "wasm-api-tests", ], - # This needs to stay in sync with group("v8_default") in test/BUILD.gn. + # This needs to stay in sync with test/default.isolate. "default": [ "debugger", "mjsunit", @@ -80,9 +77,8 @@ TEST_MAP = { "preparser", "intl", "unittests", - "wasm-api-tests", ], - # This needs to stay in sync with group("v8_d8_default") in test/BUILD.gn. + # This needs to stay in sync with test/d8_default.isolate. "d8_default": [ "debugger", "mjsunit", @@ -91,7 +87,7 @@ TEST_MAP = { "preparser", "intl", ], - # This needs to stay in sync with "v8_optimize_for_size" in test/BUILD.gn. + # This needs to stay in sync with test/optimize_for_size.isolate. "optimize_for_size": [ "debugger", "mjsunit",