commit 8f15f3ec1dbfbb6b6c0bc98eedc7a323d02fe317
author Andrii Shyshkalov <tandrii@chromium.org> Wed Dec 14 15:43:49 2016 +0100
committer Commit Bot <commit-bot@chromium.org> Wed Dec 14 15:36:41 2016 +0000
tree 6f44d2aa64125affcb31388657f6f3f90eccbba2
parent a5a1eea537ca2d82a0296dd703e038af7eee2fde

git cl refactor: s/should_git_number/should_add_git_number

R=sergiyb@google.com
BUG=chromium:642493

Change-Id: Id7b732d82c25ce88a3295ff54c79cf2a66e0d5b9
Reviewed-on: https://chromium-review.googlesource.com/419437
Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

tests/git_cl_test.py

diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index d68129d..71f3ed6 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -1169,7 +1169,7 @@
         remote_url='https://chromium.googlesource.com/chromium/tools/build',
         remote_ref='refs/whatever')
     self.assertEqual(res.pending_prefix, None)
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
   def test_GitNumbererState_fail_fetch(self):
     self.mock(git_cl.sys, 'stdout', StringIO.StringIO())
@@ -1185,7 +1185,7 @@
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/whatever')
     self.assertEqual(res.pending_prefix, 'refs/pending-prefix/')
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
   def test_GitNumbererState_fail_gnumbd_and_validator(self):
     self.mock(git_cl.sys, 'stdout', StringIO.StringIO())
@@ -1203,7 +1203,7 @@
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/whatever')
     self.assertEqual(res.pending_prefix, None)
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
   def test_GitNumbererState_valid_configs(self):
     class NamedTempFileStab(StringIO.StringIO):
@@ -1248,19 +1248,19 @@
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/heads/master')
     self.assertEqual(res.pending_prefix, 'refs/pending/')
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
     res = git_cl._GitNumbererState.load(
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/heads/test')
     self.assertEqual(res.pending_prefix, None)
-    self.assertEqual(res.should_git_number, True)
+    self.assertEqual(res.should_add_git_number, True)
 
     res = git_cl._GitNumbererState.load(
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/heads/disabled')
     self.assertEqual(res.pending_prefix, None)
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
     # Validator is disabled by default, even if it's not explicitely in disabled
     # refglobs.
@@ -1268,7 +1268,7 @@
         remote_url='https://chromium.googlesource.com/chromium/src',
         remote_ref='refs/arbitrary/ref')
     self.assertEqual(res.pending_prefix, None)
-    self.assertEqual(res.should_git_number, False)
+    self.assertEqual(res.should_add_git_number, False)
 
   @classmethod
   def _gerrit_ensure_auth_calls(cls, issue=None, skip_auth_check=False):