| OLD | NEW |
| 1 #!/usr/bin/env python | 1 #!/usr/bin/env python |
| 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 3 # Use of this source code is governed by a BSD-style license that can be | 3 # Use of this source code is governed by a BSD-style license that can be |
| 4 # found in the LICENSE file. | 4 # found in the LICENSE file. |
| 5 | 5 |
| 6 """Unit tests for gcl.py.""" | 6 """Unit tests for gcl.py.""" |
| 7 | 7 |
| 8 # pylint: disable=E1103,E1101,E1120 | 8 # pylint: disable=E1103,E1101,E1120 |
| 9 | 9 |
| 10 import os | 10 import os |
| (...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 49 change_info.files = files | 49 change_info.files = files |
| 50 change_info.GetFiles = lambda : change_info.files | 50 change_info.GetFiles = lambda : change_info.files |
| 51 change_info.GetIssueDescription = lambda : change_info.description | 51 change_info.GetIssueDescription = lambda : change_info.description |
| 52 change_info.GetFileNames = lambda : [f[1] for f in change_info.files] | 52 change_info.GetFileNames = lambda : [f[1] for f in change_info.files] |
| 53 change_info.GetLocalRoot = lambda : 'proout' | 53 change_info.GetLocalRoot = lambda : 'proout' |
| 54 change_info.patch = None | 54 change_info.patch = None |
| 55 change_info.rietveld = 'https://my_server' | 55 change_info.rietveld = 'https://my_server' |
| 56 change_info.reviewers = None | 56 change_info.reviewers = None |
| 57 change_info._closed = False | 57 change_info._closed = False |
| 58 change_info._deleted = False | 58 change_info._deleted = False |
| 59 change_info._comments_added = [] |
| 60 |
| 61 def AddComment(comment): |
| 62 # pylint: disable=W0212 |
| 63 change_info._comments_added.append(comment) |
| 64 change_info.AddComment = AddComment |
| 59 | 65 |
| 60 def Delete(): | 66 def Delete(): |
| 61 change_info._deleted = True | 67 change_info._deleted = True |
| 62 change_info.Delete = Delete | 68 change_info.Delete = Delete |
| 63 | 69 |
| 64 def CloseIssue(): | 70 def CloseIssue(): |
| 65 change_info._closed = True | 71 change_info._closed = True |
| 66 change_info.CloseIssue = CloseIssue | 72 change_info.CloseIssue = CloseIssue |
| 67 | 73 |
| 68 return change_info | 74 return change_info |
| (...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 173 | 179 |
| 174 class ChangeInfoUnittest(GclTestsBase): | 180 class ChangeInfoUnittest(GclTestsBase): |
| 175 def setUp(self): | 181 def setUp(self): |
| 176 GclTestsBase.setUp(self) | 182 GclTestsBase.setUp(self) |
| 177 self.mox.StubOutWithMock(gcl, 'GetChangelistInfoFile') | 183 self.mox.StubOutWithMock(gcl, 'GetChangelistInfoFile') |
| 178 self.mox.StubOutWithMock(gcl, 'GetRepositoryRoot') | 184 self.mox.StubOutWithMock(gcl, 'GetRepositoryRoot') |
| 179 | 185 |
| 180 def testChangeInfoMembers(self): | 186 def testChangeInfoMembers(self): |
| 181 self.mox.ReplayAll() | 187 self.mox.ReplayAll() |
| 182 members = [ | 188 members = [ |
| 183 'CloseIssue', 'Delete', 'Exists', 'GetFiles', 'GetFileNames', | 189 'AddComment', 'CloseIssue', 'Delete', 'Exists', 'GetFiles', |
| 184 'GetLocalRoot', 'GetIssueDescription', 'Load', 'MissingTests', | 190 'GetFileNames', 'GetLocalRoot', 'GetIssueDescription', 'Load', |
| 185 'NeedsUpload', 'PrimeLint', 'RpcServer', 'Save', 'SendToRietveld', | 191 'MissingTests', 'NeedsUpload', 'PrimeLint', 'RpcServer', 'Save', |
| 192 'SendToRietveld', |
| 186 'SEPARATOR', | 193 'SEPARATOR', |
| 187 'UpdateRietveldDescription', | 194 'UpdateRietveldDescription', |
| 188 'description', 'issue', 'name', | 195 'description', 'issue', 'name', |
| 189 'needs_upload', 'patch', 'patchset', 'reviewers', 'rietveld', | 196 'needs_upload', 'patch', 'patchset', 'reviewers', 'rietveld', |
| 190 ] | 197 ] |
| 191 # If this test fails, you should add the relevant test. | 198 # If this test fails, you should add the relevant test. |
| 192 self.compareMembers( | 199 self.compareMembers( |
| 193 gcl.ChangeInfo('', 0, 0, '', None, self.fake_root_dir, 'foo', False), | 200 gcl.ChangeInfo('', 0, 0, '', None, self.fake_root_dir, 'foo', False), |
| 194 members) | 201 members) |
| 195 | 202 |
| (...skipping 383 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 579 | 586 |
| 580 self.mox.ReplayAll() | 587 self.mox.ReplayAll() |
| 581 | 588 |
| 582 retval = gcl.CMDcommit(['naame']) | 589 retval = gcl.CMDcommit(['naame']) |
| 583 self.assertEquals(retval, 0) | 590 self.assertEquals(retval, 0) |
| 584 self.assertEquals(change_info.description, | 591 self.assertEquals(change_info.description, |
| 585 'deescription\n\nCommitted: http://view/12345') | 592 'deescription\n\nCommitted: http://view/12345') |
| 586 # pylint: disable=W0212 | 593 # pylint: disable=W0212 |
| 587 self.assertTrue(change_info._deleted) | 594 self.assertTrue(change_info._deleted) |
| 588 self.assertTrue(change_info._closed) | 595 self.assertTrue(change_info._closed) |
| 596 self.assertEqual( |
| 597 change_info._comments_added, |
| 598 ["Committed manually as r12345 (presubmit successful)."]) |
| 589 | 599 |
| 590 | 600 |
| 591 if __name__ == '__main__': | 601 if __name__ == '__main__': |
| 592 import unittest | 602 import unittest |
| 593 unittest.main() | 603 unittest.main() |
| OLD | NEW |