commit cce4f83d2ed1a6073275c0b7609935e1846ad89a
author Mike Frysinger <vapier@chromium.org> Thu Nov 14 22:44:38 2019 -0500
committer Commit Bot <commit-bot@chromium.org> Wed Nov 27 00:37:42 2019 +0000
tree 3d736b090caa935b3edb1d6184f35b23f428fbf8
parent 1f2e337bf280627efd664589bfc30b25aaaa1daa

deploy_unittest: fix tests under Python 3

We use & assume num_updates is a number everywhere, including 0.
The unittest sets it to None for some reason which doesn't work
as Python 3 doesn't allow comparing None to ints with < and >.

BUG=chromium:997354
TEST=`./run_tests` passes

Change-Id: I0dc4315f2f5baf459911c52d0fb54d182a36a49b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1935251
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

cli/deploy_unittest.py

diff --git a/cli/deploy_unittest.py b/cli/deploy_unittest.py
index 3dce908..fc7927b 100644
--- a/cli/deploy_unittest.py
+++ b/cli/deploy_unittest.py
@@ -115,7 +115,7 @@
     self.pkgs = packages
     self.cpvs = packages_cpvs or packages
     self.listed = []
-    self.num_updates = None
+    self.num_updates = 0
     self.pkgs_attrs = pkgs_attrs
 
   def Run(self, _device, _root, _packages, _update, _deep, _deep_rev):