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

Side by Side Diff: mojo/tools/apptest_runner.py

Issue 1162923004: Move the logic to set up a local origin for shell run to shell_arguments.py. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 6 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/tools/android_mojo_shell.py ('k') | 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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """A test runner for gtest application tests.""" 6 """A test runner for application tests."""
7 7
8 import argparse 8 import argparse
9 import logging 9 import logging
10 import sys 10 import sys
11 11
12 import devtools 12 import devtools
13 devtools.add_lib_to_path() 13 devtools.add_lib_to_path()
14 from devtoolslib.android_shell import AndroidShell 14 from devtoolslib.android_shell import AndroidShell
15 from devtoolslib.linux_shell import LinuxShell 15 from devtoolslib.linux_shell import LinuxShell
16 from devtoolslib.apptest_runner import run_apptests 16 from devtoolslib.apptest_runner import run_apptests
17 from devtoolslib import shell_arguments
17 18
18 from mopy import gtest 19 from mopy import gtest
19 from mopy.config import Config 20 from mopy.config import Config
20 from mopy.gn import ConfigForGNArgs, ParseGNConfig 21 from mopy.gn import ConfigForGNArgs, ParseGNConfig
21 from mopy.log import InitLogging 22 from mopy.log import InitLogging
22 from mopy.paths import Paths 23 from mopy.paths import Paths
23 24
24 25
25 _logger = logging.getLogger() 26 _logger = logging.getLogger()
26 27
(...skipping 10 matching lines...) Expand all
37 help="the build output directory") 38 help="the build output directory")
38 args = parser.parse_args() 39 args = parser.parse_args()
39 40
40 InitLogging(args.verbose_count) 41 InitLogging(args.verbose_count)
41 config = ConfigForGNArgs(ParseGNConfig(args.build_dir)) 42 config = ConfigForGNArgs(ParseGNConfig(args.build_dir))
42 paths = Paths(config) 43 paths = Paths(config)
43 extra_args = [] 44 extra_args = []
44 if config.target_os == Config.OS_ANDROID: 45 if config.target_os == Config.OS_ANDROID:
45 shell = AndroidShell(paths.adb_path) 46 shell = AndroidShell(paths.adb_path)
46 shell.InstallApk(paths.target_mojo_shell_path) 47 shell.InstallApk(paths.target_mojo_shell_path)
47 extra_args.append(shell.SetUpLocalOrigin(paths.build_dir, fixed_port=False)) 48 extra_args.extend(shell_arguments.ConfigureLocalOrigin(
49 shell, paths.build_dir, fixed_port=False))
48 else: 50 else:
49 shell = LinuxShell(paths.mojo_shell_path) 51 shell = LinuxShell(paths.mojo_shell_path)
50 52
51 gtest.set_color() 53 gtest.set_color()
52 54
53 test_list_globals = {"config": config} 55 test_list_globals = {"config": config}
54 exec args.test_list_file in test_list_globals 56 exec args.test_list_file in test_list_globals
55 apptests_result = run_apptests(shell, extra_args, test_list_globals["tests"]) 57 apptests_result = run_apptests(shell, extra_args, test_list_globals["tests"])
56 return 0 if apptests_result else 1 58 return 0 if apptests_result else 1
57 59
58 60
59 if __name__ == '__main__': 61 if __name__ == '__main__':
60 sys.exit(main()) 62 sys.exit(main())
OLDNEW
« no previous file with comments | « mojo/tools/android_mojo_shell.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698