LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 1 | # Copyright 2019 The Chromium OS Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Unittests for Android operations.""" |
| 6 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 7 | from chromite.api import api_config |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 8 | from chromite.api.controller import android |
| 9 | from chromite.api.gen.chromite.api import android_pb2 |
| 10 | from chromite.api.gen.chromiumos import common_pb2 |
Mike Frysinger | 40ffb53 | 2021-02-12 07:36:08 -0500 | [diff] [blame] | 11 | from chromite.lib import build_target_lib |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 12 | from chromite.lib import constants |
| 13 | from chromite.lib import cros_build_lib |
| 14 | from chromite.lib import cros_test_lib |
| 15 | from chromite.lib import osutils |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 16 | from chromite.service import android as service_android |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 17 | from chromite.service import packages |
Mike Frysinger | 40ffb53 | 2021-02-12 07:36:08 -0500 | [diff] [blame] | 18 | from chromite.third_party import mock |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 19 | |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 20 | |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 21 | class GetLatestBuildTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
| 22 | """Unittests for GetLatestBuild.""" |
| 23 | |
| 24 | def setUp(self): |
| 25 | self._mock = self.PatchObject(service_android, 'GetLatestBuild') |
| 26 | self._mock.return_value = ('7123456', {}) |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 27 | self._mock_branch_for_package = self.PatchObject( |
| 28 | service_android, 'GetAndroidBranchForPackage', |
| 29 | return_value='android-branch-for-package') |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 30 | self._output_proto = android_pb2.GetLatestBuildResponse() |
| 31 | |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 32 | def _GetRequest(self, android_build_branch=None, android_package=None): |
| 33 | req = android_pb2.GetLatestBuildRequest() |
| 34 | if android_build_branch is not None: |
| 35 | req.android_build_branch = android_build_branch |
| 36 | if android_package is not None: |
| 37 | req.android_package = android_package |
| 38 | return req |
| 39 | |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 40 | def testValidateOnly(self): |
| 41 | """Test that a validate only call does not execute any logic.""" |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 42 | req = self._GetRequest(android_package='android-package') |
| 43 | android.GetLatestBuild(req, self._output_proto, self.validate_only_config) |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 44 | self._mock.assert_not_called() |
| 45 | |
| 46 | def testMockCall(self): |
| 47 | """Test that a mock call does not execute logic, returns mocked value.""" |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 48 | req = self._GetRequest(android_package='android-package') |
| 49 | android.GetLatestBuild(req, self._output_proto, self.mock_call_config) |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 50 | self._mock.assert_not_called() |
| 51 | self.assertEqual(self._output_proto.android_version, '7123456') |
| 52 | |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 53 | def testFailsIfBranchAndPackageMissing(self): |
| 54 | """Fails if both android_build_branch and android_package are missing.""" |
| 55 | req = self._GetRequest() |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 56 | with self.assertRaises(cros_build_lib.DieSystemExit): |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 57 | android.GetLatestBuild(req, self._output_proto, self.api_config) |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 58 | self._mock.assert_not_called() |
| 59 | |
Shao-Chuan Lee | 6e8784a | 2021-05-13 09:34:46 +0900 | [diff] [blame] | 60 | def testBranchSpecified(self): |
| 61 | """Test calling with Android branch specified.""" |
| 62 | req = self._GetRequest(android_build_branch='android-branch') |
| 63 | android.GetLatestBuild(req, self._output_proto, self.api_config) |
| 64 | self._mock.assert_called_once_with('android-branch') |
| 65 | self._mock_branch_for_package.assert_not_called() |
| 66 | self.assertEqual(self._output_proto.android_version, '7123456') |
| 67 | |
| 68 | def testPackageSpecified(self): |
| 69 | """Test calling with Android package specified.""" |
| 70 | req = self._GetRequest(android_package='android-package') |
| 71 | android.GetLatestBuild(req, self._output_proto, self.api_config) |
| 72 | self._mock.assert_called_once_with('android-branch-for-package') |
| 73 | self._mock_branch_for_package.assert_called_once_with('android-package') |
| 74 | self.assertEqual(self._output_proto.android_version, '7123456') |
| 75 | |
| 76 | def testBranchAndPackageSpecified(self): |
| 77 | """Test calling with both Android branch and package specified.""" |
| 78 | req = self._GetRequest(android_build_branch='android-branch', |
| 79 | android_package='android-package') |
| 80 | android.GetLatestBuild(req, self._output_proto, self.api_config) |
| 81 | self._mock.assert_called_once_with('android-branch') |
| 82 | self._mock_branch_for_package.assert_not_called() |
Shao-Chuan Lee | 01dee22 | 2021-04-09 15:28:08 +0900 | [diff] [blame] | 83 | self.assertEqual(self._output_proto.android_version, '7123456') |
| 84 | |
| 85 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 86 | class MarkStableTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 87 | """Unittests for MarkStable.""" |
| 88 | |
| 89 | def setUp(self): |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 90 | self.uprev = self.PatchObject(packages, 'uprev_android') |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 91 | |
| 92 | self.input_proto = android_pb2.MarkStableRequest() |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 93 | self.input_proto.package_name = 'android-package-name' |
| 94 | self.input_proto.android_build_branch = 'android_build_branch' |
Shao-Chuan Lee | a4b4f30 | 2021-05-12 14:40:20 +0900 | [diff] [blame] | 95 | self.input_proto.android_version = 'android-version' |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 96 | self.input_proto.build_targets.add().name = 'foo' |
| 97 | self.input_proto.build_targets.add().name = 'bar' |
Shao-Chuan Lee | 85ba7ce | 2021-02-09 13:50:11 +0900 | [diff] [blame] | 98 | self.input_proto.skip_commit = True |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 99 | |
Alex Klein | 26e472b | 2020-03-10 14:35:01 -0600 | [diff] [blame] | 100 | self.build_targets = [build_target_lib.BuildTarget('foo'), |
| 101 | build_target_lib.BuildTarget('bar')] |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 102 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 103 | self.response = android_pb2.MarkStableResponse() |
| 104 | |
| 105 | def testValidateOnly(self): |
| 106 | """Sanity check that a validate only call does not execute any logic.""" |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 107 | android.MarkStable(self.input_proto, self.response, |
| 108 | self.validate_only_config) |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 109 | self.uprev.assert_not_called() |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 110 | |
Michael Mortensen | 2562644 | 2019-11-22 10:06:59 -0700 | [diff] [blame] | 111 | def testMockCall(self): |
| 112 | """Test that a mock call does not execute logic, returns mocked value.""" |
| 113 | android.MarkStable(self.input_proto, self.response, |
| 114 | self.mock_call_config) |
| 115 | self.uprev.assert_not_called() |
| 116 | self.assertEqual(self.response.status, |
| 117 | android_pb2.MARK_STABLE_STATUS_SUCCESS) |
| 118 | self.assertEqual(self.response.android_atom.category, 'category') |
| 119 | self.assertEqual(self.response.android_atom.package_name, |
| 120 | 'android-package-name') |
| 121 | self.assertEqual(self.response.android_atom.version, '1.2') |
| 122 | |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 123 | def testFailsIfPackageNameMissing(self): |
| 124 | """Fails if package_name is missing.""" |
| 125 | self.input_proto.package_name = '' |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 126 | with self.assertRaises(cros_build_lib.DieSystemExit): |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 127 | android.MarkStable(self.input_proto, self.response, self.api_config) |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 128 | self.uprev.assert_not_called() |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 129 | |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 130 | def testCallsCommandCorrectly(self): |
| 131 | """Test that commands.MarkAndroidAsStable is called correctly.""" |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 132 | self.uprev.return_value = 'cat/android-1.2.3' |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 133 | atom = common_pb2.PackageInfo() |
| 134 | atom.category = 'cat' |
| 135 | atom.package_name = 'android' |
| 136 | atom.version = '1.2.3' |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 137 | android.MarkStable(self.input_proto, self.response, self.api_config) |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 138 | self.uprev.assert_called_once_with( |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 139 | android_package=self.input_proto.package_name, |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 140 | chroot=mock.ANY, |
| 141 | build_targets=self.build_targets, |
Shao-Chuan Lee | a4b4f30 | 2021-05-12 14:40:20 +0900 | [diff] [blame] | 142 | android_build_branch=self.input_proto.android_build_branch, |
Shao-Chuan Lee | 85ba7ce | 2021-02-09 13:50:11 +0900 | [diff] [blame] | 143 | android_version=self.input_proto.android_version, |
| 144 | skip_commit=self.input_proto.skip_commit, |
| 145 | ) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 146 | self.assertEqual(self.response.android_atom, atom) |
| 147 | self.assertEqual(self.response.status, |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 148 | android_pb2.MARK_STABLE_STATUS_SUCCESS) |
| 149 | |
| 150 | def testHandlesEarlyExit(self): |
| 151 | """Test that early exit is handled correctly.""" |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 152 | self.uprev.return_value = '' |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 153 | android.MarkStable(self.input_proto, self.response, self.api_config) |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 154 | self.uprev.assert_called_once_with( |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 155 | android_package=self.input_proto.package_name, |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 156 | chroot=mock.ANY, |
| 157 | build_targets=self.build_targets, |
Shao-Chuan Lee | a4b4f30 | 2021-05-12 14:40:20 +0900 | [diff] [blame] | 158 | android_build_branch=self.input_proto.android_build_branch, |
Shao-Chuan Lee | 85ba7ce | 2021-02-09 13:50:11 +0900 | [diff] [blame] | 159 | android_version=self.input_proto.android_version, |
| 160 | skip_commit=self.input_proto.skip_commit, |
| 161 | ) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 162 | self.assertEqual(self.response.status, |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 163 | android_pb2.MARK_STABLE_STATUS_EARLY_EXIT) |
| 164 | |
| 165 | def testHandlesPinnedUprevError(self): |
| 166 | """Test that pinned error is handled correctly.""" |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 167 | self.uprev.side_effect = packages.AndroidIsPinnedUprevError('pin/xx-1.1') |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 168 | atom = common_pb2.PackageInfo() |
| 169 | atom.category = 'pin' |
| 170 | atom.package_name = 'xx' |
| 171 | atom.version = '1.1' |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 172 | android.MarkStable(self.input_proto, self.response, self.api_config) |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 173 | self.uprev.assert_called_once_with( |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 174 | android_package=self.input_proto.package_name, |
Alex Klein | 4de25e8 | 2019-08-05 15:58:39 -0600 | [diff] [blame] | 175 | chroot=mock.ANY, |
| 176 | build_targets=self.build_targets, |
Shao-Chuan Lee | a4b4f30 | 2021-05-12 14:40:20 +0900 | [diff] [blame] | 177 | android_build_branch=self.input_proto.android_build_branch, |
Shao-Chuan Lee | 85ba7ce | 2021-02-09 13:50:11 +0900 | [diff] [blame] | 178 | android_version=self.input_proto.android_version, |
| 179 | skip_commit=self.input_proto.skip_commit, |
| 180 | ) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 181 | self.assertEqual(self.response.android_atom, atom) |
| 182 | self.assertEqual(self.response.status, |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 183 | android_pb2.MARK_STABLE_STATUS_PINNED) |
| 184 | |
| 185 | |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 186 | class UnpinVersionTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin): |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 187 | """Unittests for UnpinVersion.""" |
| 188 | |
| 189 | def testCallsUnlink(self): |
| 190 | """SetAndroid calls service with correct args.""" |
| 191 | safeunlink = self.PatchObject(osutils, 'SafeUnlink') |
| 192 | self.PatchObject(constants, '_FindSourceRoot', return_value='SRCROOT') |
| 193 | |
| 194 | # This has the side effect of making sure that input and output proto are |
| 195 | # not actually used. |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 196 | android.UnpinVersion(None, None, self.api_config) |
LaMont Jones | 8a1644f | 2019-04-16 14:30:17 -0600 | [diff] [blame] | 197 | safeunlink.assert_called_once_with(android.ANDROIDPIN_MASK_PATH) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 198 | |
| 199 | def testValidateOnly(self): |
| 200 | """Sanity check that a validate only call does not execute any logic.""" |
| 201 | safeunlink = self.PatchObject(osutils, 'SafeUnlink') |
| 202 | |
| 203 | android.UnpinVersion(None, None, self.validate_only_config) |
| 204 | safeunlink.assert_not_called() |
Michael Mortensen | 2562644 | 2019-11-22 10:06:59 -0700 | [diff] [blame] | 205 | |
| 206 | def testMockCall(self): |
| 207 | """Test that a mock call does not execute logic.""" |
| 208 | safeunlink = self.PatchObject(osutils, 'SafeUnlink') |
| 209 | |
| 210 | android.UnpinVersion(None, None, self.mock_call_config) |
| 211 | safeunlink.assert_not_called() |
| 212 | # android.UnpinVersion does not modify the response. |