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

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

Issue 1316403005: view-manager: Use --override-use-gl-with-osmesa-for-tests for apptests on non-android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 3 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 | « no previous file | no next file » | 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 view_manager_apptests = { 'test': 'mojo:view_manager_apptests' }
25 # osmesa is not used on android, so no flags are needed for
26 # view_manager_apptests.
27 if config.target_os != config.OS_ANDROID:
28 view_manager_apptests['args'] = [ '--use-x11-test-config',
29 '--override-use-gl-with-osmesa-for-tests' ]
30
24 tests = [ 31 tests = [
25 { 32 {
26 'test': 'mojo:clipboard_apptests', 33 'test': 'mojo:clipboard_apptests',
27 'type': 'gtest_isolated', 34 'type': 'gtest_isolated',
28 }, 35 },
29 { 36 {
30 'test': 'mojo:network_service_apptests', 37 'test': 'mojo:network_service_apptests',
31 'type': 'gtest_isolated', 38 'type': 'gtest_isolated',
32 }, 39 },
33 { 40 {
34 'test': 'mojo:resource_provider_apptests', 41 'test': 'mojo:resource_provider_apptests',
35 'type': 'gtest_isolated', 42 'type': 'gtest_isolated',
36 }, 43 },
37 # TODO(msw|jam): Fix and enable the runner_apptests: http://crbug.com/479316 44 # TODO(msw|jam): Fix and enable the runner_apptests: http://crbug.com/479316
38 #{ 45 #{
39 # 'test': 'mojo:runner_apptests', 46 # 'test': 'mojo:runner_apptests',
40 # 'type': 'gtest_isolated', 47 # 'type': 'gtest_isolated',
41 #}, 48 #},
42 { 49 view_manager_apptests,
43 'test': 'mojo:view_manager_apptests',
44 'args': ['--use-x11-test-config',
45 '--override-use-gl-with-osmesa-for-tests']
46 },
47 ] 50 ]
48 51
49 # TODO(msw): Get these tests passing on Android too. http://crbug.com/486220 52 # TODO(msw): Get these tests passing on Android too. http://crbug.com/486220
50 if config.target_os != config.OS_ANDROID: 53 if config.target_os != config.OS_ANDROID:
51 tests += [ 54 tests += [
52 { 55 {
53 'test': 'mojo:filesystem_apptests', 56 'test': 'mojo:filesystem_apptests',
54 'type': 'gtest_isolated', 57 'type': 'gtest_isolated',
55 }, 58 },
56 { 59 {
(...skipping 28 matching lines...) Expand all
85 # http://crbug.com/501417 88 # http://crbug.com/501417
86 { 89 {
87 'test': 'mojo:media_apptests', 90 'test': 'mojo:media_apptests',
88 'type': 'gtest_isolated', 91 'type': 'gtest_isolated',
89 }, 92 },
90 { 93 {
91 'test': 'mojo:sql_apptests', 94 'test': 'mojo:sql_apptests',
92 'type': 'gtest_isolated', 95 'type': 'gtest_isolated',
93 }, 96 },
94 ] 97 ]
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698