commit 978b65a5baf929aad758927fdc38be155aadf77e
author Kuang-che Wu <kcwu@chromium.org> Tue Mar 12 09:50:40 2019 +0800
committer chrome-bot <chrome-bot@chromium.org> Tue Mar 12 04:04:42 2019 -0700
tree 424e86d2286af1e1d6d69e27706610ca13ea3278
parent f3d03caac9faf66d0ec183c35da0564199b457e1

bisect-kit: handle too much skip properly

BUG=b:127452142
TEST=unittest

Change-Id: Id56c150204d3b3463fdf29917cb2bed4b0e9ee4e
Reviewed-on: https://chromium-review.googlesource.com/1516893
Commit-Ready: Kuang-che Wu <kcwu@chromium.org>
Tested-by: Kuang-che Wu <kcwu@chromium.org>
Reviewed-by: Chi-Ngai Wan <cnwan@google.com>

bisect_kit/cli_test.py

diff --git a/bisect_kit/cli_test.py b/bisect_kit/cli_test.py
index 1f630a4..f96cfd1 100644
--- a/bisect_kit/cli_test.py
+++ b/bisect_kit/cli_test.py
@@ -276,8 +276,10 @@
     bisector.main('run', '-1')
     self.assertEqual(bisector.strategy.get_range(), (20, 30))
 
-    bisector.main('skip', '20*10')
-    bisector.main('skip', '30*10')
+    with self.assertRaises(errors.UnableToProceed):
+      bisector.main('skip', '20*10')
+    with self.assertRaises(errors.UnableToProceed):
+      bisector.main('skip', '30*10')
     self.assertEqual(bisector.strategy.get_range(), (20, 30))
 
   def test_cmd_switch(self):