blob: 6ea3d3b96b4abb359b16ee3f8be6df46cb2b62ba [file] [log] [blame]
Kuang-che Wu6e4beca2018-06-27 17:45:02 +08001# -*- coding: utf-8 -*-
Kuang-che Wu88875db2017-07-20 10:47:53 +08002# Copyright 2017 The Chromium OS Authors. All rights reserved.
3# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
5"""Test cli module."""
6
7from __future__ import print_function
8import random
9import shutil
10import tempfile
11import unittest
12import StringIO
13
14import mock
15
16from bisect_kit import cli
Kuang-che Wue1b329e2018-03-31 11:39:33 +080017from bisect_kit import common
Kuang-che Wue80bb872018-11-15 19:45:25 +080018from bisect_kit import configure
Kuang-che Wu88875db2017-07-20 10:47:53 +080019from bisect_kit import core
Kuang-che Wue121fae2018-11-09 16:18:39 +080020from bisect_kit import errors
Kuang-che Wu88875db2017-07-20 10:47:53 +080021
22
23class TestCli(unittest.TestCase):
24 """Test functions in cli module."""
25
26 def test_argtype_int(self):
27 self.assertEqual(cli.argtype_int('123456'), '123456')
28 self.assertEqual(cli.argtype_int('-123456'), '-123456')
29 with self.assertRaises(cli.ArgTypeError):
30 cli.argtype_int('foobar')
31
32 def test_argtype_notempty(self):
33 self.assertEqual(cli.argtype_notempty('123456'), '123456')
34 with self.assertRaises(cli.ArgTypeError):
35 cli.argtype_notempty('')
36
Kuang-che Wu603cdad2019-01-18 21:32:55 +080037 def test_argtype_re(self):
38 argtype = cli.argtype_re(r'^r\d+$', 'r123')
39 self.assertEqual(argtype('r123'), 'r123')
40
41 with self.assertRaises(cli.ArgTypeError):
42 argtype('hello')
43
Kuang-che Wu88875db2017-07-20 10:47:53 +080044 def test_argtype_multiplexer(self):
Kuang-che Wu603cdad2019-01-18 21:32:55 +080045 argtype = cli.argtype_multiplexer(cli.argtype_int,
46 cli.argtype_re('foobar', 'foobar'),
47 cli.argtype_re(r'^r\d+$', 'r123'))
Kuang-che Wu88875db2017-07-20 10:47:53 +080048 self.assertEqual(argtype('123456'), '123456')
49 self.assertEqual(argtype('foobar'), 'foobar')
50 self.assertEqual(argtype('r123'), 'r123')
51
52 with self.assertRaises(cli.ArgTypeError):
53 argtype('hello')
54
55 def test_argtype_multiplier(self):
56 argtype = cli.argtype_multiplier(cli.argtype_int)
57 self.assertEqual(argtype('123'), ('123', 1))
58 self.assertEqual(argtype('123*5'), ('123', 5))
59
60 # Make sure there is multiplier example in the message.
61 with self.assertRaisesRegexp(cli.ArgTypeError, r'\d+\*\d+'):
62 argtype('hello')
63
64 def test_argtype_path(self):
65 self.assertEqual(cli.argtype_dir_path('/'), '/')
66 self.assertEqual(cli.argtype_dir_path('/etc/'), '/etc')
67
68 with self.assertRaises(cli.ArgTypeError):
69 cli.argtype_dir_path('')
70 with self.assertRaises(cli.ArgTypeError):
71 cli.argtype_dir_path('/foo/bar/not/exist/path')
72 with self.assertRaises(cli.ArgTypeError):
73 cli.argtype_dir_path('/etc/passwd')
74
75 def test_collect_bisect_result_values(self):
76 # pylint: disable=protected-access
77 values = []
78 cli._collect_bisect_result_values(values, '')
79 self.assertEqual(values, [])
80
81 values = []
82 cli._collect_bisect_result_values(values, 'foo\n')
83 cli._collect_bisect_result_values(values, 'bar\n')
84 self.assertEqual(values, [])
85
86 values = []
87 cli._collect_bisect_result_values(values, 'BISECT_RESULT_VALUES=1 2\n')
88 cli._collect_bisect_result_values(values, 'fooBISECT_RESULT_VALUES=3 4\n')
89 cli._collect_bisect_result_values(values, 'BISECT_RESULT_VALUES=5\n')
90 self.assertEqual(values, [1, 2, 5])
91
Kuang-che Wue121fae2018-11-09 16:18:39 +080092 with self.assertRaises(errors.InternalError):
Kuang-che Wu88875db2017-07-20 10:47:53 +080093 cli._collect_bisect_result_values(values, 'BISECT_RESULT_VALUES=hello\n')
94
Kuang-che Wu88518882017-09-22 16:57:25 +080095 def test_check_executable(self):
96 self.assertEqual(cli.check_executable('/bin/true'), None)
97
98 self.assertRegexpMatches(cli.check_executable('/'), r'Not a file')
99 self.assertRegexpMatches(cli.check_executable('LICENSE'), r'chmod')
100 self.assertRegexpMatches(cli.check_executable('what'), r'PATH')
101 self.assertRegexpMatches(cli.check_executable('eval-manually.sh'), r'\./')
102
Kuang-che Wu88875db2017-07-20 10:47:53 +0800103
104class DummyDomain(core.BisectDomain):
105 """Dummy subclass of BisectDomain."""
106 revtype = staticmethod(cli.argtype_notempty)
107
108 @staticmethod
109 def add_init_arguments(parser):
110 parser.add_argument('--num', required=True, type=int)
111 parser.add_argument('--ans', type=int)
112 parser.add_argument('--old_p', type=float, default=0.0)
113 parser.add_argument('--new_p', type=float, default=1.0)
114
115 @staticmethod
116 def init(opts):
117 config = dict(
118 ans=opts.ans,
119 old_p=opts.old_p,
120 new_p=opts.new_p,
121 old=opts.old,
122 new=opts.new)
123
124 revlist = map(str, range(opts.num))
125 return config, revlist
126
127 def __init__(self, config):
128 self.config = config
129
130 def setenv(self, env, rev):
131 env['ANS'] = str(self.config['ans'])
132 env['OLD_P'] = str(self.config['old_p'])
133 env['NEW_P'] = str(self.config['new_p'])
134 env['REV'] = rev
135
136
137@mock.patch('bisect_kit.common.config_logging', mock.Mock())
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800138class TestBisectorCommandLine(unittest.TestCase):
139 """Test cli.BisectorCommandLine class."""
Kuang-che Wu88875db2017-07-20 10:47:53 +0800140
141 def setUp(self):
142 self.session_base = tempfile.mkdtemp()
Kuang-che Wu41e8b592018-09-25 17:01:30 +0800143 self.patcher = mock.patch.object(common, 'DEFAULT_SESSION_BASE',
Kuang-che Wu88875db2017-07-20 10:47:53 +0800144 self.session_base)
145 self.patcher.start()
146
147 def tearDown(self):
148 shutil.rmtree(self.session_base)
149 self.patcher.stop()
Kuang-che Wue1b329e2018-03-31 11:39:33 +0800150 configure.reset()
Kuang-che Wu88875db2017-07-20 10:47:53 +0800151
152 def test_run_true(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800153 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800154 bisector.main('init', '--num=10', '--old=0', '--new=9')
155 bisector.main('config', 'switch', 'true')
156 bisector.main('config', 'eval', 'true')
Kuang-che Wue121fae2018-11-09 16:18:39 +0800157 with self.assertRaises(errors.VerificationFailed):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800158 bisector.main('run')
159
Kuang-che Wu15874b62019-01-11 21:10:27 +0800160 result = bisector.current_status()
161 self.assertEqual(result.get('inited'), True)
162 self.assertEqual(result.get('verified'), False)
163
Kuang-che Wu88875db2017-07-20 10:47:53 +0800164 def test_run_false(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800165 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800166 bisector.main('init', '--num=10', '--old=0', '--new=9')
167 bisector.main('config', 'switch', 'true')
168 bisector.main('config', 'eval', 'false')
Kuang-che Wue121fae2018-11-09 16:18:39 +0800169 with self.assertRaises(errors.VerificationFailed):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800170 bisector.main('run')
171
Kuang-che Wu15874b62019-01-11 21:10:27 +0800172 result = bisector.current_status()
173 self.assertEqual(result.get('inited'), True)
174 self.assertEqual(result.get('verified'), False)
175
Kuang-che Wu88875db2017-07-20 10:47:53 +0800176 def test_simple(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800177 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800178 bisector.main('init', '--num=20', '--old=3', '--new=15')
179 bisector.main('config', 'switch', 'true')
180 bisector.main('config', 'eval', 'sh', '-c', '[ "$BISECT_REV" -lt 7 ]')
181 bisector.main('run')
182 self.assertEqual(bisector.strategy.get_best_guess(), 7)
183
Kuang-che Wu849da582018-09-06 18:11:52 +0800184 with mock.patch('sys.stdout', new_callable=StringIO.StringIO):
185 # Only make sure no exceptions. No output verification.
186 bisector.main('log')
187
Kuang-che Wu88875db2017-07-20 10:47:53 +0800188 def test_switch_fail(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800189 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800190 bisector.main('init', '--num=20', '--old=3', '--new=15')
191 bisector.main('config', 'switch', 'false')
192 bisector.main('config', 'eval', 'sh', '-c', '[ "$BISECT_REV" -lt 7 ]')
Kuang-che Wue121fae2018-11-09 16:18:39 +0800193 with self.assertRaises(errors.UnableToProceed):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800194 bisector.main('run')
195
Kuang-che Wu15874b62019-01-11 21:10:27 +0800196 result = bisector.current_status()
197 self.assertEqual(result.get('inited'), True)
198 self.assertEqual(result.get('verified'), False)
199
Kuang-che Wu88875db2017-07-20 10:47:53 +0800200 def test_run_classic(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800201 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800202 bisector.main('init', '--num=200', '--old=0', '--new=99')
203 bisector.main('config', 'switch', 'true')
204 bisector.main('config', 'eval', 'false')
205
206 def do_evaluate(_cmd, _domain, rev):
207 if int(rev) < 42:
208 return 'old', []
209 return 'new', []
210
211 with mock.patch('bisect_kit.cli.do_evaluate', do_evaluate):
212 # two verify
213 with mock.patch(
214 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
215 bisector.main('next')
216 self.assertEqual(mock_stdout.getvalue(), '99\n')
217 bisector.main('run', '-1')
218 with mock.patch(
219 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
220 bisector.main('next')
221 self.assertEqual(mock_stdout.getvalue(), '0\n')
222 bisector.main('run', '-1')
223 self.assertEqual(bisector.strategy.get_range(), (0, 99))
224
225 bisector.main('run', '-1')
226 self.assertEqual(bisector.strategy.get_range(), (0, 49))
227
228 bisector.main('run')
229 self.assertEqual(bisector.strategy.get_best_guess(), 42)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800230
231 with mock.patch(
232 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
233 bisector.main('next')
234 self.assertEqual(mock_stdout.getvalue(), 'done\n')
235
236 def test_run_noisy(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800237 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800238 bisector.main('init', '--num=100', '--old=0', '--new=99',
239 '--noisy=old=1/10,new=9/10')
240 bisector.main('config', 'switch', 'true')
241 bisector.main('config', 'eval', 'false')
242
243 def do_evaluate(_cmd, _domain, rev):
244 if int(rev) < 42:
245 p = 0.1
246 else:
247 p = 0.9
248 if random.random() < p:
249 return 'new', []
250 return 'old', []
251
252 with mock.patch('bisect_kit.cli.do_evaluate', do_evaluate):
253 bisector.main('run')
254 self.assertEqual(bisector.strategy.get_best_guess(), 42)
255
256 with mock.patch(
257 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
258 bisector.main('next')
259 # There might be small math error near the boundary of confidence.
260 self.assertIn(mock_stdout.getvalue(), ['done\n', '41\n', '42\n'])
261
262 def test_cmd_old_and_new(self):
263 """Tests cmd_old and cmd_new"""
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800264 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800265 bisector.main('init', '--num=100', '--old=0', '--new=99')
266 bisector.main('config', 'switch', 'true')
267 bisector.main('config', 'eval', 'false')
268 bisector.main('old', '0')
269 bisector.main('new', '99')
270 bisector.main('run', '-1')
271 self.assertEqual(bisector.strategy.get_range(), (0, 49))
272
273 bisector.main('old', '20')
274 bisector.main('new', '40')
275 bisector.main('run', '-1')
276 self.assertEqual(bisector.strategy.get_range(), (20, 30))
277
Kuang-che Wu978b65a2019-03-12 09:50:40 +0800278 with self.assertRaises(errors.UnableToProceed):
279 bisector.main('skip', '20*10')
280 with self.assertRaises(errors.UnableToProceed):
281 bisector.main('skip', '30*10')
Kuang-che Wu88875db2017-07-20 10:47:53 +0800282 self.assertEqual(bisector.strategy.get_range(), (20, 30))
283
284 def test_cmd_switch(self):
285 """Test cmd_switch"""
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800286 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800287 bisector.main('init', '--num=100', '--old=0', '--new=99')
288 bisector.main('config', 'switch', 'true')
289 bisector.main('config', 'eval', 'false')
290
291 switched = []
292
293 def do_switch(_cmd, _domain, rev):
294 switched.append(rev)
295
296 with mock.patch('bisect_kit.cli.do_switch', do_switch):
297 bisector.main('switch', 'next')
298 bisector.main('switch', '3')
299 bisector.main('switch', '5')
300 bisector.main('switch', '4')
301 self.assertEqual(switched, ['99', '3', '5', '4'])
302
303 def test_cmd_view(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800304 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800305 bisector.main('init', '--num=100', '--old=10', '--new=90')
Kuang-che Wue80bb872018-11-15 19:45:25 +0800306 with mock.patch.object(DummyDomain, 'fill_candidate_summary') as mock_view:
Kuang-che Wu88875db2017-07-20 10:47:53 +0800307 bisector.main('view')
Kuang-che Wue80bb872018-11-15 19:45:25 +0800308 mock_view.assert_called()
Kuang-che Wu88875db2017-07-20 10:47:53 +0800309
310 def test_cmd_config_confidence(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800311 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800312 bisector.main('init', '--num=100', '--old=10', '--new=90',
313 '--confidence=0.75')
314
Kuang-che Wue121fae2018-11-09 16:18:39 +0800315 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800316 bisector.main('config', 'confidence', 'foo')
Kuang-che Wue121fae2018-11-09 16:18:39 +0800317 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800318 bisector.main('config', 'confidence', '0.9', '0.8')
319
320 self.assertEqual(bisector.states.config['confidence'], 0.75)
321 bisector.main('config', 'confidence', '0.875')
322 self.assertEqual(bisector.states.config['confidence'], 0.875)
323
324 def test_cmd_config_noisy(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800325 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800326 bisector.main('init', '--num=100', '--old=10', '--new=90',
327 '--noisy=new=9/10')
328
Kuang-che Wue121fae2018-11-09 16:18:39 +0800329 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800330 bisector.main('config', 'noisy', 'hello', 'world')
331
332 self.assertEqual(bisector.states.config['noisy'], 'new=9/10')
333 bisector.main('config', 'noisy', 'old=1/10,new=8/9')
334 self.assertEqual(bisector.states.config['noisy'], 'old=1/10,new=8/9')
335
Kuang-che Wua8c987f2019-01-18 14:26:43 +0800336 with mock.patch('sys.stdout', new_callable=StringIO.StringIO):
337 # Only make sure no exceptions. No output verification.
338 bisector.main('view')
339
Kuang-che Wu88875db2017-07-20 10:47:53 +0800340 def test_current_status(self):
Kuang-che Wue1b329e2018-03-31 11:39:33 +0800341 common.init()
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800342 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800343 result = bisector.current_status()
344 self.assertEqual(result.get('inited'), False)
345
346 bisector.main('init', '--num=100', '--old=10', '--new=90',
347 '--noisy=new=9/10')
348 result = bisector.current_status()
349 self.assertEqual(result.get('inited'), True)
350
351
352if __name__ == '__main__':
353 unittest.main()