[toolchain-utils] Fix remaining lint errors in toolchain-utils.
In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).
BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
Ran afe_lock_machine.py (check machine status)
Ran full crosperf test (octane, speedometer, BootPerf) on alex.
Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
diff --git a/remote_test.py b/remote_test.py
index 82f54ed..62598d5 100755
--- a/remote_test.py
+++ b/remote_test.py
@@ -1,4 +1,4 @@
-#!/usr/bin/python2
+#!/usr/bin/env python2
#
# Copyright 2010 Google Inc. All Rights Reserved.
"""Script to wrap test_that script.
@@ -26,14 +26,13 @@
def Main(argv):
parser = argparse.ArgumentParser()
- parser.add_argument('-c',
- '--chromeos_root',
- dest='chromeos_root',
- help='ChromeOS root checkout directory')
- parser.add_argument('-r',
- '--remote',
- dest='remote',
- help='Remote chromeos device.')
+ parser.add_argument(
+ '-c',
+ '--chromeos_root',
+ dest='chromeos_root',
+ help='ChromeOS root checkout directory')
+ parser.add_argument(
+ '-r', '--remote', dest='remote', help='Remote chromeos device.')
options = parser.parse_args(argv)
if options.chromeos_root is None:
Usage(parser, 'chromeos_root must be given')
@@ -45,50 +44,55 @@
command = 'ls -lt /'
ce = command_executer.GetCommandExecuter()
- ce.CrosRunCommand(command,
- chromeos_root=options.chromeos_root,
- machine=options.remote)
+ ce.CrosRunCommand(
+ command, chromeos_root=options.chromeos_root, machine=options.remote)
version_dir_path, script_name = misc.GetRoot(sys.argv[0])
version_dir = misc.GetRoot(version_dir_path)[1]
# Tests to copy directories and files to the chromeos box.
- ce.CopyFiles(version_dir_path,
- '/tmp/' + version_dir,
- dest_machine=options.remote,
- dest_cros=True,
- chromeos_root=options.chromeos_root)
- ce.CopyFiles(version_dir_path,
- '/tmp/' + version_dir + '1',
- dest_machine=options.remote,
- dest_cros=True,
- chromeos_root=options.chromeos_root)
- ce.CopyFiles(sys.argv[0],
- '/tmp/' + script_name,
- recursive=False,
- dest_machine=options.remote,
- dest_cros=True,
- chromeos_root=options.chromeos_root)
- ce.CopyFiles(sys.argv[0],
- '/tmp/' + script_name + '1',
- recursive=False,
- dest_machine=options.remote,
- dest_cros=True,
- chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ version_dir_path,
+ '/tmp/' + version_dir,
+ dest_machine=options.remote,
+ dest_cros=True,
+ chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ version_dir_path,
+ '/tmp/' + version_dir + '1',
+ dest_machine=options.remote,
+ dest_cros=True,
+ chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ sys.argv[0],
+ '/tmp/' + script_name,
+ recursive=False,
+ dest_machine=options.remote,
+ dest_cros=True,
+ chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ sys.argv[0],
+ '/tmp/' + script_name + '1',
+ recursive=False,
+ dest_machine=options.remote,
+ dest_cros=True,
+ chromeos_root=options.chromeos_root)
# Test to copy directories and files from the chromeos box.
- ce.CopyFiles('/tmp/' + script_name,
- '/tmp/hello',
- recursive=False,
- src_machine=options.remote,
- src_cros=True,
- chromeos_root=options.chromeos_root)
- ce.CopyFiles('/tmp/' + script_name,
- '/tmp/' + script_name,
- recursive=False,
- src_machine=options.remote,
- src_cros=True,
- chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ '/tmp/' + script_name,
+ '/tmp/hello',
+ recursive=False,
+ src_machine=options.remote,
+ src_cros=True,
+ chromeos_root=options.chromeos_root)
+ ce.CopyFiles(
+ '/tmp/' + script_name,
+ '/tmp/' + script_name,
+ recursive=False,
+ src_machine=options.remote,
+ src_cros=True,
+ chromeos_root=options.chromeos_root)
board = ce.CrosLearnBoard(options.chromeos_root, options.remote)
print(board)
return 0