[test] Add a d8_default test suite collection

This adds a reduced version of bot_default with only d8-based test suites.
Those can be used for gc stress testing, fuzzing and predictable mode.

This also adds a missing explicit build rule for the debugger test suite.
It's working as it's part of the isolate file, but incremental
re-isolation might break on code changes.

Running the tests is still commented out to allow the infra-side to
land. Afterwards, we'll activate the extra testing as a V8 CL
commenting the lines back in.

Bug: v8:7285
Change-Id: I7b758c212f0c4ae3d2f79beea5ec597cee479565
Reviewed-on: https://chromium-review.googlesource.com/864045Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#50545}

gypfiles/all.gyp

@@ -33,6 +33,7 @@
             '../test/benchmarks/benchmarks.gyp:*',
             '../test/debugger/debugger.gyp:*',
             '../test/default.gyp:*',
+            '../test/d8_default.gyp:*',
             '../test/intl/intl.gyp:*',
             '../test/message/message.gyp:*',
             '../test/mjsunit/mjsunit.gyp:*',

test/BUILD.gn

@@ -28,6 +28,7 @@ group("gn_all") {
       ":benchmarks_run",
       ":bot_default_run",
       ":default_run",
+      ":d8_default_run",
       ":mozilla_run",
       "test262:test262_run",
     ]
@@ -44,6 +45,7 @@ group("default_tests") {
   if (v8_test_isolation_mode != "noop") {
     deps = [
       ":cctest_run",
+      ":debugger_run",
       ":fuzzer_run",
       ":inspector-test_run",
       ":intl_run",
@@ -74,6 +76,19 @@ v8_isolate_run("default") {
   isolate = "default.isolate"
 }
 
+v8_isolate_run("d8_default") {
+  deps = [
+    ":debugger_run",
+    ":intl_run",
+    ":message_run",
+    ":mjsunit_run",
+    ":preparser_run",
+    ":webkit_run",
+  ]
+
+  isolate = "d8_default.isolate"
+}
+
 v8_isolate_run("optimize_for_size") {
   deps = [
     ":cctest_run",
@@ -123,6 +138,14 @@ v8_isolate_run("cctest_exe") {
   isolate = "cctest/cctest_exe.isolate"
 }
 
+v8_isolate_run("debugger") {
+  deps = [
+    "..:d8_run",
+  ]
+
+  isolate = "debugger/debugger.isolate"
+}
+
 v8_isolate_run("fuzzer") {
   deps = [
     "..:v8_simple_json_fuzzer",

test/bot_default.gyp

@@ -11,6 +11,7 @@
           'type': 'none',
           'dependencies': [
             'cctest/cctest.gyp:cctest_run',
+            'debugger/debugger.gyp:debugger_run',
             'fuzzer/fuzzer.gyp:fuzzer_run',
             'inspector/inspector.gyp:inspector-test_run',
             'intl/intl.gyp:intl_run',

test/d8_default.gyp

+# Copyright 2018 the V8 project authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+  'conditions': [
+    ['test_isolation_mode != "noop"', {
+      'targets': [
+        {
+          'target_name': 'd8_default_run',
+          'type': 'none',
+          'dependencies': [
+            'debugger/debugger.gyp:debugger_run',
+            'intl/intl.gyp:intl_run',
+            'message/message.gyp:message_run',
+            'mjsunit/mjsunit.gyp:mjsunit_run',
+            'preparser/preparser.gyp:preparser_run',
+            'webkit/webkit.gyp:webkit_run',
+          ],
+          'includes': [
+            '../gypfiles/features.gypi',
+            '../gypfiles/isolate.gypi',
+          ],
+          'sources': [
+            'd8_default.isolate',
+          ],
+        },
+      ],
+    }],
+  ],
+}

test/d8_default.isolate

+# Copyright 2018 the V8 project authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+{
+  'variables': {
+    'command': [
+      '../tools/run-tests.py',
+    ],
+  },
+  'includes': [
+    'debugger/debugger.isolate',
+    'intl/intl.isolate',
+    'message/message.isolate',
+    'mjsunit/mjsunit.isolate',
+    'preparser/preparser.isolate',
+    'webkit/webkit.isolate',
+  ],
+}

test/default.gyp

@@ -11,6 +11,7 @@
           'type': 'none',
           'dependencies': [
             'cctest/cctest.gyp:cctest_run',
+            'debugger/debugger.gyp:debugger_run',
             'fuzzer/fuzzer.gyp:fuzzer_run',
             'inspector/inspector.gyp:inspector-test_run',
             'intl/intl.gyp:intl_run',

test/optimize_for_size.gyp

@@ -11,6 +11,7 @@
           'type': 'none',
           'dependencies': [
             'cctest/cctest.gyp:cctest_run',
+            'debugger/debugger.gyp:debugger_run',
             'intl/intl.gyp:intl_run',
             'mjsunit/mjsunit.gyp:mjsunit_run',
             'webkit/webkit.gyp:webkit_run',

tools/testrunner/base_runner.py

@@ -62,6 +62,16 @@ TEST_MAP = {
     "intl",
     "unittests",
   ],
+  # This needs to stay in sync with test/d8_default.isolate.
+  "d8_default": [
+    # TODO(machenbach): uncomment after infra side lands.
+    #"debugger",
+    "mjsunit",
+    "webkit",
+    #"message",
+    #"preparser",
+    #"intl",
+  ],
   # This needs to stay in sync with test/optimize_for_size.isolate.
   "optimize_for_size": [
     "debugger",

tools/testrunner/standard_runner.py

@@ -336,6 +336,8 @@ class StandardTestRunner(base_runner.BaseTestRunner):
       if self.build_config.no_i18n:
         base_runner.TEST_MAP["bot_default"].remove("intl")
         base_runner.TEST_MAP["default"].remove("intl")
+        # TODO(machenbach): uncomment after infra side lands.
+        # base_runner.TEST_MAP["d8_default"].remove("intl")
 
     def _setup_env(self):
       super(StandardTestRunner, self)._setup_env()