commit 5e07e0608c00b3650e521a49cb85fb732d40e809
author sbc@chromium.org <sbc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Feb 28 23:55:44 2013 +0000
committer sbc@chromium.org <sbc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Feb 28 23:55:44 2013 +0000
tree 61025c25c36e6583670f5f613b374b9170a52190
parent dbf8b4edd4ca617530a9edf5fc8a3bb91606c777

git cl upload now diffs HEAD rather than working tree

BUG=179131
TEST=./tests/git_cl_test.py


Review URL: https://chromiumcodereview.appspot.com/12388027

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@185366 0039d316-1c4b-4281-b951-d872f2087c98

tests/git_cl_test.py

diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index 4b089b3..d8ee317 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -120,10 +120,10 @@
     if find_copies:
       stat_call = ((['git', 'diff', '--no-ext-diff', '--stat',
                    '--find-copies-harder', '-l100000', '-C'+similarity,
-                   'fake_ancestor_sha'],), '+dat')
+                   'fake_ancestor_sha', 'HEAD'],), '+dat')
     else:
       stat_call = ((['git', 'diff', '--no-ext-diff', '--stat',
-                   '-M'+similarity, 'fake_ancestor_sha'],), '+dat')
+                   '-M'+similarity, 'fake_ancestor_sha', 'HEAD'],), '+dat')
 
     return [
       ((['git', 'config', 'rietveld.server'],), 'codereview.example.com'),
@@ -149,7 +149,7 @@
       ((['git', 'config', 'user.email'],), 'me@example.com'),
       stat_call,
       ((['git', 'config', 'gerrit.host'],), ''),
-      ((['git', 'log', '--pretty=format:%s\n\n%b', 'fake_ancestor_sha..'],),
+      ((['git', 'log', '--pretty=format:%s\n\n%b', 'fake_ancestor_sha..HEAD'],),
        'desc\n'),
     ]
 
@@ -287,7 +287,7 @@
         '--cc', 'joe@example.com',
         '--git_similarity', similarity or '50'
     ] + (['--git_no_find_copies'] if find_copies == False else []) + [
-        'fake_ancestor_sha'
+        'fake_ancestor_sha', 'HEAD'
     ]
 
   def _run_reviewer_test(
@@ -457,7 +457,7 @@
          'foo'),
         ((['git', 'config', 'user.email'],), 'me@example.com'),
         ((['git', 'diff', '--no-ext-diff', '--stat', '--find-copies-harder',
-           '-l100000', '-C50', 'fake_ancestor_sha'],),
+           '-l100000', '-C50', 'fake_ancestor_sha', 'HEAD'],),
          '+dat'),
         ]
 
@@ -465,16 +465,19 @@
   def _gerrit_upload_calls(description, reviewers):
     calls = [
         ((['git', 'config', 'gerrit.host'],), 'gerrit.example.com'),
-        ((['git', 'log', '--pretty=format:%s\n\n%b', 'fake_ancestor_sha..'],),
+        ((['git', 'log', '--pretty=format:%s\n\n%b',
+           'fake_ancestor_sha..HEAD'],),
          description)
         ]
     if git_cl.CHANGE_ID not in description:
       calls += [
-          ((['git', 'log', '--pretty=format:%s\n\n%b', 'fake_ancestor_sha..'],),
+          ((['git', 'log', '--pretty=format:%s\n\n%b',
+             'fake_ancestor_sha..HEAD'],),
            description),
           ((['git', 'commit', '--amend', '-m', description],),
            ''),
-          ((['git', 'log', '--pretty=format:%s\n\n%b', 'fake_ancestor_sha..'],),
+          ((['git', 'log', '--pretty=format:%s\n\n%b',
+             'fake_ancestor_sha..HEAD'],),
            description)
           ]
     calls += [