Revert "AndroidService: Refactor uprev and unpin endpoints."
This reverts commit f878992e96bd45cd5c42fc55a19b7c338276aa8a.
Reason for revert: Causing failures.
Original change's description:
> AndroidService: Refactor uprev and unpin endpoints.
>
> The previous refactor caused errors, and was too out of date to
> reland. Reimplemented in new CL with fixes and updates.
>
> BUG=chromium:956244, chromium:982121
> TEST=run_tests
> TEST=android pfq tryjob
>
> Change-Id: I1dc2aacfa68112d3b190d90eeb91e2db5aeb8674
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1753024
> Tested-by: Alex Klein <saklein@chromium.org>
> Commit-Queue: Alex Klein <saklein@chromium.org>
> Reviewed-by: David Burger <dburger@chromium.org>
Bug: chromium:956244, chromium:982121
Change-Id: I6e0785606dbcdd575bda9633230a9e9aba5f3eb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1762278
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Tested-by: Alex Klein <saklein@chromium.org>
diff --git a/api/controller/android_unittest.py b/api/controller/android_unittest.py
index dee7b35..59f2342 100644
--- a/api/controller/android_unittest.py
+++ b/api/controller/android_unittest.py
@@ -7,80 +7,79 @@
from __future__ import print_function
-import mock
-
from chromite.api import api_config
from chromite.api.controller import android
from chromite.api.gen.chromite.api import android_pb2
from chromite.api.gen.chromiumos import common_pb2
+# TODO(crbug/904939): implement service/android.py
+from chromite.cbuildbot import commands
from chromite.lib import constants
from chromite.lib import cros_build_lib
from chromite.lib import cros_test_lib
from chromite.lib import osutils
-from chromite.lib.build_target_util import BuildTarget
-from chromite.service import packages
-
class MarkStableTest(cros_test_lib.MockTestCase, api_config.ApiConfigMixin):
"""Unittests for MarkStable."""
def setUp(self):
- self.uprev = self.PatchObject(packages, 'uprev_android')
+ self.command = self.PatchObject(commands, 'MarkAndroidAsStable')
self.input_proto = android_pb2.MarkStableRequest()
self.input_proto.tracking_branch = 'tracking-branch'
self.input_proto.package_name = 'android-package-name'
self.input_proto.android_build_branch = 'android_build_branch'
- self.input_proto.build_targets.add().name = 'foo'
- self.input_proto.build_targets.add().name = 'bar'
-
- self.build_targets = [BuildTarget('foo'), BuildTarget('bar')]
+ self.board1 = self.input_proto.boards.add()
+ self.board1.name = 'board1'
+ self.board2 = self.input_proto.boards.add()
+ self.board2.name = 'board2'
self.response = android_pb2.MarkStableResponse()
def testValidateOnly(self):
"""Sanity check that a validate only call does not execute any logic."""
+ patch = self.PatchObject(commands, 'MarkAndroidAsStable')
+
android.MarkStable(self.input_proto, self.response,
self.validate_only_config)
- self.uprev.assert_not_called()
+ patch.assert_not_called()
def testFailsIfTrackingBranchMissing(self):
"""Fails if tracking_branch is missing."""
self.input_proto.tracking_branch = ''
with self.assertRaises(cros_build_lib.DieSystemExit):
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_not_called()
+ self.command.assert_not_called()
def testFailsIfPackageNameMissing(self):
"""Fails if package_name is missing."""
self.input_proto.package_name = ''
with self.assertRaises(cros_build_lib.DieSystemExit):
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_not_called()
+ self.command.assert_not_called()
def testFailsIfAndroidBuildBranchMissing(self):
"""Fails if android_build_branch is missing."""
self.input_proto.android_build_branch = ''
with self.assertRaises(cros_build_lib.DieSystemExit):
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_not_called()
+ self.command.assert_not_called()
def testCallsCommandCorrectly(self):
"""Test that commands.MarkAndroidAsStable is called correctly."""
self.input_proto.android_version = 'android-version'
self.input_proto.android_gts_build_branch = 'gts-branch'
- self.uprev.return_value = 'cat/android-1.2.3'
+ self.command.return_value = 'cat/android-1.2.3'
atom = common_pb2.PackageInfo()
atom.category = 'cat'
atom.package_name = 'android'
atom.version = '1.2.3'
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_called_once_with(
+ self.command.assert_called_once_with(
+ buildroot=self.input_proto.buildroot,
tracking_branch=self.input_proto.tracking_branch,
android_package=self.input_proto.package_name,
android_build_branch=self.input_proto.android_build_branch,
- chroot=mock.ANY,
- build_targets=self.build_targets,
+ boards=self.input_proto.boards,
android_version=self.input_proto.android_version,
android_gts_build_branch=self.input_proto.android_gts_build_branch)
self.assertEqual(self.response.android_atom, atom)
@@ -91,14 +90,14 @@
"""Test that early exit is handled correctly."""
self.input_proto.android_version = 'android-version'
self.input_proto.android_gts_build_branch = 'gts-branch'
- self.uprev.return_value = ''
+ self.command.return_value = ''
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_called_once_with(
+ self.command.assert_called_once_with(
+ buildroot=self.input_proto.buildroot,
tracking_branch=self.input_proto.tracking_branch,
android_package=self.input_proto.package_name,
android_build_branch=self.input_proto.android_build_branch,
- chroot=mock.ANY,
- build_targets=self.build_targets,
+ boards=self.input_proto.boards,
android_version=self.input_proto.android_version,
android_gts_build_branch=self.input_proto.android_gts_build_branch)
self.assertEqual(self.response.status,
@@ -108,18 +107,18 @@
"""Test that pinned error is handled correctly."""
self.input_proto.android_version = 'android-version'
self.input_proto.android_gts_build_branch = 'gts-branch'
- self.uprev.side_effect = packages.AndroidIsPinnedUprevError('pin/xx-1.1')
+ self.command.side_effect = commands.AndroidIsPinnedUprevError('pin/xx-1.1')
atom = common_pb2.PackageInfo()
atom.category = 'pin'
atom.package_name = 'xx'
atom.version = '1.1'
android.MarkStable(self.input_proto, self.response, self.api_config)
- self.uprev.assert_called_once_with(
+ self.command.assert_called_once_with(
+ buildroot=self.input_proto.buildroot,
tracking_branch=self.input_proto.tracking_branch,
android_package=self.input_proto.package_name,
android_build_branch=self.input_proto.android_build_branch,
- chroot=mock.ANY,
- build_targets=self.build_targets,
+ boards=self.input_proto.boards,
android_version=self.input_proto.android_version,
android_gts_build_branch=self.input_proto.android_gts_build_branch)
self.assertEqual(self.response.android_atom, atom)