blob: 5b1329e1d7fc2569b08eb59d64043a2653d05911 [file] [log] [blame]
Chris Sosadad0d322011-01-31 16:37:33 -08001#!/usr/bin/python
Mike Frysinger6cb624a2012-05-24 18:17:38 -04002# Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Chris Sosadad0d322011-01-31 16:37:33 -08003# 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 Frysinger383367e2014-09-16 15:06:17 -04008from __future__ import print_function
9
Mike Frysinger6cb624a2012-05-24 18:17:38 -040010import os
Chris Sosadad0d322011-01-31 16:37:33 -080011import sys
Chris Sosadad0d322011-01-31 16:37:33 -080012
Mike Frysinger6cb624a2012-05-24 18:17:38 -040013sys.path.insert(0, os.path.join(os.path.dirname(os.path.realpath(__file__)),
14 '..', '..'))
J. Richard Barnetted422f622011-11-17 09:39:46 -080015from chromite.lib import cros_build_lib
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040016from chromite.lib import cros_build_lib_unittest
Brian Harringc92788f2012-09-21 18:07:15 -070017from chromite.lib import cros_test_lib
David James97d95872012-11-16 15:09:56 -080018from chromite.lib import git
David James59a0a2b2013-03-22 14:04:44 -070019from chromite.lib import osutils
Mike Frysingerde5ab0e2013-03-21 20:48:36 -040020from chromite.lib import parallel_unittest
21from chromite.lib import partial_mock
Mike Frysinger6cb624a2012-05-24 18:17:38 -040022from chromite.scripts import cros_mark_as_stable
Chris Sosadad0d322011-01-31 16:37:33 -080023
Mike Frysingeref6ccb22014-11-07 14:59:48 -050024# TODO(build): Finish test wrapper (http://crosbug.com/37517).
25# Until then, this has to be after the chromite imports.
26import mock
Chris Sosa62ad8522011-03-08 17:46:17 -080027
Mike Frysingeref6ccb22014-11-07 14:59:48 -050028
Mike Frysingerc9785342014-12-08 00:47:08 -050029class 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 Frysingeref6ccb22014-11-07 14:59:48 -050038
39
Mike Frysingerc9785342014-12-08 00:47:08 -050040class NonClassTests(cros_test_lib.MockTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -070041 """Test the flow for pushing a change."""
Mike Frysingerc9785342014-12-08 00:47:08 -050042
Chris Sosadad0d322011-01-31 16:37:33 -080043 def setUp(self):
Chris Sosadad0d322011-01-31 16:37:33 -080044 self._branch = 'test_branch'
Brian Harringeb237932012-05-07 02:08:06 -070045 self._target_manifest_branch = 'cros/master'
Chris Sosadad0d322011-01-31 16:37:33 -080046
Matt Tennantcb522052013-11-25 14:23:43 -080047 def _TestPushChange(self, bad_cls):
Mike Frysingerc9785342014-12-08 00:47:08 -050048 side_effect = Exception('unittest says this should not be called')
49
Chris Sosadad0d322011-01-31 16:37:33 -080050 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 Frysingerc9785342014-12-08 00:47:08 -050052 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 Sosadad0d322011-01-31 16:37:33 -080058
Mike Frysingerc9785342014-12-08 00:47:08 -050059 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 Tennantcb522052013-11-25 14:23:43 -080065
Matt Tennantcb522052013-11-25 14:23:43 -080066 cmd = ['log', '--format=short', '--perl-regexp', '--author',
67 '^(?!chrome-bot)', 'refs/remotes/gerrit/master..%s' % self._branch]
68
69 if bad_cls:
Mike Frysingerc9785342014-12-08 00:47:08 -050070 push_mock.side_effect = side_effect
71 create_mock.side_effect = side_effect
72 git_mock.AddCmdResult(cmd, output='Found bad stuff')
Matt Tennantcb522052013-11-25 14:23:43 -080073 else:
Mike Frysingerc9785342014-12-08 00:47:08 -050074 git_mock.AddCmdResult(cmd, output='\n')
Matt Tennantcb522052013-11-25 14:23:43 -080075 cmd = ['log', '--format=format:%s%n%n%b',
76 'refs/remotes/gerrit/master..%s' % self._branch]
Mike Frysingerc9785342014-12-08 00:47:08 -050077 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 Tennantcb522052013-11-25 14:23:43 -080081
Mike Frysingerc9785342014-12-08 00:47:08 -050082 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 Sosadad0d322011-01-31 16:37:33 -080090
Matt Tennantcb522052013-11-25 14:23:43 -080091 def testPushChange(self):
Mike Frysingerc9785342014-12-08 00:47:08 -050092 """Verify pushing changes works."""
Matt Tennantcb522052013-11-25 14:23:43 -080093 self._TestPushChange(bad_cls=False)
94
95 def testPushChangeBadCls(self):
Mike Frysingerc9785342014-12-08 00:47:08 -050096 """Verify we do not push bad CLs."""
Matt Tennantcb522052013-11-25 14:23:43 -080097 self.assertRaises(AssertionError, self._TestPushChange, bad_cls=True)
98
Chris Sosadad0d322011-01-31 16:37:33 -080099
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400100class CleanStalePackagesTest(cros_build_lib_unittest.RunCommandTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -0700101 """Tests for cros_mark_as_stable.CleanStalePackages."""
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400102
David James59a0a2b2013-03-22 14:04:44 -0700103 def setUp(self):
104 self.PatchObject(osutils, 'FindMissingBinaries', return_value=[])
105
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400106 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 James59a0a2b2013-03-22 14:04:44 -0700120 cros_mark_as_stable.CleanStalePackages,
121 (), ['no/pkg'])
Mike Frysingerde5ab0e2013-03-21 20:48:36 -0400122
123
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500124class GitBranchTest(cros_test_lib.MockTestCase):
Don Garrett25f309a2014-03-19 14:02:12 -0700125 """Tests for cros_mark_as_stable.GitBranch."""
Chris Sosadad0d322011-01-31 16:37:33 -0800126
127 def setUp(self):
Chris Sosadad0d322011-01-31 16:37:33 -0800128 # Always stub RunCommmand out as we use it in every method.
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500129 self.rc_mock = self.PatchObject(cros_build_lib, 'RunCommand')
130
Chris Sosadad0d322011-01-31 16:37:33 -0800131 self._branch_name = 'test_branch'
Brian Harringeb237932012-05-07 02:08:06 -0700132 self._target_manifest_branch = 'cros/test'
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500133 self._branch = cros_mark_as_stable.GitBranch(
134 branch_name=self._branch_name,
135 tracking_branch=self._target_manifest_branch,
136 cwd='.')
Chris Sosadad0d322011-01-31 16:37:33 -0800137
138 def testCheckoutCreate(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500139 """Test init with no previous branch existing."""
140 self.PatchObject(self._branch, 'Exists', return_value=False)
Chris Sosadad0d322011-01-31 16:37:33 -0800141 cros_mark_as_stable.GitBranch.Checkout(self._branch)
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500142 self.rc_mock.assert_call(mock.call(
143 ['repo', 'start', self._branch_name, '.'],
144 print_cmd=False, cwd='.', capture_output=True))
Chris Sosadad0d322011-01-31 16:37:33 -0800145
146 def testCheckoutNoCreate(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500147 """Test init with previous branch existing."""
148 self.PatchObject(self._branch, 'Exists', return_value=True)
Chris Sosadad0d322011-01-31 16:37:33 -0800149 cros_mark_as_stable.GitBranch.Checkout(self._branch)
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500150 self.rc_mock.assert_call(mock.call(
151 ['git', 'checkout', '-f', self._branch_name],
152 print_cmd=False, cwd='.', capture_output=True))
Chris Sosadad0d322011-01-31 16:37:33 -0800153
Chris Sosadad0d322011-01-31 16:37:33 -0800154 def testExists(self):
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500155 """Test if branch exists that is created."""
Mike Frysinger7dafd0e2012-05-08 15:47:16 -0400156 result = cros_build_lib.CommandResult(output=self._branch_name + '\n')
Mike Frysingeref6ccb22014-11-07 14:59:48 -0500157 self.PatchObject(git, 'RunGit', return_value=result)
158 self.assertTrue(self._branch.Exists())
Chris Sosadad0d322011-01-31 16:37:33 -0800159
160
Chris Sosadad0d322011-01-31 16:37:33 -0800161if __name__ == '__main__':
Brian Harringc92788f2012-09-21 18:07:15 -0700162 cros_test_lib.main()