Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 1 | #!/usr/bin/python |
| 2 | |
Mike Frysinger | 6cb624a | 2012-05-24 18:17:38 -0400 | [diff] [blame] | 3 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 4 | # Use of this source code is governed by a BSD-style license that can be |
| 5 | # found in the LICENSE file. |
| 6 | |
| 7 | """Unit tests for cros_mark_as_stable.py.""" |
| 8 | |
Mike Frysinger | 6cb624a | 2012-05-24 18:17:38 -0400 | [diff] [blame] | 9 | import os |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 10 | import sys |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 11 | |
Mike Frysinger | 6cb624a | 2012-05-24 18:17:38 -0400 | [diff] [blame] | 12 | sys.path.insert(0, os.path.join(os.path.dirname(os.path.realpath(__file__)), |
| 13 | '..', '..')) |
J. Richard Barnette | d422f62 | 2011-11-17 09:39:46 -0800 | [diff] [blame] | 14 | from chromite.lib import cros_build_lib |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 15 | from chromite.lib import cros_build_lib_unittest |
Brian Harring | c92788f | 2012-09-21 18:07:15 -0700 | [diff] [blame] | 16 | from chromite.lib import cros_test_lib |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 17 | from chromite.lib import git |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 18 | from chromite.lib import osutils |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 19 | from chromite.lib import parallel_unittest |
| 20 | from chromite.lib import partial_mock |
Mike Frysinger | 6cb624a | 2012-05-24 18:17:38 -0400 | [diff] [blame] | 21 | from chromite.scripts import cros_mark_as_stable |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 22 | |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 23 | |
David James | fba499d | 2011-10-19 10:47:21 -0700 | [diff] [blame] | 24 | # pylint: disable=W0212,R0904 |
Brian Harring | c92788f | 2012-09-21 18:07:15 -0700 | [diff] [blame] | 25 | class NonClassTests(cros_test_lib.MoxTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame^] | 26 | """Test the flow for pushing a change.""" |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 27 | def setUp(self): |
David James | 3a37309 | 2011-11-18 15:56:31 -0800 | [diff] [blame] | 28 | self.mox.StubOutWithMock(cros_build_lib, 'RunCommand') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 29 | self._branch = 'test_branch' |
Brian Harring | eb23793 | 2012-05-07 02:08:06 -0700 | [diff] [blame] | 30 | self._target_manifest_branch = 'cros/master' |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 31 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 32 | def _TestPushChange(self, bad_cls): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 33 | git_log = 'Marking test_one as stable\nMarking test_two as stable\n' |
| 34 | fake_description = 'Marking set of ebuilds as stable\n\n%s' % git_log |
| 35 | self.mox.StubOutWithMock(cros_mark_as_stable, '_DoWeHaveLocalCommits') |
| 36 | self.mox.StubOutWithMock(cros_mark_as_stable.GitBranch, 'CreateBranch') |
| 37 | self.mox.StubOutWithMock(cros_mark_as_stable.GitBranch, 'Exists') |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 38 | self.mox.StubOutWithMock(git, 'PushWithRetry') |
| 39 | self.mox.StubOutWithMock(git, 'GetTrackingBranch') |
| 40 | self.mox.StubOutWithMock(git, 'SyncPushBranch') |
| 41 | self.mox.StubOutWithMock(git, 'CreatePushBranch') |
| 42 | self.mox.StubOutWithMock(git, 'RunGit') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 43 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 44 | # Run the flow. |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 45 | cros_mark_as_stable._DoWeHaveLocalCommits( |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 46 | self._branch, self._target_manifest_branch, '.').AndReturn(True) |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 47 | git.GetTrackingBranch('.', for_push=True).AndReturn( |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 48 | ['gerrit', 'refs/remotes/gerrit/master']) |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 49 | git.SyncPushBranch('.', 'gerrit', 'refs/remotes/gerrit/master') |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 50 | cros_mark_as_stable._DoWeHaveLocalCommits( |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 51 | self._branch, 'refs/remotes/gerrit/master', '.').AndReturn(True) |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 52 | |
| 53 | # Look for bad CLs. |
| 54 | cmd = ['log', '--format=short', '--perl-regexp', '--author', |
| 55 | '^(?!chrome-bot)', 'refs/remotes/gerrit/master..%s' % self._branch] |
| 56 | |
| 57 | if bad_cls: |
| 58 | result = cros_build_lib.CommandResult(output='Found bad stuff') |
| 59 | git.RunGit('.', cmd).AndReturn(result) |
| 60 | else: |
| 61 | result = cros_build_lib.CommandResult(output='\n') |
| 62 | git.RunGit('.', cmd).AndReturn(result) |
| 63 | result = cros_build_lib.CommandResult(output=git_log) |
| 64 | cmd = ['log', '--format=format:%s%n%n%b', |
| 65 | 'refs/remotes/gerrit/master..%s' % self._branch] |
| 66 | git.RunGit('.', cmd).AndReturn(result) |
| 67 | git.CreatePushBranch('merge_branch', '.') |
| 68 | git.RunGit('.', ['merge', '--squash', self._branch]) |
| 69 | git.RunGit('.', ['commit', '-m', fake_description]) |
| 70 | git.RunGit('.', ['config', 'push.default', 'tracking']) |
| 71 | git.PushWithRetry('merge_branch', '.', dryrun=False) |
| 72 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 73 | self.mox.ReplayAll() |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 74 | cros_mark_as_stable.PushChange(self._branch, self._target_manifest_branch, |
| 75 | False, '.') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 76 | self.mox.VerifyAll() |
| 77 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 78 | def testPushChange(self): |
| 79 | self._TestPushChange(bad_cls=False) |
| 80 | |
| 81 | def testPushChangeBadCls(self): |
| 82 | self.assertRaises(AssertionError, self._TestPushChange, bad_cls=True) |
| 83 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 84 | |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 85 | class CleanStalePackagesTest(cros_build_lib_unittest.RunCommandTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame^] | 86 | """Tests for cros_mark_as_stable.CleanStalePackages.""" |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 87 | |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 88 | def setUp(self): |
| 89 | self.PatchObject(osutils, 'FindMissingBinaries', return_value=[]) |
| 90 | |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 91 | def testNormalClean(self): |
| 92 | """Clean up boards/packages with normal success""" |
| 93 | cros_mark_as_stable.CleanStalePackages(('board1', 'board2'), ['cow', 'car']) |
| 94 | |
| 95 | def testNothingToUnmerge(self): |
| 96 | """Clean up packages that don't exist (portage will exit 1)""" |
| 97 | self.rc.AddCmdResult(partial_mock.In('emerge'), returncode=1) |
| 98 | cros_mark_as_stable.CleanStalePackages((), ['no/pkg']) |
| 99 | |
| 100 | def testUnmergeError(self): |
| 101 | """Make sure random exit errors are not ignored""" |
| 102 | self.rc.AddCmdResult(partial_mock.In('emerge'), returncode=123) |
| 103 | with parallel_unittest.ParallelMock(): |
| 104 | self.assertRaises(cros_build_lib.RunCommandError, |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 105 | cros_mark_as_stable.CleanStalePackages, |
| 106 | (), ['no/pkg']) |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 107 | |
| 108 | |
Brian Harring | c92788f | 2012-09-21 18:07:15 -0700 | [diff] [blame] | 109 | class GitBranchTest(cros_test_lib.MoxTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame^] | 110 | """Tests for cros_mark_as_stable.GitBranch.""" |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 111 | |
| 112 | def setUp(self): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 113 | # Always stub RunCommmand out as we use it in every method. |
Mike Frysinger | 7dafd0e | 2012-05-08 15:47:16 -0400 | [diff] [blame] | 114 | self.mox.StubOutWithMock(cros_build_lib, 'RunCommand') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 115 | self._branch = self.mox.CreateMock(cros_mark_as_stable.GitBranch) |
| 116 | self._branch_name = 'test_branch' |
| 117 | self._branch.branch_name = self._branch_name |
Brian Harring | eb23793 | 2012-05-07 02:08:06 -0700 | [diff] [blame] | 118 | self._target_manifest_branch = 'cros/test' |
| 119 | self._branch.tracking_branch = self._target_manifest_branch |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 120 | self._branch.cwd = '.' |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 121 | |
| 122 | def testCheckoutCreate(self): |
| 123 | # Test init with no previous branch existing. |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 124 | self._branch.Exists(self._branch_name).AndReturn(False) |
Yu-Ju Hong | 3add443 | 2014-01-30 11:46:15 -0800 | [diff] [blame] | 125 | cros_build_lib.RunCommand(['repo', 'start', self._branch_name, '.'], |
| 126 | print_cmd=False, cwd='.', capture_output=True) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 127 | self.mox.ReplayAll() |
| 128 | cros_mark_as_stable.GitBranch.Checkout(self._branch) |
| 129 | self.mox.VerifyAll() |
| 130 | |
| 131 | def testCheckoutNoCreate(self): |
| 132 | # Test init with previous branch existing. |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 133 | self._branch.Exists(self._branch_name).AndReturn(True) |
Yu-Ju Hong | 3add443 | 2014-01-30 11:46:15 -0800 | [diff] [blame] | 134 | cros_build_lib.RunCommand(['git', 'checkout', '-f', self._branch_name], |
| 135 | print_cmd=False, cwd='.', capture_output=True) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 136 | self.mox.ReplayAll() |
| 137 | cros_mark_as_stable.GitBranch.Checkout(self._branch) |
| 138 | self.mox.VerifyAll() |
| 139 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 140 | def testExists(self): |
| 141 | branch = cros_mark_as_stable.GitBranch(self._branch_name, |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 142 | self._target_manifest_branch, '.') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 143 | # Test if branch exists that is created |
Mike Frysinger | 7dafd0e | 2012-05-08 15:47:16 -0400 | [diff] [blame] | 144 | result = cros_build_lib.CommandResult(output=self._branch_name + '\n') |
David James | 67d7325 | 2013-09-19 17:33:12 -0700 | [diff] [blame] | 145 | git.RunGit('.', ['branch']).AndReturn(result) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 146 | self.mox.ReplayAll() |
| 147 | self.assertTrue(branch.Exists()) |
| 148 | self.mox.VerifyAll() |
| 149 | |
| 150 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 151 | if __name__ == '__main__': |
Brian Harring | c92788f | 2012-09-21 18:07:15 -0700 | [diff] [blame] | 152 | cros_test_lib.main() |