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

Side by Side Diff: mojo/tools/data/apptests

Issue 1771393002: Convert mus apptests to shelltests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « mojo/shell/tests/mojo_shell_unittests.cc ('k') | testing/buildbot/chromium.linux.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # This file contains a list of Mojo gtest unit tests. 1 # This file contains a list of Mojo gtest unit tests.
2 # 2 #
3 # This must be valid Python. It may use the |config| global that will be a 3 # This must be valid Python. It may use the |config| global that will be a
4 # mopy.config.Config object, and must set a |tests| global that will contain the 4 # mopy.config.Config object, and must set a |tests| global that will contain the
5 # list of tests to run. 5 # list of tests to run.
6 # 6 #
7 # The entries in |tests| are dictionaries of the following form: 7 # The entries in |tests| are dictionaries of the following form:
8 # { 8 # {
9 # # Required URL for apptest. 9 # # Required URL for apptest.
10 # 'test': 'mojo:test_app_url', 10 # 'test': 'mojo:test_app_url',
11 # # Optional display name (otherwise the entry for 'test' above is used). 11 # # Optional display name (otherwise the entry for 'test' above is used).
12 # 'name': 'mojo:test_app_url (more details)', 12 # 'name': 'mojo:test_app_url (more details)',
13 # # Optional test type. Valid values: 13 # # Optional test type. Valid values:
14 # # * 'gtest': (default) 14 # # * 'gtest': (default)
15 # # * 'gtest_isolated': like 'gtest', but run with fixture isolation. 15 # # * 'gtest_isolated': like 'gtest', but run with fixture isolation.
16 # # i.e., each test in a fresh mojo_shell 16 # # i.e., each test in a fresh mojo_shell
17 # 'type': 'gtest', 17 # 'type': 'gtest',
18 # # Optional arguments for the shell or test. 18 # # Optional arguments for the shell or test.
19 # 'args': ['--some-flag-for-the-shell', '--some-flag-for-the-test'], 19 # 'args': ['--some-flag-for-the-shell', '--some-flag-for-the-test'],
20 # } 20 # }
21 # 21 #
22 # TODO(vtl|msw): Add a way of specifying data dependencies. 22 # TODO(vtl|msw): Add a way of specifying data dependencies.
23 23
24 mus_apptests = { 'test': 'mojo:mus_apptests' }
25 # osmesa is not used on android, so no flags are needed for
26 # mus_apptests.
27 if config.target_os != config.OS_ANDROID:
28 mus_apptests['args'] = [ '--use-x11-test-config',
29 '--override-use-gl-with-osmesa-for-tests' ]
30
31 # WARNING: If you add a test make sure the gn target mojo_apptests depends upon 24 # WARNING: If you add a test make sure the gn target mojo_apptests depends upon
32 # it. 25 # it.
33 tests = [ 26 tests = [
34 # TODO(rockot): Flaky. Failing with "Unrecognized --gtest_list_tests output" 27 # TODO(rockot): Flaky. Failing with "Unrecognized --gtest_list_tests output"
35 # http://crbug.com/569367. 28 # http://crbug.com/569367.
36 # { 29 # {
37 # 'test': 'mojo:network_service_apptests', 30 # 'test': 'mojo:network_service_apptests',
38 # 'type': 'gtest_isolated', 31 # 'type': 'gtest_isolated',
39 # }, 32 # },
40 { 33 {
(...skipping 23 matching lines...) Expand all
64 { 57 {
65 'test': 'mojo:media_apptests', 58 'test': 'mojo:media_apptests',
66 'type': 'gtest_isolated', 59 'type': 'gtest_isolated',
67 }, 60 },
68 # TODO(media): these tests fail in debug, which has no bot-coverage. 61 # TODO(media): these tests fail in debug, which has no bot-coverage.
69 # http://crbug.com/587523 62 # http://crbug.com/587523
70 # { 63 # {
71 # 'test': 'mojo:media_pipeline_integration_apptests', 64 # 'test': 'mojo:media_pipeline_integration_apptests',
72 # 'type': 'gtest_isolated', 65 # 'type': 'gtest_isolated',
73 # }, 66 # },
74 # TODO(crbug.com/560626): Fix and enable mus_apptests on Android.
75 mus_apptests,
76 # TODO(sky): enable when works (http://crbug.com/577274 and likely
77 # http://crbug.com/569367).
78 # {
79 # 'test': 'mojo:views_apptests',
80 # 'type': 'gtest_isolated',
81 # 'args': ['--use-x11-test-config',
82 # '--override-use-gl-with-osmesa-for-tests', '--enable-logging']
83 # },
84 { 67 {
85 'test': 'mojo:leveldb_apptests', 68 'test': 'mojo:leveldb_apptests',
86 'type': 'gtest_isolated', 69 'type': 'gtest_isolated',
87 } 70 }
88 ] 71 ]
OLDNEW
« no previous file with comments | « mojo/shell/tests/mojo_shell_unittests.cc ('k') | testing/buildbot/chromium.linux.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698