commit 11ce4e7b56992300ba20c1a2959108fda224856f
author Keith Haddow <haddowk@chromium.org> Wed Oct 17 13:05:39 2018 -0700
committer Ilja H. Friedel <ihf@chromium.org> Wed Oct 17 20:36:04 2018 +0000
tree 0fdccec59012a330e4f8da803374c921eb0d792d
parent 9a4bfc38d4ffadd38c780a3aeb18fb1e178d78e8

[autotest] Fix syntax error in tradefed test

Missing brackets around a list of params for string format of an
error.

BUG=chromium:896408
TEST=Manual testing

Change-Id: I96a36a54b90c6ef5c8fb5d3d5001fd6a3d86bbf7
Reviewed-on: https://chromium-review.googlesource.com/c/1287112
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Trybot-Ready: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>

server/cros/tradefed_test.py

diff --git a/server/cros/tradefed_test.py b/server/cros/tradefed_test.py
index 7194a23..674f9b7 100644
--- a/server/cros/tradefed_test.py
+++ b/server/cros/tradefed_test.py
@@ -932,7 +932,7 @@
         except (shutil.Error, OSError, IOError) as e:
             raise error.TestFail(
                 'Error: failed to copy test subplan %s to CTS bundle. %s' %
-                test_subplan_file, e)
+                (test_subplan_file, e))
 
     def _should_skip_test(self, _bundle):
         """Some tests are expected to fail and are skipped.