Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 1 | # Copyright (c) 2013 The Chromium OS Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Unittests for pushimage.py""" |
| 6 | |
Mike Frysinger | 6791b1d | 2014-03-04 20:52:22 -0500 | [diff] [blame] | 7 | from __future__ import print_function |
| 8 | |
Don Garrett | 9fd20a8 | 2014-09-04 11:37:22 -0700 | [diff] [blame] | 9 | import mock |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 10 | import os |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 11 | |
Mike Frysinger | 6791b1d | 2014-03-04 20:52:22 -0500 | [diff] [blame] | 12 | from chromite.lib import cros_build_lib |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 13 | from chromite.lib import cros_test_lib |
Don Garrett | 9fd20a8 | 2014-09-04 11:37:22 -0700 | [diff] [blame] | 14 | from chromite.lib import gs |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 15 | from chromite.lib import gs_unittest |
| 16 | from chromite.lib import osutils |
Mike Frysinger | 4495b03 | 2014-03-05 17:24:03 -0500 | [diff] [blame] | 17 | from chromite.lib import partial_mock |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 18 | from chromite.lib import signing |
| 19 | from chromite.scripts import pushimage |
| 20 | |
| 21 | |
| 22 | class InputInsnsTest(cros_test_lib.MockTestCase): |
| 23 | """Tests for InputInsns""" |
| 24 | |
Don Garrett | ac52977 | 2015-04-29 15:09:26 -0700 | [diff] [blame] | 25 | def setUp(self): |
| 26 | self.StartPatcher(gs_unittest.GSContextMock()) |
| 27 | |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 28 | def testBasic(self): |
| 29 | """Simple smoke test""" |
Don Garrett | ac52977 | 2015-04-29 15:09:26 -0700 | [diff] [blame] | 30 | insns = pushimage.InputInsns('test.board') |
| 31 | insns.GetInsnFile('recovery') |
| 32 | self.assertEqual(insns.GetChannels(), ['dev', 'canary']) |
| 33 | self.assertEqual(insns.GetKeysets(), ['stumpy-mp-v3']) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 34 | |
| 35 | def testGetInsnFile(self): |
| 36 | """Verify various inputs result in right insns path""" |
| 37 | testdata = ( |
| 38 | ('UPPER_CAPS', 'UPPER_CAPS'), |
| 39 | ('recovery', 'test.board'), |
| 40 | ('firmware', 'test.board.firmware'), |
| 41 | ('factory', 'test.board.factory'), |
| 42 | ) |
| 43 | insns = pushimage.InputInsns('test.board') |
| 44 | for image_type, filename in testdata: |
| 45 | ret = insns.GetInsnFile(image_type) |
| 46 | self.assertEqual(os.path.basename(ret), '%s.instructions' % (filename)) |
| 47 | |
| 48 | def testSplitCfgField(self): |
| 49 | """Verify splitting behavior behaves""" |
| 50 | testdata = ( |
| 51 | ('', []), |
| 52 | ('a b c', ['a', 'b', 'c']), |
| 53 | ('a, b', ['a', 'b']), |
| 54 | ('a,b', ['a', 'b']), |
| 55 | ('a,\tb', ['a', 'b']), |
| 56 | ('a\tb', ['a', 'b']), |
| 57 | ) |
| 58 | for val, exp in testdata: |
| 59 | ret = pushimage.InputInsns.SplitCfgField(val) |
| 60 | self.assertEqual(ret, exp) |
| 61 | |
| 62 | def testOutputInsnsBasic(self): |
| 63 | """Verify output instructions are sane""" |
| 64 | exp_content = """[insns] |
| 65 | keyset = stumpy-mp-v3 |
| 66 | channel = dev canary |
| 67 | chromeos_shell = false |
| 68 | ensure_no_password = true |
| 69 | firmware_update = true |
| 70 | security_checks = true |
| 71 | create_nplusone = true |
| 72 | |
| 73 | [general] |
| 74 | """ |
| 75 | |
| 76 | insns = pushimage.InputInsns('test.board') |
| 77 | m = self.PatchObject(osutils, 'WriteFile') |
| 78 | insns.OutputInsns('recovery', '/bogus', {}, {}) |
| 79 | self.assertTrue(m.called) |
| 80 | content = m.call_args_list[0][0][1] |
| 81 | self.assertEqual(content.rstrip(), exp_content.rstrip()) |
| 82 | |
| 83 | def testOutputInsnsReplacements(self): |
| 84 | """Verify output instructions can be updated""" |
| 85 | exp_content = """[insns] |
| 86 | keyset = batman |
| 87 | channel = dev |
| 88 | chromeos_shell = false |
| 89 | ensure_no_password = true |
| 90 | firmware_update = true |
| 91 | security_checks = true |
| 92 | create_nplusone = true |
| 93 | |
| 94 | [general] |
| 95 | board = board |
| 96 | config_board = test.board |
| 97 | """ |
| 98 | sect_insns = { |
| 99 | 'channel': 'dev', |
| 100 | 'keyset': 'batman', |
| 101 | } |
| 102 | sect_general = { |
| 103 | 'config_board': 'test.board', |
| 104 | 'board': 'board', |
| 105 | } |
| 106 | |
| 107 | insns = pushimage.InputInsns('test.board') |
| 108 | m = self.PatchObject(osutils, 'WriteFile') |
| 109 | insns.OutputInsns('recovery', '/a/file', sect_insns, sect_general) |
| 110 | self.assertTrue(m.called) |
| 111 | content = m.call_args_list[0][0][1] |
| 112 | self.assertEqual(content.rstrip(), exp_content.rstrip()) |
| 113 | |
| 114 | |
Mike Frysinger | 4495b03 | 2014-03-05 17:24:03 -0500 | [diff] [blame] | 115 | class MarkImageToBeSignedTest(gs_unittest.AbstractGSContextTest): |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 116 | """Tests for MarkImageToBeSigned()""" |
| 117 | |
| 118 | def setUp(self): |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 119 | # Minor optimization -- we call this for logging purposes in the main |
| 120 | # code, but don't really care about it for testing. It just slows us. |
Mike Frysinger | 6430d13 | 2014-10-27 23:43:30 -0400 | [diff] [blame] | 121 | self.PatchObject(cros_build_lib, 'MachineDetails', return_value='1234\n') |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 122 | |
| 123 | def testBasic(self): |
| 124 | """Simple smoke test""" |
| 125 | tbs_base = 'gs://some-bucket' |
| 126 | insns_path = 'chan/board/ver/file.instructions' |
| 127 | tbs_file = '%s/tobesigned/90,chan,board,ver,file.instructions' % tbs_base |
| 128 | ret = pushimage.MarkImageToBeSigned(self.ctx, tbs_base, insns_path, 90) |
| 129 | self.assertEqual(ret, tbs_file) |
| 130 | |
| 131 | def testPriority(self): |
| 132 | """Verify diff priority values get used correctly""" |
| 133 | for prio, sprio in ((0, '00'), (9, '09'), (35, '35'), (99, '99')): |
| 134 | ret = pushimage.MarkImageToBeSigned(self.ctx, '', '', prio) |
| 135 | self.assertEquals(ret, '/tobesigned/%s,' % sprio) |
| 136 | |
| 137 | def testBadPriority(self): |
| 138 | """Verify we reject bad priority values""" |
| 139 | for prio in (-10, -1, 100, 91239): |
| 140 | self.assertRaises(ValueError, pushimage.MarkImageToBeSigned, self.ctx, |
| 141 | '', '', prio) |
| 142 | |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 143 | def testTbsUpload(self): |
| 144 | """Make sure we actually try to upload the file""" |
| 145 | pushimage.MarkImageToBeSigned(self.ctx, '', '', 50) |
| 146 | self.gs_mock.assertCommandContains(['cp', '--']) |
| 147 | |
| 148 | |
Mike Frysinger | 4495b03 | 2014-03-05 17:24:03 -0500 | [diff] [blame] | 149 | class PushImageTests(gs_unittest.AbstractGSContextTest): |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 150 | """Tests for PushImage()""" |
| 151 | |
| 152 | def setUp(self): |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 153 | self.mark_mock = self.PatchObject(pushimage, 'MarkImageToBeSigned') |
| 154 | |
| 155 | def testBasic(self): |
| 156 | """Simple smoke test""" |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 157 | EXPECTED = { |
| 158 | 'canary': [ |
Mike Frysinger | d6e2df0 | 2014-11-26 02:55:04 -0500 | [diff] [blame] | 159 | ('gs://chromeos-releases/canary-channel/test.board-hi/5126.0.0/' |
| 160 | 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')], |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 161 | 'dev': [ |
Mike Frysinger | d6e2df0 | 2014-11-26 02:55:04 -0500 | [diff] [blame] | 162 | ('gs://chromeos-releases/dev-channel/test.board-hi/5126.0.0/' |
| 163 | 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')], |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 164 | } |
Don Garrett | 9fd20a8 | 2014-09-04 11:37:22 -0700 | [diff] [blame] | 165 | with mock.patch.object(gs.GSContext, 'Exists', return_value=True): |
| 166 | urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0', |
| 167 | profile='hi') |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 168 | |
| 169 | self.assertEqual(urls, EXPECTED) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 170 | |
Amey Deshpande | 3c48795 | 2015-09-03 12:11:13 -0700 | [diff] [blame^] | 171 | def testBasic_SignTypesEmptyList(self): |
| 172 | """Tests PushImage behavior when |sign_types| is empty instead of None. |
| 173 | |
| 174 | As part of the buildbots, PushImage function always receives a tuple for |
| 175 | |sign_types| argument. This test checks the behavior for empty tuple. |
| 176 | """ |
| 177 | EXPECTED = { |
| 178 | 'canary': [ |
| 179 | ('gs://chromeos-releases/canary-channel/test.board-hi/5126.0.0/' |
| 180 | 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')], |
| 181 | 'dev': [ |
| 182 | ('gs://chromeos-releases/dev-channel/test.board-hi/5126.0.0/' |
| 183 | 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')], |
| 184 | } |
| 185 | with mock.patch.object(gs.GSContext, 'Exists', return_value=True): |
| 186 | urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0', |
| 187 | profile='hi', sign_types=()) |
| 188 | |
| 189 | self.assertEqual(urls, EXPECTED) |
| 190 | |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 191 | def testBasicMock(self): |
| 192 | """Simple smoke test in mock mode""" |
Don Garrett | 9fd20a8 | 2014-09-04 11:37:22 -0700 | [diff] [blame] | 193 | with mock.patch.object(gs.GSContext, 'Exists', return_value=True): |
| 194 | pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0', |
| 195 | dry_run=True, mock=True) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 196 | |
| 197 | def testBadVersion(self): |
| 198 | """Make sure we barf on bad version strings""" |
| 199 | self.assertRaises(ValueError, pushimage.PushImage, '', '', 'asdf') |
| 200 | |
| 201 | def testNoInsns(self): |
| 202 | """Boards w/out insn files should get skipped""" |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 203 | urls = pushimage.PushImage('/src', 'a bad bad board', 'R34-5126.0.0') |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 204 | self.assertEqual(self.gs_mock.call_count, 0) |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 205 | self.assertEqual(urls, None) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 206 | |
| 207 | def testSignTypesRecovery(self): |
| 208 | """Only sign the requested recovery type""" |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 209 | EXPECTED = { |
| 210 | 'canary': [ |
Mike Frysinger | d6e2df0 | 2014-11-26 02:55:04 -0500 | [diff] [blame] | 211 | ('gs://chromeos-releases/canary-channel/test.board/5126.0.0/' |
| 212 | 'ChromeOS-recovery-R34-5126.0.0-test.board.instructions')], |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 213 | 'dev': [ |
Mike Frysinger | d6e2df0 | 2014-11-26 02:55:04 -0500 | [diff] [blame] | 214 | ('gs://chromeos-releases/dev-channel/test.board/5126.0.0/' |
| 215 | 'ChromeOS-recovery-R34-5126.0.0-test.board.instructions')], |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 216 | } |
| 217 | |
Don Garrett | fd97b40 | 2015-09-03 00:59:21 +0000 | [diff] [blame] | 218 | urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0', |
| 219 | sign_types=['recovery']) |
Don Garrett | 861d47f | 2015-08-24 16:46:20 -0700 | [diff] [blame] | 220 | self.assertEqual(self.gs_mock.call_count, 20) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 221 | self.assertTrue(self.mark_mock.called) |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 222 | self.assertEqual(urls, EXPECTED) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 223 | |
| 224 | def testSignTypesNone(self): |
| 225 | """Verify nothing is signed when we request an unavailable type""" |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 226 | urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0', |
| 227 | sign_types=['nononononono']) |
Don Garrett | 861d47f | 2015-08-24 16:46:20 -0700 | [diff] [blame] | 228 | self.assertEqual(self.gs_mock.call_count, 18) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 229 | self.assertFalse(self.mark_mock.called) |
Don Garrett | 9459c2f | 2014-01-22 18:20:24 -0800 | [diff] [blame] | 230 | self.assertEqual(urls, {}) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 231 | |
Mike Frysinger | 4495b03 | 2014-03-05 17:24:03 -0500 | [diff] [blame] | 232 | def testGsError(self): |
| 233 | """Verify random GS errors don't make us blow up entirely""" |
| 234 | self.gs_mock.AddCmdResult(partial_mock.In('stat'), returncode=1, |
| 235 | output='gobblety gook\n') |
| 236 | with cros_test_lib.LoggingCapturer('chromite'): |
| 237 | self.assertRaises(pushimage.PushError, pushimage.PushImage, '/src', |
| 238 | 'test.board', 'R34-5126.0.0') |
| 239 | |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 240 | |
| 241 | class MainTests(cros_test_lib.MockTestCase): |
| 242 | """Tests for main()""" |
| 243 | |
| 244 | def setUp(self): |
| 245 | self.PatchObject(pushimage, 'PushImage') |
| 246 | |
| 247 | def testBasic(self): |
| 248 | """Simple smoke test""" |
Mike Frysinger | 09fe012 | 2014-02-09 02:44:05 -0500 | [diff] [blame] | 249 | pushimage.main(['--board', 'test.board', '/src', '--yes']) |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 250 | |
| 251 | |
Mike Frysinger | ea838d1 | 2014-12-08 11:55:32 -0500 | [diff] [blame] | 252 | def main(_argv): |
Mike Frysinger | d13faeb | 2013-09-05 16:00:46 -0400 | [diff] [blame] | 253 | # Use our local copy of insns for testing as the main one is not |
| 254 | # available in the public manifest. |
| 255 | signing.INPUT_INSN_DIR = signing.TEST_INPUT_INSN_DIR |
| 256 | |
| 257 | # Run the tests. |
Mike Frysinger | ba16737 | 2015-01-21 10:37:03 -0500 | [diff] [blame] | 258 | cros_test_lib.main(level='info', module=__name__) |