blob: ae6f6c861e45c20f6a554671ee6a364bfa292cfb [file] [log] [blame]
Mike Frysingerd13faeb2013-09-05 16:00:46 -04001# 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 Frysinger6791b1d2014-03-04 20:52:22 -05007from __future__ import print_function
8
Don Garrett9fd20a82014-09-04 11:37:22 -07009import mock
Mike Frysingerd13faeb2013-09-05 16:00:46 -040010import os
Mike Frysingerd13faeb2013-09-05 16:00:46 -040011
Mike Frysinger6791b1d2014-03-04 20:52:22 -050012from chromite.lib import cros_build_lib
Mike Frysingerd13faeb2013-09-05 16:00:46 -040013from chromite.lib import cros_test_lib
Don Garrett9fd20a82014-09-04 11:37:22 -070014from chromite.lib import gs
Mike Frysingerd13faeb2013-09-05 16:00:46 -040015from chromite.lib import gs_unittest
16from chromite.lib import osutils
Mike Frysinger4495b032014-03-05 17:24:03 -050017from chromite.lib import partial_mock
Mike Frysingerd13faeb2013-09-05 16:00:46 -040018from chromite.lib import signing
19from chromite.scripts import pushimage
20
21
22class InputInsnsTest(cros_test_lib.MockTestCase):
23 """Tests for InputInsns"""
24
Don Garrettac529772015-04-29 15:09:26 -070025 def setUp(self):
26 self.StartPatcher(gs_unittest.GSContextMock())
27
Mike Frysingerd13faeb2013-09-05 16:00:46 -040028 def testBasic(self):
29 """Simple smoke test"""
Don Garrettac529772015-04-29 15:09:26 -070030 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 Frysingerd13faeb2013-09-05 16:00:46 -040034
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]
65keyset = stumpy-mp-v3
66channel = dev canary
67chromeos_shell = false
68ensure_no_password = true
69firmware_update = true
70security_checks = true
71create_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]
86keyset = batman
87channel = dev
88chromeos_shell = false
89ensure_no_password = true
90firmware_update = true
91security_checks = true
92create_nplusone = true
93
94[general]
95board = board
96config_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 Frysinger4495b032014-03-05 17:24:03 -0500115class MarkImageToBeSignedTest(gs_unittest.AbstractGSContextTest):
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400116 """Tests for MarkImageToBeSigned()"""
117
118 def setUp(self):
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400119 # 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 Frysinger6430d132014-10-27 23:43:30 -0400121 self.PatchObject(cros_build_lib, 'MachineDetails', return_value='1234\n')
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400122
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 Frysingerd13faeb2013-09-05 16:00:46 -0400143 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 Frysinger4495b032014-03-05 17:24:03 -0500149class PushImageTests(gs_unittest.AbstractGSContextTest):
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400150 """Tests for PushImage()"""
151
152 def setUp(self):
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400153 self.mark_mock = self.PatchObject(pushimage, 'MarkImageToBeSigned')
154
155 def testBasic(self):
156 """Simple smoke test"""
Don Garrett9459c2f2014-01-22 18:20:24 -0800157 EXPECTED = {
158 'canary': [
Mike Frysingerd6e2df02014-11-26 02:55:04 -0500159 ('gs://chromeos-releases/canary-channel/test.board-hi/5126.0.0/'
160 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')],
Don Garrett9459c2f2014-01-22 18:20:24 -0800161 'dev': [
Mike Frysingerd6e2df02014-11-26 02:55:04 -0500162 ('gs://chromeos-releases/dev-channel/test.board-hi/5126.0.0/'
163 'ChromeOS-recovery-R34-5126.0.0-test.board-hi.instructions')],
Don Garrett9459c2f2014-01-22 18:20:24 -0800164 }
Don Garrett9fd20a82014-09-04 11:37:22 -0700165 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 Garrett9459c2f2014-01-22 18:20:24 -0800168
169 self.assertEqual(urls, EXPECTED)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400170
Amey Deshpande3c487952015-09-03 12:11:13 -0700171 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
Amey Deshpandea936c622015-08-12 17:27:54 -0700191 def testBasic_RealBoardName(self):
192 """Runs a simple smoke test using a real board name."""
193 EXPECTED = {
194 'canary': [
195 ('gs://chromeos-releases/canary-channel/x86-alex/5126.0.0/'
196 'ChromeOS-recovery-R34-5126.0.0-x86-alex.instructions')],
197 'dev': [
198 ('gs://chromeos-releases/dev-channel/x86-alex/5126.0.0/'
199 'ChromeOS-recovery-R34-5126.0.0-x86-alex.instructions')],
200 }
201 with mock.patch.object(gs.GSContext, 'Exists', return_value=True):
202 urls = pushimage.PushImage('/src', 'x86-alex', 'R34-5126.0.0')
203
204 self.assertEqual(urls, EXPECTED)
205
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400206 def testBasicMock(self):
207 """Simple smoke test in mock mode"""
Don Garrett9fd20a82014-09-04 11:37:22 -0700208 with mock.patch.object(gs.GSContext, 'Exists', return_value=True):
209 pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0',
210 dry_run=True, mock=True)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400211
212 def testBadVersion(self):
213 """Make sure we barf on bad version strings"""
214 self.assertRaises(ValueError, pushimage.PushImage, '', '', 'asdf')
215
216 def testNoInsns(self):
217 """Boards w/out insn files should get skipped"""
Don Garrett9459c2f2014-01-22 18:20:24 -0800218 urls = pushimage.PushImage('/src', 'a bad bad board', 'R34-5126.0.0')
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400219 self.assertEqual(self.gs_mock.call_count, 0)
Don Garrett9459c2f2014-01-22 18:20:24 -0800220 self.assertEqual(urls, None)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400221
222 def testSignTypesRecovery(self):
223 """Only sign the requested recovery type"""
Don Garrett9459c2f2014-01-22 18:20:24 -0800224 EXPECTED = {
225 'canary': [
Mike Frysingerd6e2df02014-11-26 02:55:04 -0500226 ('gs://chromeos-releases/canary-channel/test.board/5126.0.0/'
227 'ChromeOS-recovery-R34-5126.0.0-test.board.instructions')],
Don Garrett9459c2f2014-01-22 18:20:24 -0800228 'dev': [
Mike Frysingerd6e2df02014-11-26 02:55:04 -0500229 ('gs://chromeos-releases/dev-channel/test.board/5126.0.0/'
230 'ChromeOS-recovery-R34-5126.0.0-test.board.instructions')],
Don Garrett9459c2f2014-01-22 18:20:24 -0800231 }
232
Amey Deshpandea936c622015-08-12 17:27:54 -0700233 with mock.patch.object(gs.GSContext, 'Exists', return_value=True):
234 urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0',
235 sign_types=['recovery'])
David Rileya04d19d2015-09-04 16:11:50 -0700236 self.assertEqual(self.gs_mock.call_count, 22)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400237 self.assertTrue(self.mark_mock.called)
Don Garrett9459c2f2014-01-22 18:20:24 -0800238 self.assertEqual(urls, EXPECTED)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400239
Amey Deshpandea936c622015-08-12 17:27:54 -0700240 def testSignTypesBase(self):
241 """Only sign the requested recovery type"""
242 EXPECTED = {
243 'canary': [
244 ('gs://chromeos-releases/canary-channel/test.board/5126.0.0/'
245 'ChromeOS-base-R34-5126.0.0-test.board.instructions')],
246 'dev': [
247 ('gs://chromeos-releases/dev-channel/test.board/5126.0.0/'
248 'ChromeOS-base-R34-5126.0.0-test.board.instructions')],
249 }
250
251 with mock.patch.object(gs.GSContext, 'Exists', return_value=True):
252 urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0',
253 sign_types=['base'])
David Rileya04d19d2015-09-04 16:11:50 -0700254 self.assertEqual(self.gs_mock.call_count, 24)
Amey Deshpandea936c622015-08-12 17:27:54 -0700255 self.assertTrue(self.mark_mock.called)
256 self.assertEqual(urls, EXPECTED)
257
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400258 def testSignTypesNone(self):
259 """Verify nothing is signed when we request an unavailable type"""
Don Garrett9459c2f2014-01-22 18:20:24 -0800260 urls = pushimage.PushImage('/src', 'test.board', 'R34-5126.0.0',
261 sign_types=['nononononono'])
David Rileya04d19d2015-09-04 16:11:50 -0700262 self.assertEqual(self.gs_mock.call_count, 20)
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400263 self.assertFalse(self.mark_mock.called)
Don Garrett9459c2f2014-01-22 18:20:24 -0800264 self.assertEqual(urls, {})
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400265
Mike Frysinger4495b032014-03-05 17:24:03 -0500266 def testGsError(self):
267 """Verify random GS errors don't make us blow up entirely"""
268 self.gs_mock.AddCmdResult(partial_mock.In('stat'), returncode=1,
269 output='gobblety gook\n')
270 with cros_test_lib.LoggingCapturer('chromite'):
271 self.assertRaises(pushimage.PushError, pushimage.PushImage, '/src',
272 'test.board', 'R34-5126.0.0')
273
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400274
275class MainTests(cros_test_lib.MockTestCase):
276 """Tests for main()"""
277
278 def setUp(self):
279 self.PatchObject(pushimage, 'PushImage')
280
281 def testBasic(self):
282 """Simple smoke test"""
Mike Frysinger09fe0122014-02-09 02:44:05 -0500283 pushimage.main(['--board', 'test.board', '/src', '--yes'])
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400284
285
Mike Frysingerea838d12014-12-08 11:55:32 -0500286def main(_argv):
Mike Frysingerd13faeb2013-09-05 16:00:46 -0400287 # Use our local copy of insns for testing as the main one is not
288 # available in the public manifest.
289 signing.INPUT_INSN_DIR = signing.TEST_INPUT_INSN_DIR
290
291 # Run the tests.
Mike Frysingerba167372015-01-21 10:37:03 -0500292 cros_test_lib.main(level='info', module=__name__)