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