diff --git a/git_cl.py b/git_cl.py index 46e60fb203..92c6cce5b2 100755 --- a/git_cl.py +++ b/git_cl.py @@ -2474,8 +2474,9 @@ def CMDupload(parser, args): options.squash = ((settings.GetSquashGerritUploads() or options.squash) and not options.no_squash) - return GerritUpload(options, args, cl, change) - ret = RietveldUpload(options, args, cl, change) + ret = GerritUpload(options, args, cl, change) + else: + ret = RietveldUpload(options, args, cl, change) if not ret: git_set_branch_value('last-upload-hash', RunGit(['rev-parse', 'HEAD']).strip()) diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py index c24a4d8fec..53815f64c9 100755 --- a/tests/git_cl_test.py +++ b/tests/git_cl_test.py @@ -227,6 +227,11 @@ class TestGitCl(TestCase): 'https://codereview.example.com'],), ''), ((['git', 'config', 'branch.master.rietveldpatchset', '2'],), ''), + ] + cls._git_post_upload_calls() + + @classmethod + def _git_post_upload_calls(cls): + return [ ((['git', 'rev-parse', 'HEAD'],), 'hash'), ((['git', 'symbolic-ref', 'HEAD'],), 'hash'), ((['git', @@ -574,8 +579,8 @@ class TestGitCl(TestCase): '+dat'), ] - @staticmethod - def _gerrit_upload_calls(description, reviewers, squash, + @classmethod + def _gerrit_upload_calls(cls, description, reviewers, squash, expected_upstream_ref='origin/refs/heads/master'): calls = [ ((['git', 'config', '--bool', 'gerrit.squash-uploads'],), 'false'), @@ -640,7 +645,7 @@ class TestGitCl(TestCase): 'refs/heads/git_cl_uploads/master', 'abcdef0123456789'],), '') ] - + calls += cls._git_post_upload_calls() return calls def _run_gerrit_upload_test(