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