Mike Frysinger | f1ba7ad | 2022-09-12 05:42:57 -0400 | [diff] [blame] | 1 | # Copyright 2019 The ChromiumOS Authors |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """SDK tests.""" |
| 6 | |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 7 | import os |
Greg Edelston | 01ae594 | 2023-01-30 16:26:54 -0700 | [diff] [blame] | 8 | import pathlib |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 9 | from typing import List, Optional |
Mike Frysinger | 166fea0 | 2021-02-12 05:30:33 -0500 | [diff] [blame] | 10 | from unittest import mock |
| 11 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 12 | from chromite.api import api_config |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 13 | from chromite.api.controller import sdk as sdk_controller |
Alex Klein | 7107bdd | 2019-03-14 17:14:31 -0600 | [diff] [blame] | 14 | from chromite.api.gen.chromite.api import sdk_pb2 |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 15 | from chromite.api.gen.chromiumos import common_pb2 |
| 16 | from chromite.lib import constants |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 17 | from chromite.lib import cros_build_lib |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 18 | from chromite.lib import cros_test_lib |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 19 | from chromite.service import sdk as sdk_service |
| 20 | |
| 21 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 22 | class SdkCreateTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 23 | """Create tests.""" |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 24 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 25 | def setUp(self): |
| 26 | """Setup method.""" |
| 27 | # We need to run the command outside the chroot. |
| 28 | self.PatchObject(cros_build_lib, "IsInsideChroot", return_value=False) |
| 29 | self.response = sdk_pb2.CreateResponse() |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 30 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 31 | def _GetRequest( |
| 32 | self, |
| 33 | no_replace=False, |
Chris McDonald | 5dcdb89 | 2020-02-07 15:10:46 -0700 | [diff] [blame] | 34 | bootstrap=False, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 35 | cache_path=None, |
| 36 | chroot_path=None, |
| 37 | sdk_version=None, |
| 38 | skip_chroot_upgrade=False, |
| 39 | ): |
| 40 | """Helper to build a create request message.""" |
| 41 | request = sdk_pb2.CreateRequest() |
| 42 | request.flags.no_replace = no_replace |
| 43 | request.flags.bootstrap = bootstrap |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 44 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 45 | if cache_path: |
| 46 | request.chroot.cache_dir = cache_path |
| 47 | if chroot_path: |
| 48 | request.chroot.path = chroot_path |
| 49 | if sdk_version: |
| 50 | request.sdk_version = sdk_version |
| 51 | if skip_chroot_upgrade: |
| 52 | request.skip_chroot_upgrade = skip_chroot_upgrade |
Alex Klein | 19c4cc4 | 2019-02-27 14:47:57 -0700 | [diff] [blame] | 53 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 54 | return request |
| 55 | |
| 56 | def testValidateOnly(self): |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 57 | """Verify a validate-only call does not execute any logic.""" |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 58 | patch = self.PatchObject(sdk_service, "Create") |
| 59 | |
| 60 | sdk_controller.Create( |
| 61 | self._GetRequest(), self.response, self.validate_only_config |
| 62 | ) |
| 63 | patch.assert_not_called() |
| 64 | |
| 65 | def testMockCall(self): |
| 66 | """Sanity check that a mock call does not execute any logic.""" |
| 67 | patch = self.PatchObject(sdk_service, "Create") |
| 68 | |
| 69 | rc = sdk_controller.Create( |
| 70 | self._GetRequest(), self.response, self.mock_call_config |
| 71 | ) |
| 72 | patch.assert_not_called() |
| 73 | self.assertFalse(rc) |
| 74 | self.assertTrue(self.response.version.version) |
| 75 | |
| 76 | def testSuccess(self): |
| 77 | """Test the successful call output handling.""" |
| 78 | self.PatchObject(sdk_service, "Create", return_value=1) |
| 79 | |
| 80 | request = self._GetRequest() |
| 81 | |
| 82 | sdk_controller.Create(request, self.response, self.api_config) |
| 83 | |
| 84 | self.assertEqual(1, self.response.version.version) |
| 85 | |
| 86 | def testFalseArguments(self): |
| 87 | """Test False argument handling.""" |
| 88 | # Create the patches. |
| 89 | self.PatchObject(sdk_service, "Create", return_value=1) |
| 90 | args_patch = self.PatchObject(sdk_service, "CreateArguments") |
| 91 | |
| 92 | # Flag translation tests. |
| 93 | # Test all false values in the message. |
| 94 | request = self._GetRequest( |
Brian Norris | 35a7ed0 | 2023-02-23 12:50:14 -0800 | [diff] [blame] | 95 | no_replace=False, |
| 96 | bootstrap=False, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 97 | ) |
| 98 | sdk_controller.Create(request, self.response, self.api_config) |
| 99 | args_patch.assert_called_with( |
| 100 | replace=True, |
| 101 | bootstrap=False, |
Brian Norris | 67374d8 | 2023-05-01 12:31:12 -0700 | [diff] [blame] | 102 | chroot=mock.ANY, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 103 | sdk_version=mock.ANY, |
| 104 | skip_chroot_upgrade=mock.ANY, |
| 105 | ) |
| 106 | |
| 107 | def testTrueArguments(self): |
| 108 | """Test True arguments handling.""" |
| 109 | # Create the patches. |
| 110 | self.PatchObject(sdk_service, "Create", return_value=1) |
| 111 | args_patch = self.PatchObject(sdk_service, "CreateArguments") |
| 112 | |
| 113 | # Test all True values in the message. |
| 114 | request = self._GetRequest( |
| 115 | no_replace=True, |
| 116 | bootstrap=True, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 117 | sdk_version="foo", |
| 118 | skip_chroot_upgrade=True, |
| 119 | ) |
| 120 | sdk_controller.Create(request, self.response, self.api_config) |
| 121 | args_patch.assert_called_with( |
| 122 | replace=False, |
| 123 | bootstrap=True, |
Brian Norris | 67374d8 | 2023-05-01 12:31:12 -0700 | [diff] [blame] | 124 | chroot=mock.ANY, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 125 | sdk_version="foo", |
| 126 | skip_chroot_upgrade=True, |
| 127 | ) |
Mike Frysinger | cb8992a | 2020-02-11 05:13:13 +0000 | [diff] [blame] | 128 | |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 129 | |
George Engelbrecht | 86cfae6 | 2023-04-05 10:57:41 -0600 | [diff] [blame] | 130 | class SdkCleanTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
| 131 | """Clean tests.""" |
| 132 | |
| 133 | def setUp(self): |
| 134 | """Setup method.""" |
| 135 | # We need to run the command outside the chroot. |
| 136 | self.PatchObject(cros_build_lib, "IsInsideChroot", return_value=False) |
| 137 | self.response = sdk_pb2.CleanResponse() |
| 138 | |
| 139 | def _GetRequest(self, chroot_path=None, incrementals=False): |
| 140 | """Helper to build a clean request message.""" |
| 141 | request = sdk_pb2.CleanRequest() |
| 142 | if chroot_path: |
| 143 | request.chroot.path = chroot_path |
| 144 | |
| 145 | request.incrementals = incrementals |
| 146 | |
| 147 | return request |
| 148 | |
| 149 | def testMockCall(self): |
| 150 | """Sanity check that a mock call does not execute any logic.""" |
| 151 | patch = self.PatchObject(sdk_service, "Clean") |
| 152 | |
| 153 | rc = sdk_controller.Clean( |
| 154 | self._GetRequest(), self.response, self.mock_call_config |
| 155 | ) |
| 156 | patch.assert_not_called() |
| 157 | self.assertFalse(rc) |
| 158 | |
| 159 | def testSuccess(self): |
| 160 | """Test the successful call by verifying service invocation.""" |
| 161 | patch = self.PatchObject(sdk_service, "Clean", return_value=0) |
| 162 | |
| 163 | request = self._GetRequest(incrementals=True) |
| 164 | |
| 165 | sdk_controller.Clean(request, self.response, self.api_config) |
| 166 | patch.assert_called_once_with( |
| 167 | mock.ANY, |
| 168 | safe=False, |
| 169 | images=False, |
| 170 | sysroots=False, |
| 171 | tmp=False, |
| 172 | cache=False, |
| 173 | logs=False, |
| 174 | workdirs=False, |
| 175 | incrementals=True, |
| 176 | ) |
| 177 | |
| 178 | def testDefaults(self): |
| 179 | """Test the successful call by verifying service invocation.""" |
| 180 | patch = self.PatchObject(sdk_service, "Clean", return_value=0) |
| 181 | |
| 182 | request = self._GetRequest() |
| 183 | |
| 184 | sdk_controller.Clean(request, self.response, self.api_config) |
| 185 | patch.assert_called_once_with(mock.ANY, safe=True, sysroots=True) |
| 186 | |
| 187 | |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 188 | class SdkDeleteTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
George Engelbrecht | 86cfae6 | 2023-04-05 10:57:41 -0600 | [diff] [blame] | 189 | """Delete tests.""" |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 190 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 191 | def setUp(self): |
| 192 | """Setup method.""" |
| 193 | # We need to run the command outside the chroot. |
| 194 | self.PatchObject(cros_build_lib, "IsInsideChroot", return_value=False) |
| 195 | self.response = sdk_pb2.DeleteResponse() |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 196 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 197 | def _GetRequest(self, chroot_path=None): |
| 198 | """Helper to build a delete request message.""" |
| 199 | request = sdk_pb2.DeleteRequest() |
| 200 | if chroot_path: |
| 201 | request.chroot.path = chroot_path |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 202 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 203 | return request |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 204 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 205 | def testValidateOnly(self): |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 206 | """Verify a validate-only call does not execute any logic.""" |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 207 | patch = self.PatchObject(sdk_service, "Delete") |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 208 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 209 | sdk_controller.Delete( |
| 210 | self._GetRequest(), self.response, self.validate_only_config |
| 211 | ) |
| 212 | patch.assert_not_called() |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 213 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 214 | def testMockCall(self): |
| 215 | """Sanity check that a mock call does not execute any logic.""" |
| 216 | patch = self.PatchObject(sdk_service, "Delete") |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 217 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 218 | rc = sdk_controller.Delete( |
| 219 | self._GetRequest(), self.response, self.mock_call_config |
| 220 | ) |
| 221 | patch.assert_not_called() |
| 222 | self.assertFalse(rc) |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 223 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 224 | def testSuccess(self): |
| 225 | """Test the successful call by verifying service invocation.""" |
| 226 | patch = self.PatchObject(sdk_service, "Delete", return_value=1) |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 227 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 228 | request = self._GetRequest() |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 229 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 230 | sdk_controller.Delete(request, self.response, self.api_config) |
| 231 | # Verify that by default sdk_service.Delete is called with force=True. |
| 232 | patch.assert_called_once_with(mock.ANY, force=True) |
Michael Mortensen | e87d8a6 | 2020-07-06 11:44:35 -0600 | [diff] [blame] | 233 | |
| 234 | |
Brian Norris | f624bf4 | 2023-03-02 12:57:49 -0800 | [diff] [blame] | 235 | class SdkUnmountTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
| 236 | """SDK Unmount tests.""" |
| 237 | |
| 238 | def testNoop(self): |
| 239 | """Unmount is a deprecated noop.""" |
| 240 | request = sdk_pb2.UnmountRequest() |
| 241 | response = sdk_pb2.UnmountResponse() |
| 242 | rc = sdk_controller.Unmount(request, response, self.api_config) |
| 243 | self.assertFalse(rc) |
| 244 | |
| 245 | |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 246 | class SdkUnmountPathTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 247 | """Update tests.""" |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 248 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 249 | def setUp(self): |
| 250 | """Setup method.""" |
| 251 | self.response = sdk_pb2.UnmountPathResponse() |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 252 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 253 | def _UnmountPathRequest(self, path=None): |
| 254 | """Helper to build a delete request message.""" |
| 255 | request = sdk_pb2.UnmountPathRequest() |
| 256 | if path: |
| 257 | request.path.path = path |
| 258 | return request |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 259 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 260 | def testValidateOnly(self): |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 261 | """Verify a validate-only call does not execute any logic.""" |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 262 | patch = self.PatchObject(sdk_service, "UnmountPath") |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 263 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 264 | sdk_controller.UnmountPath( |
| 265 | self._UnmountPathRequest("/test/path"), |
| 266 | self.response, |
| 267 | self.validate_only_config, |
| 268 | ) |
| 269 | patch.assert_not_called() |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 270 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 271 | def testMockCall(self): |
| 272 | """Sanity check that a mock call does not execute any logic.""" |
| 273 | patch = self.PatchObject(sdk_service, "UnmountPath") |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 274 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 275 | rc = sdk_controller.UnmountPath( |
| 276 | self._UnmountPathRequest(), self.response, self.mock_call_config |
| 277 | ) |
| 278 | patch.assert_not_called() |
| 279 | self.assertFalse(rc) |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 280 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 281 | def testSuccess(self): |
| 282 | """Test the successful call by verifying service invocation.""" |
| 283 | patch = self.PatchObject(sdk_service, "UnmountPath", return_value=1) |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 284 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 285 | request = self._UnmountPathRequest("/test/path") |
| 286 | sdk_controller.UnmountPath(request, self.response, self.api_config) |
| 287 | patch.assert_called_once_with("/test/path") |
Michael Mortensen | 52a98ac | 2020-07-28 16:00:18 -0600 | [diff] [blame] | 288 | |
| 289 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 290 | class SdkUpdateTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 291 | """Update tests.""" |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 292 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 293 | def setUp(self): |
| 294 | """Setup method.""" |
| 295 | # We need to run the command inside the chroot. |
| 296 | self.PatchObject(cros_build_lib, "IsInsideChroot", return_value=True) |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 297 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 298 | self.response = sdk_pb2.UpdateResponse() |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 299 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 300 | def _GetRequest(self, build_source=False, targets=None): |
| 301 | """Helper to simplify building a request instance.""" |
| 302 | request = sdk_pb2.UpdateRequest() |
| 303 | request.flags.build_source = build_source |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 304 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 305 | for target in targets or []: |
| 306 | added = request.toolchain_targets.add() |
| 307 | added.name = target |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 308 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 309 | return request |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 310 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 311 | def testValidateOnly(self): |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 312 | """Verify a validate-only call does not execute any logic.""" |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 313 | patch = self.PatchObject(sdk_service, "Update") |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 314 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 315 | sdk_controller.Update( |
| 316 | self._GetRequest(), self.response, self.validate_only_config |
| 317 | ) |
| 318 | patch.assert_not_called() |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 319 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 320 | def testMockCall(self): |
| 321 | """Sanity check that a mock call does not execute any logic.""" |
| 322 | patch = self.PatchObject(sdk_service, "Update") |
Alex Klein | 076841b | 2019-08-29 15:19:39 -0600 | [diff] [blame] | 323 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 324 | rc = sdk_controller.Create( |
| 325 | self._GetRequest(), self.response, self.mock_call_config |
| 326 | ) |
| 327 | patch.assert_not_called() |
| 328 | self.assertFalse(rc) |
| 329 | self.assertTrue(self.response.version.version) |
Alex Klein | 076841b | 2019-08-29 15:19:39 -0600 | [diff] [blame] | 330 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 331 | def testSuccess(self): |
| 332 | """Successful call output handling test.""" |
| 333 | expected_version = 1 |
| 334 | self.PatchObject(sdk_service, "Update", return_value=expected_version) |
| 335 | request = self._GetRequest() |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 336 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 337 | sdk_controller.Update(request, self.response, self.api_config) |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 338 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 339 | self.assertEqual(expected_version, self.response.version.version) |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 340 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 341 | def testArgumentHandling(self): |
| 342 | """Test the proto argument handling.""" |
| 343 | args = sdk_service.UpdateArguments() |
| 344 | self.PatchObject(sdk_service, "Update", return_value=1) |
| 345 | args_patch = self.PatchObject( |
| 346 | sdk_service, "UpdateArguments", return_value=args |
| 347 | ) |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 348 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 349 | # No boards and flags False. |
| 350 | request = self._GetRequest(build_source=False) |
| 351 | sdk_controller.Update(request, self.response, self.api_config) |
| 352 | args_patch.assert_called_with( |
| 353 | build_source=False, toolchain_targets=[], toolchain_changed=False |
| 354 | ) |
Alex Klein | aa5c417 | 2019-02-27 17:12:20 -0700 | [diff] [blame] | 355 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 356 | # Multiple boards and flags True. |
| 357 | targets = ["board1", "board2"] |
| 358 | request = self._GetRequest(build_source=True, targets=targets) |
| 359 | sdk_controller.Update(request, self.response, self.api_config) |
| 360 | args_patch.assert_called_with( |
| 361 | build_source=True, |
| 362 | toolchain_targets=targets, |
| 363 | toolchain_changed=False, |
| 364 | ) |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 365 | |
| 366 | |
Greg Edelston | 01ae594 | 2023-01-30 16:26:54 -0700 | [diff] [blame] | 367 | class CreateManifestFromSdkTest( |
| 368 | cros_test_lib.MockTestCase, api_config.ApiConfigMixin |
| 369 | ): |
| 370 | """Test the SdkService/CreateManifestFromSdk endpoint.""" |
| 371 | |
| 372 | _chroot_path = "/path/to/chroot" |
| 373 | _sdk_path_relative = "build/my_sdk" |
| 374 | _dest_dir = "/build" |
| 375 | _manifest_path = "/build/my_sdk.Manifest" |
| 376 | |
| 377 | def _NewRequest(self, inside: bool) -> sdk_pb2.CreateManifestFromSdkRequest: |
| 378 | return sdk_pb2.CreateManifestFromSdkRequest( |
| 379 | chroot=common_pb2.Chroot(path=self._chroot_path), |
| 380 | sdk_path=common_pb2.Path( |
| 381 | path="/%s" % self._sdk_path_relative, |
| 382 | location=common_pb2.Path.Location.INSIDE |
| 383 | if inside |
| 384 | else common_pb2.Path.Location.OUTSIDE, |
| 385 | ), |
| 386 | dest_dir=common_pb2.Path( |
| 387 | path=self._dest_dir, |
| 388 | location=common_pb2.Path.Location.OUTSIDE, |
| 389 | ), |
| 390 | ) |
| 391 | |
| 392 | def _NewResponse(self) -> sdk_pb2.CreateManifestFromSdkResponse: |
| 393 | return sdk_pb2.CreateManifestFromSdkResponse() |
| 394 | |
| 395 | def testValidateOnly(self): |
| 396 | """Check that a validate only call does not execute any logic.""" |
| 397 | impl_patch = self.PatchObject(sdk_service, "CreateManifestFromSdk") |
Greg Edelston | e265a17 | 2023-06-26 17:20:05 -0600 | [diff] [blame^] | 398 | sdk_controller.CreateManifestFromSdk( |
Greg Edelston | 01ae594 | 2023-01-30 16:26:54 -0700 | [diff] [blame] | 399 | self._NewRequest(False), |
| 400 | self._NewResponse(), |
| 401 | self.validate_only_config, |
| 402 | ) |
| 403 | impl_patch.assert_not_called() |
| 404 | |
| 405 | def testOutside(self): |
| 406 | """Check that a call with an outside path succeeds.""" |
| 407 | impl_patch = self.PatchObject( |
| 408 | sdk_service, |
| 409 | "CreateManifestFromSdk", |
| 410 | return_value=pathlib.Path(self._manifest_path), |
| 411 | ) |
| 412 | request = self._NewRequest(inside=False) |
| 413 | response = self._NewResponse() |
| 414 | sdk_controller.CreateManifestFromSdk( |
| 415 | request, |
| 416 | response, |
| 417 | self.api_config, |
| 418 | ) |
| 419 | impl_patch.assert_called_with( |
| 420 | pathlib.Path("/", self._sdk_path_relative), |
| 421 | pathlib.Path(self._dest_dir), |
| 422 | ) |
| 423 | self.assertEqual( |
| 424 | response.manifest_path.location, common_pb2.Path.Location.OUTSIDE |
| 425 | ) |
| 426 | self.assertEqual(response.manifest_path.path, self._manifest_path) |
| 427 | |
| 428 | def testInside(self): |
| 429 | """Check that an inside path parses correctly and the call succeeds.""" |
| 430 | impl_patch = self.PatchObject( |
| 431 | sdk_service, |
| 432 | "CreateManifestFromSdk", |
| 433 | return_value=pathlib.Path(self._manifest_path), |
| 434 | ) |
| 435 | request = self._NewRequest(inside=True) |
| 436 | response = self._NewResponse() |
| 437 | sdk_controller.CreateManifestFromSdk( |
| 438 | request, |
| 439 | response, |
| 440 | self.api_config, |
| 441 | ) |
| 442 | impl_patch.assert_called_with( |
| 443 | pathlib.Path(self._chroot_path, self._sdk_path_relative), |
| 444 | pathlib.Path(self._dest_dir), |
| 445 | ) |
| 446 | self.assertEqual( |
| 447 | response.manifest_path.location, common_pb2.Path.Location.OUTSIDE |
| 448 | ) |
| 449 | self.assertEqual(response.manifest_path.path, self._manifest_path) |
| 450 | |
| 451 | |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 452 | class BuildSdkToolchainTest( |
| 453 | cros_test_lib.MockTestCase, api_config.ApiConfigMixin |
| 454 | ): |
| 455 | """Test the SdkService/BuildSdkToolchain endpoint.""" |
| 456 | |
| 457 | def setUp(self): |
| 458 | """Set up the test case.""" |
| 459 | self._chroot_path = "/path/to/chroot" |
Greg Edelston | e265a17 | 2023-06-26 17:20:05 -0600 | [diff] [blame^] | 460 | self._result_dir = "/out/toolchain-pkgs/" |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 461 | self._response = sdk_pb2.BuildSdkToolchainResponse() |
| 462 | self._generated_filenames = ( |
| 463 | "armv7a-cros-linux-gnueabihf.tar.xz", |
| 464 | "x86_64-cros-linux-gnu.tar.xz", |
| 465 | ) |
| 466 | self._paths_for_generated_files = [ |
| 467 | common_pb2.Path( |
| 468 | path=os.path.join(constants.SDK_TOOLCHAINS_OUTPUT, fname), |
| 469 | location=common_pb2.Path.Location.INSIDE, |
| 470 | ) |
| 471 | for fname in self._generated_filenames |
| 472 | ] |
| 473 | |
| 474 | def _NewRequest( |
| 475 | self, |
| 476 | chroot_path: Optional[str] = None, |
| 477 | use_flags: Optional[List[str]] = None, |
| 478 | ) -> sdk_pb2.BuildSdkToolchainRequest: |
| 479 | """Return a new BuildSdkToolchainRequest message.""" |
Greg Edelston | e265a17 | 2023-06-26 17:20:05 -0600 | [diff] [blame^] | 480 | request = sdk_pb2.BuildSdkToolchainRequest( |
| 481 | result_path=common_pb2.ResultPath( |
| 482 | path=common_pb2.Path( |
| 483 | path=self._result_dir, |
| 484 | location=common_pb2.Path.Location.OUTSIDE, |
| 485 | ) |
| 486 | ) |
| 487 | ) |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 488 | if chroot_path: |
| 489 | request.chroot.path = chroot_path |
| 490 | if use_flags: |
| 491 | request.use_flags.extend( |
| 492 | common_pb2.UseFlag(flag=flag) for flag in use_flags |
| 493 | ) |
| 494 | return request |
| 495 | |
| 496 | def _NewResponse( |
| 497 | self, generated_filenames: Optional[List[str]] = None |
| 498 | ) -> sdk_pb2.BuildSdkToolchainResponse: |
| 499 | """Return a new BuildSdkToolchainResponse message.""" |
| 500 | response = sdk_pb2.BuildSdkToolchainResponse() |
| 501 | if generated_filenames: |
| 502 | response.generated_files.extend( |
| 503 | common_pb2.Path( |
Greg Edelston | e265a17 | 2023-06-26 17:20:05 -0600 | [diff] [blame^] | 504 | path=os.path.join(self._result_dir, fname), |
| 505 | location=common_pb2.Path.Location.OUTSIDE, |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 506 | ) |
| 507 | for fname in generated_filenames |
| 508 | ) |
| 509 | return response |
| 510 | |
| 511 | def testValidateOnly(self): |
| 512 | """Check that a validate only call does not execute any logic.""" |
| 513 | impl_patch = self.PatchObject(sdk_service, "BuildSdkToolchain") |
| 514 | sdk_controller.BuildSdkToolchain( |
| 515 | self._NewRequest(), self._NewResponse(), self.validate_only_config |
| 516 | ) |
| 517 | impl_patch.assert_not_called() |
| 518 | |
| 519 | def testSuccess(self): |
| 520 | """Check that a normal call defers to the SDK service as expected.""" |
| 521 | impl_patch = self.PatchObject(sdk_service, "BuildSdkToolchain") |
| 522 | request = self._NewRequest(use_flags=[]) |
| 523 | response = self._NewResponse() |
| 524 | sdk_controller.BuildSdkToolchain( |
| 525 | request, |
| 526 | response, |
| 527 | self.api_config, |
| 528 | ) |
| 529 | # Can't use assert_called_with, since the chroot objects are equal but |
| 530 | # not identical. |
| 531 | impl_patch.assert_called_once() |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 532 | self.assertEqual(impl_patch.call_args.kwargs["extra_env"], {}) |
| 533 | |
| 534 | def testSuccessWithUseFlags(self): |
| 535 | """Check that a call with USE flags works as expected.""" |
| 536 | impl_patch = self.PatchObject(sdk_service, "BuildSdkToolchain") |
| 537 | request = self._NewRequest(use_flags=["llvm-next", "another-flag"]) |
| 538 | response = self._NewResponse() |
| 539 | sdk_controller.BuildSdkToolchain( |
| 540 | request, |
| 541 | response, |
| 542 | self.api_config, |
| 543 | ) |
| 544 | # Can't use assert_called_with, since the chroot objects are equal but |
| 545 | # not identical. |
| 546 | impl_patch.assert_called_once() |
| 547 | self.assertEqual( |
Greg Edelston | 9dcdc8a | 2023-01-11 17:07:10 -0700 | [diff] [blame] | 548 | impl_patch.call_args.kwargs["extra_env"], |
| 549 | {"USE": "llvm-next another-flag"}, |
| 550 | ) |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 551 | |
| 552 | |
Greg Edelston | d401e5a | 2023-04-28 15:29:11 -0600 | [diff] [blame] | 553 | class uprev_test(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 554 | """Test case for SdkService/Uprev() endpoint.""" |
| 555 | |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 556 | _binhost_gs_bucket = "gs://chromiumos-prebuilts/" |
Greg Edelston | 40aea81 | 2023-03-27 16:34:35 -0600 | [diff] [blame] | 557 | _latest_uprev_target_version = "2023.02.19.112358" |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 558 | |
| 559 | def setUp(self): |
| 560 | """Set up the test case.""" |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 561 | self.PatchObject( |
| 562 | sdk_service, |
Greg Edelston | d401e5a | 2023-04-28 15:29:11 -0600 | [diff] [blame] | 563 | "get_latest_uprev_target_version", |
Greg Edelston | 40aea81 | 2023-03-27 16:34:35 -0600 | [diff] [blame] | 564 | return_value=self._latest_uprev_target_version, |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 565 | ) |
| 566 | self._uprev_patch = self.PatchObject( |
| 567 | sdk_service, |
Greg Edelston | d401e5a | 2023-04-28 15:29:11 -0600 | [diff] [blame] | 568 | "uprev_sdk_and_prebuilts", |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 569 | ) |
| 570 | |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 571 | def NewRequest( |
| 572 | self, version: str = "", toolchain_tarball_template: str = "" |
| 573 | ): |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 574 | """Return a new UprevRequest with standard inputs.""" |
| 575 | return sdk_pb2.UprevRequest( |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 576 | binhost_gs_bucket=self._binhost_gs_bucket, |
| 577 | version=version, |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 578 | toolchain_tarball_template=toolchain_tarball_template, |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 579 | ) |
| 580 | |
| 581 | @staticmethod |
| 582 | def NewResponse() -> sdk_pb2.UprevResponse: |
| 583 | """Return a new empty UprevResponse.""" |
| 584 | return sdk_pb2.UprevResponse() |
| 585 | |
| 586 | def testWithVersion(self): |
| 587 | """Test the endpoint with `version` specified. |
| 588 | |
| 589 | In this case, we expect that sdk_controller.Uprev is called with the |
| 590 | version specified in the UprevRequest. |
| 591 | """ |
| 592 | specified_version = "1970.01.01.000000" |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 593 | toolchain_tarball_template = "path/to/%(version)s/toolchain" |
| 594 | request = self.NewRequest( |
| 595 | version=specified_version, |
| 596 | toolchain_tarball_template=toolchain_tarball_template, |
| 597 | ) |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 598 | response = self.NewResponse() |
| 599 | sdk_controller.Uprev(request, response, self.api_config) |
| 600 | self._uprev_patch.assert_called_with( |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 601 | binhost_gs_bucket=self._binhost_gs_bucket, |
| 602 | version=specified_version, |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 603 | toolchain_tarball_template=toolchain_tarball_template, |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 604 | ) |
| 605 | |
| 606 | def testWithoutVersion(self): |
| 607 | """Test the endpoint with `version` not specified. |
| 608 | |
| 609 | In this case, we expect that sdk_controller.Uprev is called with the |
Greg Edelston | 40aea81 | 2023-03-27 16:34:35 -0600 | [diff] [blame] | 610 | latest uprev target version, based on the remote file in gs://. This is |
| 611 | fetched via sdk_controller.GetLatestUprevTargetVersionVersion |
| 612 | (mocked here in setUp()). |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 613 | """ |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 614 | toolchain_tarball_template = "path/to/%(version)s/toolchain" |
| 615 | request = self.NewRequest( |
| 616 | toolchain_tarball_template=toolchain_tarball_template |
| 617 | ) |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 618 | response = self.NewResponse() |
| 619 | sdk_controller.Uprev(request, response, self.api_config) |
| 620 | self._uprev_patch.assert_called_with( |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 621 | binhost_gs_bucket=self._binhost_gs_bucket, |
Greg Edelston | 40aea81 | 2023-03-27 16:34:35 -0600 | [diff] [blame] | 622 | version=self._latest_uprev_target_version, |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 623 | toolchain_tarball_template=toolchain_tarball_template, |
Greg Edelston | 6733dc5 | 2023-02-15 15:20:07 -0700 | [diff] [blame] | 624 | ) |
Greg Edelston | 0382ca4 | 2023-05-04 14:00:15 -0600 | [diff] [blame] | 625 | |
| 626 | def testWithoutToolchainTarballTemplate(self): |
| 627 | """Test the endpoint with `toolchain_tarball_template` not specified.""" |
| 628 | request = self.NewRequest(version="1234") |
| 629 | response = self.NewResponse() |
| 630 | with self.assertRaises(cros_build_lib.DieSystemExit): |
| 631 | sdk_controller.Uprev(request, response, self.api_config) |