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