Remove semicolons from revinfo.

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 411 matching lines @@
   def testRunHooksDepsOs(self):
     self.gclient(['config', self.svn_base + 'trunk/src/'])
     self.gclient(['sync', '--deps', 'mac', '--revision', 'src@1'])
     out = self.parseGclient(['runhooks', '--deps', 'mac'], [])
     self.assertEquals([], out)
 
   def testRevInfo(self):
     self.gclient(['config', self.svn_base + 'trunk/src/'])
     self.gclient(['sync', '--deps', 'mac'])
     results = self.gclient(['revinfo', '--deps', 'mac'])
-    out = ('src: %(base)s/src;\n'
-           'src/file/other: File("%(base)s/other/DEPS");\n'
-           'src/other: %(base)s/other;\n'
+    out = ('src: %(base)s/src\n'
+           'src/file/other: File("%(base)s/other/DEPS")\n'
+           'src/other: %(base)s/other\n'
            'src/third_party/foo: %(base)s/third_party/foo@1\n' %
           { 'base': self.svn_base + 'trunk' })
     self.check((out, '', 0), results)
     results = self.gclient(['revinfo', '--deps', 'mac', '--actual'])
-    out = ('src: %(base)s/src@2;\n'
-           'src/file/other: %(base)s/other/DEPS@2;\n'
-           'src/other: %(base)s/other@2;\n'
+    out = ('src: %(base)s/src@2\n'
+           'src/file/other: %(base)s/other/DEPS@2\n'
+           'src/other: %(base)s/other@2\n'
            'src/third_party/foo: %(base)s/third_party/foo@1\n' %
           { 'base': self.svn_base + 'trunk' })
     self.check((out, '', 0), results)
     results = self.gclient(['revinfo', '--deps', 'mac', '--snapshot'])
     out = ('# Snapshot generated with gclient revinfo --snapshot\n'
            'solutions = [\n'
            '  { "name"        : "src",\n'
            '    "url"         : "%(base)s/src",\n'
            '    "custom_deps" : {\n'
            '      "foo/bar": None,\n'
@@ skipping 174 matching lines @@
     tree['src/git_hooked1'] = 'git_hooked1'
     tree['src/git_hooked2'] = 'git_hooked2'
     self.assertTree(tree)
 
   def testRevInfo(self):
     if not self.enabled:
       return
     self.gclient(['config', self.git_base + 'repo_1', '--name', 'src'])
     self.gclient(['sync', '--deps', 'mac'])
     results = self.gclient(['revinfo', '--deps', 'mac'])
-    out = ('src: %(base)srepo_1;\n'
-           'src/repo2: %(base)srepo_2@%(hash2)s;\n'
+    out = ('src: %(base)srepo_1\n'
+           'src/repo2: %(base)srepo_2@%(hash2)s\n'
            'src/repo2/repo_renamed: %(base)srepo_3\n' %
           {
             'base': self.git_base,
             'hash1': self.githash('repo_1', 2)[:7],
             'hash2': self.githash('repo_2', 1)[:7],
             'hash3': self.githash('repo_3', 2)[:7],
           })
     self.check((out, '', 0), results)
     results = self.gclient(['revinfo', '--deps', 'mac', '--actual'])
-    out = ('src: %(base)srepo_1@%(hash1)s;\n'
-           'src/repo2: %(base)srepo_2@%(hash2)s;\n'
+    out = ('src: %(base)srepo_1@%(hash1)s\n'
+           'src/repo2: %(base)srepo_2@%(hash2)s\n'
            'src/repo2/repo_renamed: %(base)srepo_3@%(hash3)s\n' %
           {
             'base': self.git_base,
             'hash1': self.githash('repo_1', 2),
             'hash2': self.githash('repo_2', 1),
             'hash3': self.githash('repo_3', 2),
           })
     self.check((out, '', 0), results)
 
 
@@ skipping 70 matching lines @@
     if not self.enabled:
       return
     self.gclient(['config', '--spec',
         'solutions=['
         '{"name": "src",'
         ' "url": "' + self.svn_base + 'trunk/src/"},'
         '{"name": "src-git",'
         '"url": "' + self.git_base + 'repo_1"}]'])
     self.gclient(['sync', '--deps', 'mac'])
     results = self.gclient(['revinfo', '--deps', 'mac'])
-    out = ('src: %(svn_base)s/src/;\n'
-           'src-git: %(git_base)srepo_1;\n'
-           'src/file/other: File("%(svn_base)s/other/DEPS");\n'
-           'src/other: %(svn_base)s/other;\n'
-           'src/repo2: %(git_base)srepo_2@%(hash2)s;\n'
-           'src/repo2/repo_renamed: %(git_base)srepo_3;\n'
+    out = ('src: %(svn_base)s/src/\n'
+           'src-git: %(git_base)srepo_1\n'
+           'src/file/other: File("%(svn_base)s/other/DEPS")\n'
+           'src/other: %(svn_base)s/other\n'
+           'src/repo2: %(git_base)srepo_2@%(hash2)s\n'
+           'src/repo2/repo_renamed: %(git_base)srepo_3\n'
            'src/third_party/foo: %(svn_base)s/third_party/foo@1\n') % {
                'svn_base': self.svn_base + 'trunk',
                'git_base': self.git_base,
                'hash1': self.githash('repo_1', 2)[:7],
                'hash2': self.githash('repo_2', 1)[:7],
                'hash3': self.githash('repo_3', 2)[:7],
           }
     self.check((out, '', 0), results)
     results = self.gclient(['revinfo', '--deps', 'mac', '--actual'])
-    out = ('src: %(svn_base)s/src/@2;\n'
-           'src-git: %(git_base)srepo_1@%(hash1)s;\n'
-           'src/file/other: %(svn_base)s/other/DEPS@2;\n'
-           'src/other: %(svn_base)s/other@2;\n'
-           'src/repo2: %(git_base)srepo_2@%(hash2)s;\n'
-           'src/repo2/repo_renamed: %(git_base)srepo_3@%(hash3)s;\n'
+    out = ('src: %(svn_base)s/src/@2\n'
+           'src-git: %(git_base)srepo_1@%(hash1)s\n'
+           'src/file/other: %(svn_base)s/other/DEPS@2\n'
+           'src/other: %(svn_base)s/other@2\n'
+           'src/repo2: %(git_base)srepo_2@%(hash2)s\n'
+           'src/repo2/repo_renamed: %(git_base)srepo_3@%(hash3)s\n'
            'src/third_party/foo: %(svn_base)s/third_party/foo@1\n') % {
                'svn_base': self.svn_base + 'trunk',
                'git_base': self.git_base,
                'hash1': self.githash('repo_1', 2),
                'hash2': self.githash('repo_2', 1),
                'hash3': self.githash('repo_3', 2),
           }
     self.check((out, '', 0), results)
 
   def testRecurse(self):
@@ skipping 80 matching lines @@
     self.checkString('foo', out[0][1])
     tree = self.mangle_svn_tree(
         ('trunk/webkit@2', ''),
         ('trunk/third_party/foo@1', 'foo/bar'))
     self.assertTree(tree)
 
   def testRevInfo(self):
     self.gclient(['sync', '--deps', 'mac'])
     results = self.gclient(['revinfo', '--deps', 'mac'])
     expected = (
-        './: None;\nfoo/bar: svn://127.0.0.1/svn/trunk/third_party/foo@1\n',
+        './: None\nfoo/bar: svn://127.0.0.1/svn/trunk/third_party/foo@1\n',
         '', 0)
     self.check(expected, results)
     # TODO(maruel): To be added after the refactor.
     #results = self.gclient(['revinfo', '--snapshot'])
     #expected = (
-    #    './: None;\nfoo/bar: svn://127.0.0.1/svn/trunk/third_party/foo@1\n',
+    #    './: None\nfoo/bar: svn://127.0.0.1/svn/trunk/third_party/foo@1\n',
     #    '', 0)
     #self.check(expected, results)
 
   def testRest(self):
     self.gclient(['sync'])
     # TODO(maruel): This is incorrect, it should run on ./ too.
     out = self.parseGclient(['cleanup', '--deps', 'mac'],
                             [('running', join(self.root_dir, 'foo', 'bar'))])
     out = self.parseGclient(['diff', '--deps', 'mac'],
                             [('running', join(self.root_dir, 'foo', 'bar'))])
 
 
 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()