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