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