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 | |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 24 | # TODO(build): Finish test wrapper (http://crosbug.com/37517). |
| 25 | # Until then, this has to be after the chromite imports. |
| 26 | import mock |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 27 | |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 28 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 29 | class RunGitMock(partial_mock.PartialCmdMock): |
| 30 | """Partial mock for git.RunMock.""" |
| 31 | TARGET = 'chromite.lib.git' |
| 32 | ATTRS = ('RunGit',) |
| 33 | DEFAULT_ATTR = 'RunGit' |
| 34 | |
| 35 | def RunGit(self, _git_repo, cmd, _retry=True, **kwargs): |
| 36 | return self._results['RunGit'].LookupResult( |
| 37 | (cmd,), hook_args=(cmd,), hook_kwargs=kwargs) |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 38 | |
| 39 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 40 | class NonClassTests(cros_test_lib.MockTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 41 | """Test the flow for pushing a change.""" |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 42 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 43 | def setUp(self): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 44 | self._branch = 'test_branch' |
Brian Harring | eb23793 | 2012-05-07 02:08:06 -0700 | [diff] [blame] | 45 | self._target_manifest_branch = 'cros/master' |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 46 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 47 | def _TestPushChange(self, bad_cls): |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 48 | side_effect = Exception('unittest says this should not be called') |
| 49 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 50 | git_log = 'Marking test_one as stable\nMarking test_two as stable\n' |
| 51 | fake_description = 'Marking set of ebuilds as stable\n\n%s' % git_log |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 52 | self.PatchObject(cros_mark_as_stable, '_DoWeHaveLocalCommits', |
| 53 | return_value=True) |
| 54 | self.PatchObject(cros_mark_as_stable.GitBranch, 'CreateBranch', |
| 55 | side_effect=side_effect) |
| 56 | self.PatchObject(cros_mark_as_stable.GitBranch, 'Exists', |
| 57 | side_effect=side_effect) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 58 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 59 | push_mock = self.PatchObject(git, 'PushWithRetry') |
| 60 | self.PatchObject(git, 'GetTrackingBranch', |
| 61 | return_value=['gerrit', 'refs/remotes/gerrit/master']) |
| 62 | sync_mock = self.PatchObject(git, 'SyncPushBranch') |
| 63 | create_mock = self.PatchObject(git, 'CreatePushBranch') |
| 64 | git_mock = self.StartPatcher(RunGitMock()) |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 65 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 66 | cmd = ['log', '--format=short', '--perl-regexp', '--author', |
| 67 | '^(?!chrome-bot)', 'refs/remotes/gerrit/master..%s' % self._branch] |
| 68 | |
| 69 | if bad_cls: |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 70 | push_mock.side_effect = side_effect |
| 71 | create_mock.side_effect = side_effect |
| 72 | git_mock.AddCmdResult(cmd, output='Found bad stuff') |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 73 | else: |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 74 | git_mock.AddCmdResult(cmd, output='\n') |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 75 | cmd = ['log', '--format=format:%s%n%n%b', |
| 76 | 'refs/remotes/gerrit/master..%s' % self._branch] |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 77 | git_mock.AddCmdResult(cmd, output=git_log) |
| 78 | git_mock.AddCmdResult(['merge', '--squash', self._branch]) |
| 79 | git_mock.AddCmdResult(['commit', '-m', fake_description]) |
| 80 | git_mock.AddCmdResult(['config', 'push.default', 'tracking']) |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 81 | |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 82 | try: |
| 83 | cros_mark_as_stable.PushChange(self._branch, self._target_manifest_branch, |
| 84 | False, '.') |
| 85 | finally: |
| 86 | sync_mock.assert_called_with('.', 'gerrit', 'refs/remotes/gerrit/master') |
| 87 | if not bad_cls: |
| 88 | push_mock.assert_called_with('merge_branch', '.', dryrun=False) |
| 89 | create_mock.assert_called_with('merge_branch', '.') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 90 | |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 91 | def testPushChange(self): |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 92 | """Verify pushing changes works.""" |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 93 | self._TestPushChange(bad_cls=False) |
| 94 | |
| 95 | def testPushChangeBadCls(self): |
Mike Frysinger | c978534 | 2014-12-08 00:47:08 -0500 | [diff] [blame^] | 96 | """Verify we do not push bad CLs.""" |
Matt Tennant | cb52205 | 2013-11-25 14:23:43 -0800 | [diff] [blame] | 97 | self.assertRaises(AssertionError, self._TestPushChange, bad_cls=True) |
| 98 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 99 | |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 100 | class CleanStalePackagesTest(cros_build_lib_unittest.RunCommandTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 101 | """Tests for cros_mark_as_stable.CleanStalePackages.""" |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 102 | |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 103 | def setUp(self): |
| 104 | self.PatchObject(osutils, 'FindMissingBinaries', return_value=[]) |
| 105 | |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 106 | def testNormalClean(self): |
| 107 | """Clean up boards/packages with normal success""" |
| 108 | cros_mark_as_stable.CleanStalePackages(('board1', 'board2'), ['cow', 'car']) |
| 109 | |
| 110 | def testNothingToUnmerge(self): |
| 111 | """Clean up packages that don't exist (portage will exit 1)""" |
| 112 | self.rc.AddCmdResult(partial_mock.In('emerge'), returncode=1) |
| 113 | cros_mark_as_stable.CleanStalePackages((), ['no/pkg']) |
| 114 | |
| 115 | def testUnmergeError(self): |
| 116 | """Make sure random exit errors are not ignored""" |
| 117 | self.rc.AddCmdResult(partial_mock.In('emerge'), returncode=123) |
| 118 | with parallel_unittest.ParallelMock(): |
| 119 | self.assertRaises(cros_build_lib.RunCommandError, |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 120 | cros_mark_as_stable.CleanStalePackages, |
| 121 | (), ['no/pkg']) |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 122 | |
| 123 | |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 124 | class GitBranchTest(cros_test_lib.MockTestCase): |
Don Garrett | 25f309a | 2014-03-19 14:02:12 -0700 | [diff] [blame] | 125 | """Tests for cros_mark_as_stable.GitBranch.""" |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 126 | |
| 127 | def setUp(self): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 128 | # Always stub RunCommmand out as we use it in every method. |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 129 | self.rc_mock = self.PatchObject(cros_build_lib, 'RunCommand') |
| 130 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 131 | self._branch_name = 'test_branch' |
Brian Harring | eb23793 | 2012-05-07 02:08:06 -0700 | [diff] [blame] | 132 | self._target_manifest_branch = 'cros/test' |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 133 | self._branch = cros_mark_as_stable.GitBranch( |
| 134 | branch_name=self._branch_name, |
| 135 | tracking_branch=self._target_manifest_branch, |
| 136 | cwd='.') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 137 | |
| 138 | def testCheckoutCreate(self): |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 139 | """Test init with no previous branch existing.""" |
| 140 | self.PatchObject(self._branch, 'Exists', return_value=False) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 141 | cros_mark_as_stable.GitBranch.Checkout(self._branch) |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 142 | self.rc_mock.assert_call(mock.call( |
| 143 | ['repo', 'start', self._branch_name, '.'], |
| 144 | print_cmd=False, cwd='.', capture_output=True)) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 145 | |
| 146 | def testCheckoutNoCreate(self): |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 147 | """Test init with previous branch existing.""" |
| 148 | self.PatchObject(self._branch, 'Exists', return_value=True) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 149 | cros_mark_as_stable.GitBranch.Checkout(self._branch) |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 150 | self.rc_mock.assert_call(mock.call( |
| 151 | ['git', 'checkout', '-f', self._branch_name], |
| 152 | print_cmd=False, cwd='.', capture_output=True)) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 153 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 154 | def testExists(self): |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 155 | """Test if branch exists that is created.""" |
Mike Frysinger | 7dafd0e | 2012-05-08 15:47:16 -0400 | [diff] [blame] | 156 | result = cros_build_lib.CommandResult(output=self._branch_name + '\n') |
Mike Frysinger | ef6ccb2 | 2014-11-07 14:59:48 -0500 | [diff] [blame] | 157 | self.PatchObject(git, 'RunGit', return_value=result) |
| 158 | self.assertTrue(self._branch.Exists()) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 159 | |
| 160 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 161 | if __name__ == '__main__': |
Brian Harring | c92788f | 2012-09-21 18:07:15 -0700 | [diff] [blame] | 162 | cros_test_lib.main() |