commit 256264484d75641272429b392b192ad0f54b29da
author Michael Mortensen <mmortensen@google.com> Fri Nov 22 10:06:59 2019 -0700
committer Commit Bot <commit-bot@chromium.org> Sat Nov 23 01:12:07 2019 +0000
tree e3ad244ce9f5ba4b07ba8f7ccd939eb5f0875a1a
parent 4bfedefbb37ad9671899e2613af7948e3aadf4fe

BuildAPI: Add unit tests and mocks for Android service.

BUG=chromium:1000844
TEST=run_tests

Change-Id: Id43111a4224e73ebd202aae687506bbfd9a94d25
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1930267
Reviewed-by: Alex Klein <saklein@chromium.org>
Tested-by: Michael Mortensen <mmortensen@google.com>
Commit-Queue: Michael Mortensen <mmortensen@google.com>

api/controller/android_unittest.py

diff --git a/api/controller/android_unittest.py b/api/controller/android_unittest.py
index dee7b35..4e5d65c 100644
--- a/api/controller/android_unittest.py
+++ b/api/controller/android_unittest.py
@@ -44,6 +44,18 @@
                        self.validate_only_config)
     self.uprev.assert_not_called()
 
+  def testMockCall(self):
+    """Test that a mock call does not execute logic, returns mocked value."""
+    android.MarkStable(self.input_proto, self.response,
+                       self.mock_call_config)
+    self.uprev.assert_not_called()
+    self.assertEqual(self.response.status,
+                     android_pb2.MARK_STABLE_STATUS_SUCCESS)
+    self.assertEqual(self.response.android_atom.category, 'category')
+    self.assertEqual(self.response.android_atom.package_name,
+                     'android-package-name')
+    self.assertEqual(self.response.android_atom.version, '1.2')
+
   def testFailsIfTrackingBranchMissing(self):
     """Fails if tracking_branch is missing."""
     self.input_proto.tracking_branch = ''
@@ -146,3 +158,11 @@
 
     android.UnpinVersion(None, None, self.validate_only_config)
     safeunlink.assert_not_called()
+
+  def testMockCall(self):
+    """Test that a mock call does not execute logic."""
+    safeunlink = self.PatchObject(osutils, 'SafeUnlink')
+
+    android.UnpinVersion(None, None, self.mock_call_config)
+    safeunlink.assert_not_called()
+    # android.UnpinVersion does not modify the response.