Run pyformat on all the toolchain-utils files.

This gets rid of a lot of lint issues.
Ran by doing this:

for f in *.py; do echo -n "$f " ; if [ -x $f ]; then pyformat -i
--remove_trailing_comma --yapf --force_quote_type=double $f ; else
pyformat -i --remove_shebang --remove_trailing_comma --yapf
--force_quote_type=double $f ; fi ; done

BUG=chromium:567921
TEST=Ran simple crosperf run.

Change-Id: I59778835fdaa5f706d2e1765924389f9e97433d1
Reviewed-on: https://chrome-internal-review.googlesource.com/242031
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>

1 files changed, 17 insertions, 18 deletions

diff --git a/cwp/bartlett/test/server_tester.py b/cwp/bartlett/test/server_tester.py
index 27889651..585da43a 100644
--- a/cwp/bartlett/test/server_tester.py
+++ b/cwp/bartlett/test/server_tester.py
@@ -1,4 +1,3 @@
-#!/usr/bin/python
 # Copyright 2012 Google Inc. All Rights Reserved.
 # Author: mrdmnd@ (Matt Redmond)
 """A unit test for sending data to Bartlett. Requires poster module."""
@@ -15,10 +14,9 @@ import urllib2
 from poster.encode import multipart_encode
 from poster.streaminghttp import register_openers
 
-
-SERVER_DIR = "../."
-SERVER_URL = "http://localhost:8080/"
-GET = "_ah/login?email=googler@google.com&action=Login&continue=%s"
+SERVER_DIR = '../.'
+SERVER_URL = 'http://localhost:8080/'
+GET = '_ah/login?email=googler@google.com&action=Login&continue=%s'
 AUTH_URL = SERVER_URL + GET
 
 
@@ -51,44 +49,46 @@ class ServerTest(unittest.TestCase):
 
   def _testUpload(self):  # pylint: disable-msg=C6409
     register_openers()
-    data = {"profile_data": self.profile_data,
-            "board": "x86-zgb",
-            "chromeos_version": "2409.0.2012_06_08_1114"}
+    data = {'profile_data': self.profile_data,
+            'board': 'x86-zgb',
+            'chromeos_version': '2409.0.2012_06_08_1114'}
     datagen, headers = multipart_encode(data)
-    request = urllib2.Request(SERVER_URL + "upload", datagen, headers)
+    request = urllib2.Request(SERVER_URL + 'upload', datagen, headers)
     response = urllib2.urlopen(request).read()
     self.assertTrue(response)
     return response
 
   def _testListAll(self):  # pylint: disable-msg=C6409
-    request = urllib2.Request(AUTH_URL % (SERVER_URL + "serve"))
+    request = urllib2.Request(AUTH_URL % (SERVER_URL + 'serve'))
     response = self.opener.open(request).read()
     self.assertTrue(response)
 
   def _testServeKey(self, key):  # pylint: disable-msg=C6409
-    request = urllib2.Request(AUTH_URL % (SERVER_URL + "serve/" + key))
+    request = urllib2.Request(AUTH_URL % (SERVER_URL + 'serve/' + key))
     response = self.opener.open(request).read()
     self.assertTrue(response)
 
   def _testDelKey(self, key):  # pylint: disable-msg=C6409
     # There is no response to a delete request.
     # We will check the listAll page to ensure there is no data.
-    request = urllib2.Request(AUTH_URL % (SERVER_URL + "del/" + key))
+    request = urllib2.Request(AUTH_URL % (SERVER_URL + 'del/' + key))
     response = self.opener.open(request).read()
-    request = urllib2.Request(AUTH_URL % (SERVER_URL + "serve"))
+    request = urllib2.Request(AUTH_URL % (SERVER_URL + 'serve'))
     response = self.opener.open(request).read()
     self.assertFalse(response)
 
 
 def LaunchLocalServer():
   """Launch and store an authentication cookie with a local server."""
-  proc = subprocess.Popen(["dev_appserver.py", "--clear_datastore", SERVER_DIR],
-                          stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+  proc = subprocess.Popen(
+      ['dev_appserver.py', '--clear_datastore', SERVER_DIR],
+      stdout=subprocess.PIPE,
+      stderr=subprocess.PIPE)
   # Wait for server to come up
   while True:
     time.sleep(1)
     try:
-      request = urllib2.Request(SERVER_URL + "serve")
+      request = urllib2.Request(SERVER_URL + 'serve')
       response = urllib2.urlopen(request).read()
       if response:
         break
@@ -97,6 +97,5 @@ def LaunchLocalServer():
   return proc
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
   unittest.main()
-