blob: 31f3c8f5c750e156d87e1285fc92e623d4d02c9b [file] [log] [blame]
Mike Frysinger6cb624a2012-05-24 18:17:38 -04001# Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Chris Sosadad0d322011-01-31 16:37:33 -08002# 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 Frysinger383367e2014-09-16 15:06:17 -04007from __future__ import print_function
8
Mike Frysingerea838d12014-12-08 11:55:32 -05009import mock
Chris Sosadad0d322011-01-31 16:37:33 -080010
J. Richard Barnetted422f622011-11-17 09:39:46 -080011from chromite.lib import cros_build_lib
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040012from chromite.lib import cros_build_lib_unittest
Brian Harringc92788f2012-09-21 18:07:15 -070013from chromite.lib import cros_test_lib
David James97d95872012-11-16 15:09:56 -080014from chromite.lib import git
David James59a0a2b2013-03-22 14:04:44 -070015from chromite.lib import osutils
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040016from chromite.lib import parallel_unittest
17from chromite.lib import partial_mock
Mike Frysinger6cb624a2012-05-24 18:17:38 -040018from chromite.scripts import cros_mark_as_stable
Chris Sosadad0d322011-01-31 16:37:33 -080019
Mike Frysingeref6ccb22014-11-07 14:59:48 -050020
Mike Frysingerc9785342014-12-08 00:47:08 -050021class 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 Frysingeref6ccb22014-11-07 14:59:48 -050030
31
Mike Frysingerc9785342014-12-08 00:47:08 -050032class NonClassTests(cros_test_lib.MockTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -070033 """Test the flow for pushing a change."""
Mike Frysingerc9785342014-12-08 00:47:08 -050034
Chris Sosadad0d322011-01-31 16:37:33 -080035 def setUp(self):
Chris Sosadad0d322011-01-31 16:37:33 -080036 self._branch = 'test_branch'
Brian Harringeb237932012-05-07 02:08:06 -070037 self._target_manifest_branch = 'cros/master'
Chris Sosadad0d322011-01-31 16:37:33 -080038
Matt Tennantcb522052013-11-25 14:23:43 -080039 def _TestPushChange(self, bad_cls):
Mike Frysingerc9785342014-12-08 00:47:08 -050040 side_effect = Exception('unittest says this should not be called')
41
Chris Sosadad0d322011-01-31 16:37:33 -080042 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 James47959632015-10-23 07:56:01 -070044 self.PatchObject(git, 'DoesCommitExistInRepo', return_value=True)
Mike Frysingerc9785342014-12-08 00:47:08 -050045 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 Sosadad0d322011-01-31 16:37:33 -080051
Mike Frysingerc9785342014-12-08 00:47:08 -050052 push_mock = self.PatchObject(git, 'PushWithRetry')
Don Garrett99449592015-03-25 11:01:30 -070053 self.PatchObject(
54 git, 'GetTrackingBranch',
55 return_value=git.RemoteRef('gerrit', 'refs/remotes/gerrit/master'))
Mike Frysingerc9785342014-12-08 00:47:08 -050056 sync_mock = self.PatchObject(git, 'SyncPushBranch')
57 create_mock = self.PatchObject(git, 'CreatePushBranch')
58 git_mock = self.StartPatcher(RunGitMock())
Matt Tennantcb522052013-11-25 14:23:43 -080059
Paul Hobbsf52ea8f2015-10-21 17:24:23 -070060 git_mock.AddCmdResult(['checkout', self._branch])
61
Matt Tennantcb522052013-11-25 14:23:43 -080062 cmd = ['log', '--format=short', '--perl-regexp', '--author',
63 '^(?!chrome-bot)', 'refs/remotes/gerrit/master..%s' % self._branch]
64
65 if bad_cls:
Mike Frysingerc9785342014-12-08 00:47:08 -050066 push_mock.side_effect = side_effect
67 create_mock.side_effect = side_effect
68 git_mock.AddCmdResult(cmd, output='Found bad stuff')
Matt Tennantcb522052013-11-25 14:23:43 -080069 else:
Mike Frysingerc9785342014-12-08 00:47:08 -050070 git_mock.AddCmdResult(cmd, output='\n')
Matt Tennantcb522052013-11-25 14:23:43 -080071 cmd = ['log', '--format=format:%s%n%n%b',
72 'refs/remotes/gerrit/master..%s' % self._branch]
Mike Frysingerc9785342014-12-08 00:47:08 -050073 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 Tennantcb522052013-11-25 14:23:43 -080077
Paul Hobbs72d8e392015-10-21 17:24:23 -070078 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 Xia52009062016-05-09 14:33:51 -070082 push_mock.assert_called_with('merge_branch', '.', dryrun=False,
83 staging_branch=None)
Paul Hobbsf52ea8f2015-10-21 17:24:23 -070084 create_mock.assert_called_with('merge_branch', '.',
85 remote_push_branch=mock.ANY)
Chris Sosadad0d322011-01-31 16:37:33 -080086
Matt Tennantcb522052013-11-25 14:23:43 -080087 def testPushChange(self):
Mike Frysingerc9785342014-12-08 00:47:08 -050088 """Verify pushing changes works."""
Matt Tennantcb522052013-11-25 14:23:43 -080089 self._TestPushChange(bad_cls=False)
90
91 def testPushChangeBadCls(self):
Mike Frysingerc9785342014-12-08 00:47:08 -050092 """Verify we do not push bad CLs."""
Matt Tennantcb522052013-11-25 14:23:43 -080093 self.assertRaises(AssertionError, self._TestPushChange, bad_cls=True)
94
Chris Sosadad0d322011-01-31 16:37:33 -080095
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040096class CleanStalePackagesTest(cros_build_lib_unittest.RunCommandTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -070097 """Tests for cros_mark_as_stable.CleanStalePackages."""
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040098
David James59a0a2b2013-03-22 14:04:44 -070099 def setUp(self):
100 self.PatchObject(osutils, 'FindMissingBinaries', return_value=[])
101
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400102 def testNormalClean(self):
103 """Clean up boards/packages with normal success"""
David James41124af2015-06-04 21:13:25 -0700104 cros_mark_as_stable.CleanStalePackages('.', ('board1', 'board2'),
105 ['cow', 'car'])
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400106
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 James41124af2015-06-04 21:13:25 -0700110 cros_mark_as_stable.CleanStalePackages('.', (), ['no/pkg'])
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400111
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 James59a0a2b2013-03-22 14:04:44 -0700117 cros_mark_as_stable.CleanStalePackages,
David James41124af2015-06-04 21:13:25 -0700118 '.', (), ['no/pkg'])
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400119
120
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500121class GitBranchTest(cros_test_lib.MockTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -0700122 """Tests for cros_mark_as_stable.GitBranch."""
Chris Sosadad0d322011-01-31 16:37:33 -0800123
124 def setUp(self):
Chris Sosadad0d322011-01-31 16:37:33 -0800125 # Always stub RunCommmand out as we use it in every method.
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500126 self.rc_mock = self.PatchObject(cros_build_lib, 'RunCommand')
127
Chris Sosadad0d322011-01-31 16:37:33 -0800128 self._branch_name = 'test_branch'
Brian Harringeb237932012-05-07 02:08:06 -0700129 self._target_manifest_branch = 'cros/test'
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500130 self._branch = cros_mark_as_stable.GitBranch(
131 branch_name=self._branch_name,
132 tracking_branch=self._target_manifest_branch,
133 cwd='.')
Chris Sosadad0d322011-01-31 16:37:33 -0800134
135 def testCheckoutCreate(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500136 """Test init with no previous branch existing."""
137 self.PatchObject(self._branch, 'Exists', return_value=False)
Chris Sosadad0d322011-01-31 16:37:33 -0800138 cros_mark_as_stable.GitBranch.Checkout(self._branch)
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500139 self.rc_mock.assert_call(mock.call(
140 ['repo', 'start', self._branch_name, '.'],
141 print_cmd=False, cwd='.', capture_output=True))
Chris Sosadad0d322011-01-31 16:37:33 -0800142
143 def testCheckoutNoCreate(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500144 """Test init with previous branch existing."""
145 self.PatchObject(self._branch, 'Exists', return_value=True)
Chris Sosadad0d322011-01-31 16:37:33 -0800146 cros_mark_as_stable.GitBranch.Checkout(self._branch)
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500147 self.rc_mock.assert_call(mock.call(
148 ['git', 'checkout', '-f', self._branch_name],
149 print_cmd=False, cwd='.', capture_output=True))
Chris Sosadad0d322011-01-31 16:37:33 -0800150
Chris Sosadad0d322011-01-31 16:37:33 -0800151 def testExists(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500152 """Test if branch exists that is created."""
Mike Frysinger7dafd0e2012-05-08 15:47:16 -0400153 result = cros_build_lib.CommandResult(output=self._branch_name + '\n')
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500154 self.PatchObject(git, 'RunGit', return_value=result)
155 self.assertTrue(self._branch.Exists())