Mike Frysinger | e58c0e2 | 2017-10-04 15:43:30 -0400 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 2 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
| 5 | |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 6 | """Unit tests for the deploy_chrome script.""" |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 7 | |
Mike Frysinger | 383367e | 2014-09-16 15:06:17 -0400 | [diff] [blame] | 8 | from __future__ import print_function |
| 9 | |
Anushruth | 8d79767 | 2019-10-17 12:22:31 -0700 | [diff] [blame] | 10 | import errno |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 11 | import os |
Mike Frysinger | 6165cdc | 2020-02-21 02:38:07 -0500 | [diff] [blame] | 12 | import sys |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 13 | import time |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 14 | |
Mike Frysinger | 6db648e | 2018-07-24 19:57:58 -0400 | [diff] [blame] | 15 | import mock |
| 16 | |
David Pursell | cfd5887 | 2015-03-19 09:15:48 -0700 | [diff] [blame] | 17 | from chromite.cli.cros import cros_chrome_sdk_unittest |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 18 | from chromite.lib import chrome_util |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 19 | from chromite.lib import cros_build_lib |
| 20 | from chromite.lib import cros_test_lib |
Ryan Cui | 686ec05 | 2013-02-12 16:39:41 -0800 | [diff] [blame] | 21 | from chromite.lib import osutils |
Erik Chen | 75a2f49 | 2020-08-06 19:15:11 -0700 | [diff] [blame] | 22 | from chromite.lib import parallel_unittest |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 23 | from chromite.lib import partial_mock |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 24 | from chromite.lib import remote_access |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 25 | from chromite.lib import remote_access_unittest |
| 26 | from chromite.scripts import deploy_chrome |
| 27 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 28 | |
Mike Frysinger | 6165cdc | 2020-02-21 02:38:07 -0500 | [diff] [blame] | 29 | assert sys.version_info >= (3, 6), 'This module requires Python 3.6+' |
| 30 | |
| 31 | |
Mike Frysinger | 27e21b7 | 2018-07-12 14:20:21 -0400 | [diff] [blame] | 32 | # pylint: disable=protected-access |
| 33 | |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 34 | |
| 35 | _REGULAR_TO = ('--to', 'monkey') |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 36 | _TARGET_BOARD = 'eve' |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 37 | _GS_PATH = 'gs://foon' |
| 38 | |
| 39 | |
| 40 | def _ParseCommandLine(argv): |
| 41 | return deploy_chrome._ParseCommandLine(['--log-level', 'debug'] + argv) |
| 42 | |
| 43 | |
| 44 | class InterfaceTest(cros_test_lib.OutputTestCase): |
| 45 | """Tests the commandline interface of the script.""" |
| 46 | |
| 47 | def testGsLocalPathUnSpecified(self): |
| 48 | """Test no chrome path specified.""" |
| 49 | with self.OutputCapturer(): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 50 | self.assertRaises2(SystemExit, _ParseCommandLine, |
| 51 | list(_REGULAR_TO) + ['--board', _TARGET_BOARD], |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 52 | check_attrs={'code': 2}) |
| 53 | |
| 54 | def testGsPathSpecified(self): |
| 55 | """Test case of GS path specified.""" |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 56 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH] |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 57 | _ParseCommandLine(argv) |
| 58 | |
| 59 | def testLocalPathSpecified(self): |
| 60 | """Test case of local path specified.""" |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 61 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--local-pkg-path', |
| 62 | '/path/to/chrome'] |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 63 | _ParseCommandLine(argv) |
| 64 | |
| 65 | def testNoTarget(self): |
| 66 | """Test no target specified.""" |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 67 | argv = ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH] |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 68 | self.assertParseError(argv) |
| 69 | |
Ben Pastene | 3bd3e5e | 2020-08-10 14:40:43 -0700 | [diff] [blame^] | 70 | def testToAndDevice(self): |
| 71 | """Test no target specified.""" |
| 72 | argv = ['--board', _TARGET_BOARD, '--build-dir', '/path/to/nowhere', |
| 73 | '--to', 'localhost', '--device', 'localhost'] |
| 74 | self.assertParseError(argv) |
| 75 | |
Erik Chen | 75a2f49 | 2020-08-06 19:15:11 -0700 | [diff] [blame] | 76 | def testLacros(self): |
| 77 | """Test basic lacros invocation.""" |
| 78 | argv = ['--lacros', '--nostrip', '--build-dir', '/path/to/nowhere', |
| 79 | '--to', 'localhost'] |
| 80 | options = _ParseCommandLine(argv) |
| 81 | self.assertTrue(options.lacros) |
| 82 | self.assertEqual(options.target_dir, deploy_chrome.LACROS_DIR) |
| 83 | |
| 84 | def testLacrosRequiresNostrip(self): |
| 85 | """Lacros requires --nostrip""" |
| 86 | argv = ['--lacros', '--build-dir', '/path/to/nowhere', '--to', 'localhost'] |
| 87 | self.assertRaises2(SystemExit, _ParseCommandLine, argv, |
| 88 | check_attrs={'code': 2}) |
| 89 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 90 | def assertParseError(self, argv): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 91 | with self.OutputCapturer(): |
| 92 | self.assertRaises2(SystemExit, _ParseCommandLine, argv, |
| 93 | check_attrs={'code': 2}) |
| 94 | |
Thiago Goncales | 1279331 | 2013-05-23 11:26:17 -0700 | [diff] [blame] | 95 | def testMountOptionSetsTargetDir(self): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 96 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH, |
| 97 | '--mount'] |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 98 | options = _ParseCommandLine(argv) |
Thiago Goncales | 1279331 | 2013-05-23 11:26:17 -0700 | [diff] [blame] | 99 | self.assertIsNot(options.target_dir, None) |
| 100 | |
| 101 | def testMountOptionSetsMountDir(self): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 102 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH, |
| 103 | '--mount'] |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 104 | options = _ParseCommandLine(argv) |
Thiago Goncales | 1279331 | 2013-05-23 11:26:17 -0700 | [diff] [blame] | 105 | self.assertIsNot(options.mount_dir, None) |
| 106 | |
| 107 | def testMountOptionDoesNotOverrideTargetDir(self): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 108 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH, |
| 109 | '--mount', '--target-dir', '/foo/bar/cow'] |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 110 | options = _ParseCommandLine(argv) |
Thiago Goncales | 1279331 | 2013-05-23 11:26:17 -0700 | [diff] [blame] | 111 | self.assertEqual(options.target_dir, '/foo/bar/cow') |
| 112 | |
| 113 | def testMountOptionDoesNotOverrideMountDir(self): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 114 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, '--gs-path', _GS_PATH, |
| 115 | '--mount', '--mount-dir', '/foo/bar/cow'] |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 116 | options = _ParseCommandLine(argv) |
Thiago Goncales | 1279331 | 2013-05-23 11:26:17 -0700 | [diff] [blame] | 117 | self.assertEqual(options.mount_dir, '/foo/bar/cow') |
| 118 | |
Adrian Elder | a2c548a | 2017-11-07 19:01:29 -0500 | [diff] [blame] | 119 | def testSshIdentityOptionSetsOption(self): |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 120 | argv = list(_REGULAR_TO) + ['--board', _TARGET_BOARD, |
| 121 | '--private-key', '/foo/bar/key', |
Bernie Thompson | 93b9ee6 | 2018-02-21 14:56:16 -0800 | [diff] [blame] | 122 | '--build-dir', '/path/to/nowhere'] |
Adrian Elder | a2c548a | 2017-11-07 19:01:29 -0500 | [diff] [blame] | 123 | options = _ParseCommandLine(argv) |
| 124 | self.assertEqual(options.private_key, '/foo/bar/key') |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 125 | |
| 126 | class DeployChromeMock(partial_mock.PartialMock): |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 127 | """Deploy Chrome Mock Class.""" |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 128 | |
| 129 | TARGET = 'chromite.scripts.deploy_chrome.DeployChrome' |
Erik Chen | 75a2f49 | 2020-08-06 19:15:11 -0700 | [diff] [blame] | 130 | ATTRS = ('_KillAshChromeIfNeeded', '_DisableRootfsVerification') |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 131 | |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 132 | def __init__(self): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 133 | partial_mock.PartialMock.__init__(self) |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 134 | self.remote_device_mock = remote_access_unittest.RemoteDeviceMock() |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 135 | # Target starts off as having rootfs verification enabled. |
Ryan Cui | e18f24f | 2012-12-03 18:39:55 -0800 | [diff] [blame] | 136 | self.rsh_mock = remote_access_unittest.RemoteShMock() |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 137 | self.rsh_mock.SetDefaultCmdResult(0) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 138 | self.MockMountCmd(1) |
David Haddock | 3151d91 | 2017-10-24 03:50:32 +0000 | [diff] [blame] | 139 | self.rsh_mock.AddCmdResult( |
Anushruth | 8d79767 | 2019-10-17 12:22:31 -0700 | [diff] [blame] | 140 | deploy_chrome.LSOF_COMMAND_CHROME % (deploy_chrome._CHROME_DIR,), 1) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 141 | |
| 142 | def MockMountCmd(self, returnvalue): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 143 | self.rsh_mock.AddCmdResult(deploy_chrome.MOUNT_RW_COMMAND, |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 144 | returnvalue) |
| 145 | |
| 146 | def _DisableRootfsVerification(self, inst): |
| 147 | with mock.patch.object(time, 'sleep'): |
| 148 | self.backup['_DisableRootfsVerification'](inst) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 149 | |
Ryan Cui | 4d6fca9 | 2012-12-13 16:41:56 -0800 | [diff] [blame] | 150 | def PreStart(self): |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 151 | self.remote_device_mock.start() |
Ryan Cui | 4d6fca9 | 2012-12-13 16:41:56 -0800 | [diff] [blame] | 152 | self.rsh_mock.start() |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 153 | |
Ryan Cui | 4d6fca9 | 2012-12-13 16:41:56 -0800 | [diff] [blame] | 154 | def PreStop(self): |
| 155 | self.rsh_mock.stop() |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 156 | self.remote_device_mock.stop() |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 157 | |
Erik Chen | 75a2f49 | 2020-08-06 19:15:11 -0700 | [diff] [blame] | 158 | def _KillAshChromeIfNeeded(self, _inst): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 159 | # Fully stub out for now. |
| 160 | pass |
| 161 | |
| 162 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 163 | class DeployTest(cros_test_lib.MockTempDirTestCase): |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 164 | """Setup a deploy object with a GS-path for use in tests.""" |
| 165 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 166 | def _GetDeployChrome(self, args): |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 167 | options = _ParseCommandLine(args) |
Ryan Cui | a56a71e | 2012-10-18 18:40:35 -0700 | [diff] [blame] | 168 | return deploy_chrome.DeployChrome( |
| 169 | options, self.tempdir, os.path.join(self.tempdir, 'staging')) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 170 | |
| 171 | def setUp(self): |
Ryan Cui | f1416f3 | 2013-01-22 18:43:41 -0800 | [diff] [blame] | 172 | self.deploy_mock = self.StartPatcher(DeployChromeMock()) |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 173 | self.deploy = self._GetDeployChrome(list(_REGULAR_TO) + |
| 174 | ['--board', _TARGET_BOARD, '--gs-path', |
| 175 | _GS_PATH, '--force', '--mount']) |
Luigi Semenzato | 1bc79b2 | 2016-11-22 16:32:17 -0800 | [diff] [blame] | 176 | self.remote_reboot_mock = \ |
| 177 | self.PatchObject(remote_access.RemoteAccess, 'RemoteReboot', |
| 178 | return_value=True) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 179 | |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 180 | |
| 181 | class TestCheckIfBoardMatches(DeployTest): |
| 182 | """Testing checking whether the DUT board matches the target board.""" |
| 183 | |
| 184 | def testMatchedBoard(self): |
| 185 | """Test the case where the DUT board matches the target board.""" |
| 186 | self.PatchObject(remote_access.ChromiumOSDevice, 'board', _TARGET_BOARD) |
| 187 | self.assertTrue(self.deploy.options.force) |
| 188 | self.deploy._CheckBoard() |
| 189 | self.deploy.options.force = False |
| 190 | self.deploy._CheckBoard() |
| 191 | |
| 192 | def testMismatchedBoard(self): |
| 193 | """Test the case where the DUT board does not match the target board.""" |
| 194 | self.PatchObject(remote_access.ChromiumOSDevice, 'board', 'cedar') |
| 195 | self.assertTrue(self.deploy.options.force) |
| 196 | self.deploy._CheckBoard() |
| 197 | self.deploy.options.force = False |
| 198 | self.PatchObject(cros_build_lib, 'BooleanPrompt', return_value=True) |
| 199 | self.deploy._CheckBoard() |
| 200 | self.PatchObject(cros_build_lib, 'BooleanPrompt', return_value=False) |
| 201 | self.assertRaises(deploy_chrome.DeployFailure, self.deploy._CheckBoard) |
| 202 | |
| 203 | |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 204 | class TestDisableRootfsVerification(DeployTest): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 205 | """Testing disabling of rootfs verification and RO mode.""" |
| 206 | |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 207 | def testDisableRootfsVerificationSuccess(self): |
| 208 | """Test the working case, disabling rootfs verification.""" |
| 209 | self.deploy_mock.MockMountCmd(0) |
| 210 | self.deploy._DisableRootfsVerification() |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 211 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 212 | |
| 213 | def testDisableRootfsVerificationFailure(self): |
| 214 | """Test failure to disable rootfs verification.""" |
Mike Frysinger | 27e21b7 | 2018-07-12 14:20:21 -0400 | [diff] [blame] | 215 | # pylint: disable=unused-argument |
Shuqian Zhao | 14e6109 | 2017-11-17 00:02:16 +0000 | [diff] [blame] | 216 | def RaiseRunCommandError(timeout_sec=None): |
Mike Frysinger | 929f3ba | 2019-09-12 03:24:59 -0400 | [diff] [blame] | 217 | raise cros_build_lib.RunCommandError('Mock RunCommandError') |
Luigi Semenzato | 1bc79b2 | 2016-11-22 16:32:17 -0800 | [diff] [blame] | 218 | self.remote_reboot_mock.side_effect = RaiseRunCommandError |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 219 | self.assertRaises(cros_build_lib.RunCommandError, |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 220 | self.deploy._DisableRootfsVerification) |
Luigi Semenzato | 1bc79b2 | 2016-11-22 16:32:17 -0800 | [diff] [blame] | 221 | self.remote_reboot_mock.side_effect = None |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 222 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 223 | |
| 224 | |
| 225 | class TestMount(DeployTest): |
| 226 | """Testing mount success and failure.""" |
| 227 | |
| 228 | def testSuccess(self): |
| 229 | """Test case where we are able to mount as writable.""" |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 230 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 231 | self.deploy_mock.MockMountCmd(0) |
| 232 | self.deploy._MountRootfsAsWritable() |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 233 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 234 | |
| 235 | def testMountError(self): |
| 236 | """Test that mount failure doesn't raise an exception by default.""" |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 237 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 238 | self.PatchObject(remote_access.ChromiumOSDevice, 'IsDirWritable', |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 239 | return_value=False, autospec=True) |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 240 | self.deploy._MountRootfsAsWritable() |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 241 | self.assertTrue(self.deploy._root_dir_is_still_readonly.is_set()) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 242 | |
| 243 | def testMountRwFailure(self): |
Mike Frysinger | f5a3b2d | 2019-12-12 14:36:17 -0500 | [diff] [blame] | 244 | """Test that mount failure raises an exception if check=True.""" |
David James | 88e6f03 | 2013-03-02 08:13:20 -0800 | [diff] [blame] | 245 | self.assertRaises(cros_build_lib.RunCommandError, |
Mike Frysinger | f5a3b2d | 2019-12-12 14:36:17 -0500 | [diff] [blame] | 246 | self.deploy._MountRootfsAsWritable, check=True) |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 247 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 248 | |
| 249 | def testMountTempDir(self): |
| 250 | """Test that mount succeeds if target dir is writable.""" |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 251 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 252 | self.PatchObject(remote_access.ChromiumOSDevice, 'IsDirWritable', |
Robert Flack | 1dc7ea8 | 2014-11-26 13:50:24 -0500 | [diff] [blame] | 253 | return_value=True, autospec=True) |
| 254 | self.deploy._MountRootfsAsWritable() |
Steven Bennetts | ca73efa | 2018-07-10 13:36:56 -0700 | [diff] [blame] | 255 | self.assertFalse(self.deploy._root_dir_is_still_readonly.is_set()) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 256 | |
| 257 | |
Anushruth | 8d79767 | 2019-10-17 12:22:31 -0700 | [diff] [blame] | 258 | class TestMountTarget(DeployTest): |
Mike Frysinger | df1d0b0 | 2019-11-12 17:44:12 -0500 | [diff] [blame] | 259 | """Testing mount and umount command handling.""" |
Anushruth | 8d79767 | 2019-10-17 12:22:31 -0700 | [diff] [blame] | 260 | |
| 261 | def testMountTargetUmountFailure(self): |
| 262 | """Test error being thrown if umount fails. |
| 263 | |
| 264 | Test that 'lsof' is run on mount-dir and 'mount -rbind' command is not run |
| 265 | if 'umount' cmd fails. |
| 266 | """ |
| 267 | mount_dir = self.deploy.options.mount_dir |
| 268 | target_dir = self.deploy.options.target_dir |
| 269 | self.deploy_mock.rsh_mock.AddCmdResult( |
| 270 | deploy_chrome._UMOUNT_DIR_IF_MOUNTPOINT_CMD % |
| 271 | {'dir': mount_dir}, returncode=errno.EBUSY, stderr='Target is Busy') |
| 272 | self.deploy_mock.rsh_mock.AddCmdResult(deploy_chrome.LSOF_COMMAND % |
| 273 | (mount_dir,), returncode=0, |
| 274 | stdout='process ' + mount_dir) |
| 275 | # Check for RunCommandError being thrown. |
| 276 | self.assertRaises(cros_build_lib.RunCommandError, |
| 277 | self.deploy._MountTarget) |
| 278 | # Check for the 'mount -rbind' command not run. |
| 279 | self.deploy_mock.rsh_mock.assertCommandContains( |
| 280 | (deploy_chrome._BIND_TO_FINAL_DIR_CMD % (target_dir, mount_dir)), |
| 281 | expected=False) |
| 282 | # Check for lsof command being called. |
| 283 | self.deploy_mock.rsh_mock.assertCommandContains( |
| 284 | (deploy_chrome.LSOF_COMMAND % (mount_dir,))) |
| 285 | |
| 286 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 287 | class TestUiJobStarted(DeployTest): |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 288 | """Test detection of a running 'ui' job.""" |
| 289 | |
Ryan Cui | f2d1a58 | 2013-02-19 14:08:13 -0800 | [diff] [blame] | 290 | def MockStatusUiCmd(self, **kwargs): |
David Haddock | 3151d91 | 2017-10-24 03:50:32 +0000 | [diff] [blame] | 291 | self.deploy_mock.rsh_mock.AddCmdResult('status ui', **kwargs) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 292 | |
| 293 | def testUiJobStartedFalse(self): |
| 294 | """Correct results with a stopped job.""" |
Ryan Cui | f2d1a58 | 2013-02-19 14:08:13 -0800 | [diff] [blame] | 295 | self.MockStatusUiCmd(output='ui stop/waiting') |
| 296 | self.assertFalse(self.deploy._CheckUiJobStarted()) |
| 297 | |
| 298 | def testNoUiJob(self): |
| 299 | """Correct results when the job doesn't exist.""" |
| 300 | self.MockStatusUiCmd(error='start: Unknown job: ui', returncode=1) |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 301 | self.assertFalse(self.deploy._CheckUiJobStarted()) |
| 302 | |
| 303 | def testCheckRootfsWriteableTrue(self): |
| 304 | """Correct results with a running job.""" |
Ryan Cui | f2d1a58 | 2013-02-19 14:08:13 -0800 | [diff] [blame] | 305 | self.MockStatusUiCmd(output='ui start/running, process 297') |
Ryan Cui | afd6c5c | 2012-07-30 17:48:22 -0700 | [diff] [blame] | 306 | self.assertTrue(self.deploy._CheckUiJobStarted()) |
| 307 | |
| 308 | |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 309 | class StagingTest(cros_test_lib.MockTempDirTestCase): |
| 310 | """Test user-mode and ebuild-mode staging functionality.""" |
| 311 | |
| 312 | def setUp(self): |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 313 | self.staging_dir = os.path.join(self.tempdir, 'staging') |
| 314 | self.build_dir = os.path.join(self.tempdir, 'build_dir') |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 315 | self.common_flags = ['--board', _TARGET_BOARD, |
| 316 | '--build-dir', self.build_dir, '--staging-only', |
| 317 | '--cache-dir', self.tempdir] |
Ryan Cui | a0215a7 | 2013-02-14 16:20:45 -0800 | [diff] [blame] | 318 | self.sdk_mock = self.StartPatcher(cros_chrome_sdk_unittest.SDKFetcherMock()) |
Ryan Cui | 686ec05 | 2013-02-12 16:39:41 -0800 | [diff] [blame] | 319 | self.PatchObject( |
| 320 | osutils, 'SourceEnvironment', autospec=True, |
| 321 | return_value={'STRIP': 'x86_64-cros-linux-gnu-strip'}) |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 322 | |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 323 | def testSingleFileDeployFailure(self): |
| 324 | """Default staging enforces that mandatory files are copied""" |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 325 | options = _ParseCommandLine(self.common_flags) |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 326 | osutils.Touch(os.path.join(self.build_dir, 'chrome'), makedirs=True) |
| 327 | self.assertRaises( |
| 328 | chrome_util.MissingPathError, deploy_chrome._PrepareStagingDir, |
Daniel Erat | c89829c | 2014-05-12 17:24:21 -0700 | [diff] [blame] | 329 | options, self.tempdir, self.staging_dir, chrome_util._COPY_PATHS_CHROME) |
Ryan Cui | ef91e70 | 2013-02-04 12:06:36 -0800 | [diff] [blame] | 330 | |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 331 | def testSloppyDeployFailure(self): |
| 332 | """Sloppy staging enforces that at least one file is copied.""" |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 333 | options = _ParseCommandLine(self.common_flags + ['--sloppy']) |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 334 | self.assertRaises( |
| 335 | chrome_util.MissingPathError, deploy_chrome._PrepareStagingDir, |
Daniel Erat | c89829c | 2014-05-12 17:24:21 -0700 | [diff] [blame] | 336 | options, self.tempdir, self.staging_dir, chrome_util._COPY_PATHS_CHROME) |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 337 | |
| 338 | def testSloppyDeploySuccess(self): |
| 339 | """Sloppy staging - stage one file.""" |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 340 | options = _ParseCommandLine(self.common_flags + ['--sloppy']) |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 341 | osutils.Touch(os.path.join(self.build_dir, 'chrome'), makedirs=True) |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 342 | deploy_chrome._PrepareStagingDir(options, self.tempdir, self.staging_dir, |
Daniel Erat | c89829c | 2014-05-12 17:24:21 -0700 | [diff] [blame] | 343 | chrome_util._COPY_PATHS_CHROME) |
David James | a6e0889 | 2013-03-01 13:34:11 -0800 | [diff] [blame] | 344 | |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 345 | |
| 346 | class DeployTestBuildDir(cros_test_lib.MockTempDirTestCase): |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 347 | """Set up a deploy object with a build-dir for use in deployment type tests""" |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 348 | |
| 349 | def _GetDeployChrome(self, args): |
Mike Frysinger | c3061a6 | 2015-06-04 04:16:18 -0400 | [diff] [blame] | 350 | options = _ParseCommandLine(args) |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 351 | return deploy_chrome.DeployChrome( |
| 352 | options, self.tempdir, os.path.join(self.tempdir, 'staging')) |
| 353 | |
| 354 | def setUp(self): |
| 355 | self.staging_dir = os.path.join(self.tempdir, 'staging') |
| 356 | self.build_dir = os.path.join(self.tempdir, 'build_dir') |
| 357 | self.deploy_mock = self.StartPatcher(DeployChromeMock()) |
| 358 | self.deploy = self._GetDeployChrome( |
Avery Musbach | 3edff0e | 2020-03-27 13:35:53 -0700 | [diff] [blame] | 359 | list(_REGULAR_TO) + ['--board', _TARGET_BOARD, |
| 360 | '--build-dir', self.build_dir, '--staging-only', |
| 361 | '--cache-dir', self.tempdir, '--sloppy']) |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 362 | |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 363 | def getCopyPath(self, source_path): |
| 364 | """Return a chrome_util.Path or None if not present.""" |
| 365 | paths = [p for p in self.deploy.copy_paths if p.src == source_path] |
| 366 | return paths[0] if paths else None |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 367 | |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 368 | class TestDeploymentType(DeployTestBuildDir): |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 369 | """Test detection of deployment type using build dir.""" |
| 370 | |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 371 | def testAppShellDetection(self): |
| 372 | """Check for an app_shell deployment""" |
| 373 | osutils.Touch(os.path.join(self.deploy.options.build_dir, 'app_shell'), |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 374 | makedirs=True) |
| 375 | self.deploy._CheckDeployType() |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 376 | self.assertTrue(self.getCopyPath('app_shell')) |
| 377 | self.assertFalse(self.getCopyPath('chrome')) |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 378 | |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 379 | def testChromeAndAppShellDetection(self): |
Daniel Erat | f53bd3a | 2016-12-02 11:28:36 -0700 | [diff] [blame] | 380 | """Check for a chrome deployment when app_shell also exists.""" |
Steve Fung | 63d705d | 2014-03-16 03:14:03 -0700 | [diff] [blame] | 381 | osutils.Touch(os.path.join(self.deploy.options.build_dir, 'chrome'), |
| 382 | makedirs=True) |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 383 | osutils.Touch(os.path.join(self.deploy.options.build_dir, 'app_shell'), |
Steve Fung | 63d705d | 2014-03-16 03:14:03 -0700 | [diff] [blame] | 384 | makedirs=True) |
| 385 | self.deploy._CheckDeployType() |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 386 | self.assertTrue(self.getCopyPath('chrome')) |
Daniel Erat | 9813f0e | 2014-11-12 11:00:28 -0700 | [diff] [blame] | 387 | self.assertFalse(self.getCopyPath('app_shell')) |
Steve Fung | 63d705d | 2014-03-16 03:14:03 -0700 | [diff] [blame] | 388 | |
Steve Fung | e984a53 | 2013-11-25 17:09:25 -0800 | [diff] [blame] | 389 | def testChromeDetection(self): |
| 390 | """Check for a regular chrome deployment""" |
| 391 | osutils.Touch(os.path.join(self.deploy.options.build_dir, 'chrome'), |
| 392 | makedirs=True) |
| 393 | self.deploy._CheckDeployType() |
Daniel Erat | 1ae4638 | 2014-08-14 10:23:39 -0700 | [diff] [blame] | 394 | self.assertTrue(self.getCopyPath('chrome')) |
Daniel Erat | 9813f0e | 2014-11-12 11:00:28 -0700 | [diff] [blame] | 395 | self.assertFalse(self.getCopyPath('app_shell')) |
Ben Pastene | e484b34 | 2020-06-30 18:29:27 -0700 | [diff] [blame] | 396 | |
| 397 | |
| 398 | class TestDeployTestBinaries(cros_test_lib.RunCommandTempDirTestCase): |
| 399 | """Tests _DeployTestBinaries().""" |
| 400 | |
| 401 | def setUp(self): |
| 402 | options = _ParseCommandLine(list(_REGULAR_TO) + [ |
| 403 | '--board', _TARGET_BOARD, '--force', '--mount', |
| 404 | '--build-dir', os.path.join(self.tempdir, 'build_dir'), |
| 405 | '--nostrip']) |
| 406 | self.deploy = deploy_chrome.DeployChrome( |
| 407 | options, self.tempdir, os.path.join(self.tempdir, 'staging')) |
| 408 | |
| 409 | def testFindError(self): |
| 410 | """Ensure an error is thrown if we can't inspect the device.""" |
| 411 | self.rc.AddCmdResult( |
| 412 | partial_mock.In(deploy_chrome._FIND_TEST_BIN_CMD), 1) |
| 413 | self.assertRaises( |
| 414 | deploy_chrome.DeployFailure, self.deploy._DeployTestBinaries) |
| 415 | |
| 416 | def testSuccess(self): |
| 417 | """Ensure the staging dir contains the right binaries to copy over.""" |
| 418 | test_binaries = [ |
| 419 | 'run_a_tests', |
| 420 | 'run_b_tests', |
| 421 | 'run_c_tests', |
| 422 | ] |
| 423 | # Simulate having the binaries both on the device and in our local build |
| 424 | # dir. |
| 425 | self.rc.AddCmdResult( |
| 426 | partial_mock.In(deploy_chrome._FIND_TEST_BIN_CMD), |
| 427 | stdout='\n'.join(test_binaries)) |
| 428 | for binary in test_binaries: |
| 429 | osutils.Touch(os.path.join(self.deploy.options.build_dir, binary), |
| 430 | makedirs=True, mode=0o700) |
| 431 | |
| 432 | self.deploy._DeployTestBinaries() |
| 433 | |
| 434 | # Ensure the binaries were placed in the staging dir used to copy them over. |
| 435 | staging_dir = os.path.join( |
| 436 | self.tempdir, os.path.basename(deploy_chrome._CHROME_TEST_BIN_DIR)) |
| 437 | for binary in test_binaries: |
| 438 | self.assertIn(binary, os.listdir(staging_dir)) |
Erik Chen | 75a2f49 | 2020-08-06 19:15:11 -0700 | [diff] [blame] | 439 | |
| 440 | |
| 441 | class LacrosPerformTest(cros_test_lib.RunCommandTempDirTestCase): |
| 442 | """Line coverage for Perform() method with --lacros option.""" |
| 443 | |
| 444 | def setUp(self): |
| 445 | options = _ParseCommandLine([ |
| 446 | '--lacros', '--nostrip', '--build-dir', '/path/to/nowhere', |
| 447 | '--to', 'localhost']) |
| 448 | self.deploy = deploy_chrome.DeployChrome( |
| 449 | options, self.tempdir, os.path.join(self.tempdir, 'staging')) |
| 450 | |
| 451 | # These methods being mocked are all side effects expected for a --lacros |
| 452 | # deploy. |
| 453 | self.deploy._EnsureTargetDir = mock.Mock() |
| 454 | self.deploy._GetDeviceInfo = mock.Mock() |
| 455 | self.deploy._CheckConnection = mock.Mock() |
| 456 | self.deploy._MountRootfsAsWritable = mock.Mock() |
| 457 | self.deploy._PrepareStagingDir = mock.Mock() |
| 458 | self.deploy._CheckDeviceFreeSpace = mock.Mock() |
| 459 | |
| 460 | self._ran_start_command = False |
| 461 | |
| 462 | self.StartPatcher(parallel_unittest.ParallelMock()) |
| 463 | |
| 464 | # Common mocking shared between tests. |
| 465 | def kill_procs_side_effect(): |
| 466 | self.deploy._stopped_ui = True |
| 467 | self.deploy._KillAshChromeIfNeeded = mock.Mock( |
| 468 | side_effect=kill_procs_side_effect) |
| 469 | |
| 470 | def start_ui_side_effect(*args, **kwargs): |
| 471 | # pylint: disable=unused-argument |
| 472 | self._ran_start_command = True |
| 473 | self.rc.AddCmdResult(partial_mock.In('start ui'), |
| 474 | side_effect=start_ui_side_effect) |
| 475 | |
| 476 | def testConfNotModified(self): |
| 477 | """When the conf file is not modified we don't restart chrome .""" |
| 478 | self.deploy.Perform() |
| 479 | self.deploy._KillAshChromeIfNeeded.assert_not_called() |
| 480 | self.assertFalse(self._ran_start_command) |
| 481 | |
| 482 | def testConfModified(self): |
| 483 | """When the conf file is modified we restart chrome.""" |
| 484 | |
| 485 | # We intentionally add '\n' to MODIFIED_CONF_FILE to simulate echo adding a |
| 486 | # newline when invoked in the shell. |
| 487 | self.rc.AddCmdResult( |
| 488 | partial_mock.In(deploy_chrome.ENABLE_LACROS_VIA_CONF_COMMAND), |
| 489 | stdout=deploy_chrome.MODIFIED_CONF_FILE + '\n') |
| 490 | |
| 491 | self.deploy.Perform() |
| 492 | self.deploy._KillAshChromeIfNeeded.assert_called() |
| 493 | self.assertTrue(self._ran_start_command) |