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