Mixed bag of nits.

tests/gclient_smoketest.py

 #!/usr/bin/python
 # Copyright (c) 2010 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Smoke tests for gclient.py.
 
 Shell out 'gclient' and run basic conformance tests.
 
 This test assumes GClientSmokeBase.URL_BASE is valid.
@@ skipping 141 matching lines @@
       results = self.gclient(cmd)
       self.check(('', '', 0), results)
       self.checkString(expected, open(p, 'rU').read())
 
     test(['config', self.svn_base + 'trunk/src/'],
          'solutions = [\n'
          '  { "name"        : "src",\n'
          '    "url"         : "svn://127.0.0.1/svn/trunk/src",\n'
          '    "custom_deps" : {\n'
          '    },\n'
-         '    "safesync_url": ""\n'
-         '  },\n]\n')
+         '    "safesync_url": "",\n'
+         '  },\n'
+         ']\n')
 
     test(['config', self.git_base + 'repo_1', '--name', 'src'],
          'solutions = [\n'
          '  { "name"        : "src",\n'
          '    "url"         : "git://127.0.0.1/git/repo_1",\n'
          '    "custom_deps" : {\n'
          '    },\n'
-         '    "safesync_url": ""\n'
-         '  },\n]\n')
+         '    "safesync_url": "",\n'
+         '  },\n'
+         ']\n')
 
     test(['config', 'foo', 'faa'],
          'solutions = [\n'
          '  { "name"        : "foo",\n'
          '    "url"         : "foo",\n'
          '    "custom_deps" : {\n'
          '    },\n'
-         '    "safesync_url": "faa"\n'
-         '  },\n]\n')
+         '    "safesync_url": "faa",\n'
+         '  },\n'
+         ']\n')
 
     test(['config', '--spec', '["blah blah"]'], '["blah blah"]')
 
     os.remove(p)
     results = self.gclient(['config', 'foo', 'faa', 'fuu'])
     err = ('Usage: gclient.py config [options] [url] [safesync url]\n\n'
            'gclient.py: error: Inconsistent arguments. Use either --spec or one'
            ' or 2 args\n')
     self.check(('', err, 2), results)
     self.assertFalse(os.path.exists(join(self.root_dir, '.gclient')))
@@ skipping 472 matching lines @@
 if __name__ == '__main__':
   if '-c' in sys.argv:
     COVERAGE = True
     sys.argv.remove('-c')
     if os.path.exists('.coverage'):
       os.remove('.coverage')
     os.environ['COVERAGE_FILE'] = os.path.join(
         os.path.dirname(os.path.dirname(os.path.abspath(__file__))),
         '.coverage')
   unittest.main()