blob: 1f630a4ceae551092133da919e5f0bacb7a28967 [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)
230 self.assertEqual(bisector.states.stats['eval_count'], 2 + 7)
231
232 with mock.patch(
233 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
234 bisector.main('next')
235 self.assertEqual(mock_stdout.getvalue(), 'done\n')
236
237 def test_run_noisy(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800238 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800239 bisector.main('init', '--num=100', '--old=0', '--new=99',
240 '--noisy=old=1/10,new=9/10')
241 bisector.main('config', 'switch', 'true')
242 bisector.main('config', 'eval', 'false')
243
244 def do_evaluate(_cmd, _domain, rev):
245 if int(rev) < 42:
246 p = 0.1
247 else:
248 p = 0.9
249 if random.random() < p:
250 return 'new', []
251 return 'old', []
252
253 with mock.patch('bisect_kit.cli.do_evaluate', do_evaluate):
254 bisector.main('run')
255 self.assertEqual(bisector.strategy.get_best_guess(), 42)
256
257 with mock.patch(
258 'sys.stdout', new_callable=StringIO.StringIO) as mock_stdout:
259 bisector.main('next')
260 # There might be small math error near the boundary of confidence.
261 self.assertIn(mock_stdout.getvalue(), ['done\n', '41\n', '42\n'])
262
263 def test_cmd_old_and_new(self):
264 """Tests cmd_old and cmd_new"""
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800265 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800266 bisector.main('init', '--num=100', '--old=0', '--new=99')
267 bisector.main('config', 'switch', 'true')
268 bisector.main('config', 'eval', 'false')
269 bisector.main('old', '0')
270 bisector.main('new', '99')
271 bisector.main('run', '-1')
272 self.assertEqual(bisector.strategy.get_range(), (0, 49))
273
274 bisector.main('old', '20')
275 bisector.main('new', '40')
276 bisector.main('run', '-1')
277 self.assertEqual(bisector.strategy.get_range(), (20, 30))
278
279 bisector.main('skip', '20*10')
280 bisector.main('skip', '30*10')
281 self.assertEqual(bisector.strategy.get_range(), (20, 30))
282
283 def test_cmd_switch(self):
284 """Test cmd_switch"""
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800285 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800286 bisector.main('init', '--num=100', '--old=0', '--new=99')
287 bisector.main('config', 'switch', 'true')
288 bisector.main('config', 'eval', 'false')
289
290 switched = []
291
292 def do_switch(_cmd, _domain, rev):
293 switched.append(rev)
294
295 with mock.patch('bisect_kit.cli.do_switch', do_switch):
296 bisector.main('switch', 'next')
297 bisector.main('switch', '3')
298 bisector.main('switch', '5')
299 bisector.main('switch', '4')
300 self.assertEqual(switched, ['99', '3', '5', '4'])
301
302 def test_cmd_view(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800303 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800304 bisector.main('init', '--num=100', '--old=10', '--new=90')
Kuang-che Wue80bb872018-11-15 19:45:25 +0800305 with mock.patch.object(DummyDomain, 'fill_candidate_summary') as mock_view:
Kuang-che Wu88875db2017-07-20 10:47:53 +0800306 bisector.main('view')
Kuang-che Wue80bb872018-11-15 19:45:25 +0800307 mock_view.assert_called()
Kuang-che Wu88875db2017-07-20 10:47:53 +0800308
309 def test_cmd_config_confidence(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800310 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800311 bisector.main('init', '--num=100', '--old=10', '--new=90',
312 '--confidence=0.75')
313
Kuang-che Wue121fae2018-11-09 16:18:39 +0800314 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800315 bisector.main('config', 'confidence', 'foo')
Kuang-che Wue121fae2018-11-09 16:18:39 +0800316 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800317 bisector.main('config', 'confidence', '0.9', '0.8')
318
319 self.assertEqual(bisector.states.config['confidence'], 0.75)
320 bisector.main('config', 'confidence', '0.875')
321 self.assertEqual(bisector.states.config['confidence'], 0.875)
322
323 def test_cmd_config_noisy(self):
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800324 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800325 bisector.main('init', '--num=100', '--old=10', '--new=90',
326 '--noisy=new=9/10')
327
Kuang-che Wue121fae2018-11-09 16:18:39 +0800328 with self.assertRaises(errors.ArgumentError):
Kuang-che Wu88875db2017-07-20 10:47:53 +0800329 bisector.main('config', 'noisy', 'hello', 'world')
330
331 self.assertEqual(bisector.states.config['noisy'], 'new=9/10')
332 bisector.main('config', 'noisy', 'old=1/10,new=8/9')
333 self.assertEqual(bisector.states.config['noisy'], 'old=1/10,new=8/9')
334
Kuang-che Wua8c987f2019-01-18 14:26:43 +0800335 with mock.patch('sys.stdout', new_callable=StringIO.StringIO):
336 # Only make sure no exceptions. No output verification.
337 bisector.main('view')
338
Kuang-che Wu88875db2017-07-20 10:47:53 +0800339 def test_current_status(self):
Kuang-che Wue1b329e2018-03-31 11:39:33 +0800340 common.init()
Kuang-che Wu68db08a2018-03-30 11:50:34 +0800341 bisector = cli.BisectorCommandLine(DummyDomain)
Kuang-che Wu88875db2017-07-20 10:47:53 +0800342 result = bisector.current_status()
343 self.assertEqual(result.get('inited'), False)
344
345 bisector.main('init', '--num=100', '--old=10', '--new=90',
346 '--noisy=new=9/10')
347 result = bisector.current_status()
348 self.assertEqual(result.get('inited'), True)
349
350
351if __name__ == '__main__':
352 unittest.main()