Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(378)

Unified Diff: scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py

Issue 1737843003: Use regular chromium recipes for Chromium Mojo. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « masters/master.chromium.mojo/builders.pyl ('k') | scripts/slave/recipes/chromium_mojo.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py b/scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py
new file mode 100644
index 0000000000000000000000000000000000000000..9afa9484643bfee44cd7309ddc659b720a9a1ecf
--- /dev/null
+++ b/scripts/slave/recipe_modules/chromium_tests/chromium_mojo.py
@@ -0,0 +1,102 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+from . import steps
+
+SPEC = {
+ 'builders': {
+ 'Chromium Mojo Android': {
+ 'chromium_config': 'android',
+ 'chromium_apply_config': ['mb', 'ninja_confirm_noop'],
+ 'gclient_config': 'chromium',
+ 'gclient_apply_config': ['android'],
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ 'TARGET_PLATFORM': 'android',
+ 'TARGET_ARCH': 'arm',
+ # TODO(msw): Leave default? 'TARGET_BITS': 32,
Dirk Pranke 2016/02/26 00:15:54 uncomment.
msw 2016/02/26 01:38:01 Done.
+ },
+
+ # TODO(msw): Is any of the following necessary?
+ # See 'Android GN Builder (dbg)' and 'Android GN'
+
+ # 'android_config': 'main_builder_mb',
Dirk Pranke 2016/02/26 00:15:54 not sure about this one; uncomment or ask stip@, m
msw 2016/02/26 01:38:01 Done.
+
+ # 'test_generators': [
+ # steps.generate_gtest,
+ # steps.generate_instrumentation_test,
+ # steps.generate_isolated_script,
+ # steps.generate_script,
+ # ],
Dirk Pranke 2016/02/26 00:15:54 you need this block.
msw 2016/02/26 01:38:01 Done.
+
+ # 'tests': [
+ # steps.AndroidInstrumentationTest('MojoTest'),
+ # steps.GTestTest(
+ # 'breakpad_unittests',
+ # override_compile_targets=['breakpad_unittests_deps'],
+ # android_isolate_path='breakpad/breakpad_unittests.isolate'),
+ # steps.GTestTest(
+ # 'sandbox_linux_unittests',
+ # override_compile_targets=['sandbox_linux_unittests_deps']),
+ # steps.AndroidJunitTest('base_junit_tests'),
+ # ],
Dirk Pranke 2016/02/26 00:15:54 don't need this.
msw 2016/02/26 01:38:01 Done.
+
+ # 'bot_type': 'builder',
Dirk Pranke 2016/02/26 00:15:54 this should be 'builder_tester', and you need it.
msw 2016/02/26 01:38:01 Done.
+
+ # 'testing': {
+ # 'platform': 'linux',
+ # },
Dirk Pranke 2016/02/26 00:15:54 uncomment; needed.
msw 2016/02/26 01:38:01 Done.
+ },
+ 'Chromium Mojo Linux': {
+ 'chromium_config': 'chromium',
+ 'chromium_apply_config': ['mb', 'ninja_confirm_noop'],
+ 'gclient_config': 'chromium',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ # TODO(msw): Leave default? 'TARGET_PLATFORM': 'linux',
+ # TODO(msw): Leave default? 'TARGET_BITS': 64,
+ },
+
+ # TODO(msw): Is any of the following necessary?
+ # See 'Linux Tests' and 'Linux Builder (dbg)'
+
+ # 'bot_type': 'builder',
+
+ # 'test_generators': [
+ # steps.generate_gtest,
+ # steps.generate_script,
+ # steps.generate_isolated_script,
+ # ],
+
+ # 'testing': {
+ # 'platform': 'linux',
+ # },
+ },
+ 'Chromium Mojo Windows': {
+ 'chromium_config': 'chromium',
+ 'chromium_apply_config': ['mb', 'ninja_confirm_noop'],
+ 'gclient_config': 'chromium',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Release',
+ # TODO(msw): Leave default? 'TARGET_PLATFORM': 'win',
+ # TODO(msw): Leave default? 'TARGET_BITS': 32,
+ },
+
+ # TODO(msw): Is any of the following necessary?
+ # See 'Win8 GN (dbg)' and 'Win7 Tests (1)'
+
+ # 'bot_type': 'builder',
+
+ # 'test_generators': [
+ # steps.generate_gtest,
+ # steps.generate_script,
+ # steps.generate_isolated_script,
+ # ],
+
+ # 'testing': {
+ # 'platform': 'win',
+ # },
+ },
+ },
+}
« no previous file with comments | « masters/master.chromium.mojo/builders.pyl ('k') | scripts/slave/recipes/chromium_mojo.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698