blob: c7cd4ab1b065fb5bf007ab23172de2a401472218 [file] [log] [blame]
Luca Farsidb136442024-03-26 10:55:21 -07001# Copyright 2024, The Android Open Source Project
2#
3# Licensed under the Apache License, Version 2.0 (the "License");
4# you may not use this file except in compliance with the License.
5# You may obtain a copy of the License at
6#
7# http://www.apache.org/licenses/LICENSE-2.0
8#
9# Unless required by applicable law or agreed to in writing, software
10# distributed under the License is distributed on an "AS IS" BASIS,
11# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12# See the License for the specific language governing permissions and
13# limitations under the License.
14
15"""Tests for build_test_suites.py"""
16
Luca Farsi040fabe2024-05-22 17:21:47 -070017import argparse
Luca Farsi70a53bd2024-08-07 17:29:16 -070018import functools
Luca Farsidb136442024-03-26 10:55:21 -070019from importlib import resources
Luca Farsi040fabe2024-05-22 17:21:47 -070020import json
Luca Farsidb136442024-03-26 10:55:21 -070021import multiprocessing
22import os
23import pathlib
24import shutil
25import signal
26import stat
27import subprocess
28import sys
29import tempfile
30import textwrap
31import time
32from typing import Callable
Luca Farsi040fabe2024-05-22 17:21:47 -070033import unittest
Luca Farsidb136442024-03-26 10:55:21 -070034from unittest import mock
Luca Farsib130e792024-08-22 12:04:41 -070035from build_context import BuildContext
Luca Farsidb136442024-03-26 10:55:21 -070036import build_test_suites
37import ci_test_lib
Luca Farsi040fabe2024-05-22 17:21:47 -070038import optimized_targets
Luca Farsidb136442024-03-26 10:55:21 -070039from pyfakefs import fake_filesystem_unittest
Luca Farsi7d859712024-11-06 16:09:16 -080040import metrics_agent
Luca Farsidb136442024-03-26 10:55:21 -070041
42
43class BuildTestSuitesTest(fake_filesystem_unittest.TestCase):
44
45 def setUp(self):
46 self.setUpPyfakefs()
47
48 os_environ_patcher = mock.patch.dict('os.environ', {})
49 self.addCleanup(os_environ_patcher.stop)
50 self.mock_os_environ = os_environ_patcher.start()
51
52 subprocess_run_patcher = mock.patch('subprocess.run')
53 self.addCleanup(subprocess_run_patcher.stop)
54 self.mock_subprocess_run = subprocess_run_patcher.start()
55
Luca Farsi7d859712024-11-06 16:09:16 -080056 metrics_agent_finalize_patcher = mock.patch('metrics_agent.MetricsAgent.end_reporting')
57 self.addCleanup(metrics_agent_finalize_patcher.stop)
58 self.mock_metrics_agent_end = metrics_agent_finalize_patcher.start()
59
Luca Farsidb136442024-03-26 10:55:21 -070060 self._setup_working_build_env()
61
62 def test_missing_target_release_env_var_raises(self):
63 del os.environ['TARGET_RELEASE']
64
65 with self.assert_raises_word(build_test_suites.Error, 'TARGET_RELEASE'):
66 build_test_suites.main([])
67
68 def test_missing_target_product_env_var_raises(self):
69 del os.environ['TARGET_PRODUCT']
70
71 with self.assert_raises_word(build_test_suites.Error, 'TARGET_PRODUCT'):
72 build_test_suites.main([])
73
74 def test_missing_top_env_var_raises(self):
75 del os.environ['TOP']
76
77 with self.assert_raises_word(build_test_suites.Error, 'TOP'):
78 build_test_suites.main([])
79
80 def test_invalid_arg_raises(self):
81 invalid_args = ['--invalid_arg']
82
83 with self.assertRaisesRegex(SystemExit, '2'):
84 build_test_suites.main(invalid_args)
85
86 def test_build_failure_returns(self):
87 self.mock_subprocess_run.side_effect = subprocess.CalledProcessError(
88 42, None
89 )
90
91 with self.assertRaisesRegex(SystemExit, '42'):
92 build_test_suites.main([])
93
Luca Farsi040fabe2024-05-22 17:21:47 -070094 def test_incorrectly_formatted_build_context_raises(self):
95 build_context = self.fake_top.joinpath('build_context')
96 build_context.touch()
97 os.environ['BUILD_CONTEXT'] = str(build_context)
98
99 with self.assert_raises_word(build_test_suites.Error, 'JSON'):
100 build_test_suites.main([])
101
Luca Farsidb136442024-03-26 10:55:21 -0700102 def test_build_success_returns(self):
103 with self.assertRaisesRegex(SystemExit, '0'):
104 build_test_suites.main([])
105
106 def assert_raises_word(self, cls, word):
Luca Farsi040fabe2024-05-22 17:21:47 -0700107 return self.assertRaisesRegex(cls, rf'\b{word}\b')
Luca Farsidb136442024-03-26 10:55:21 -0700108
109 def _setup_working_build_env(self):
110 self.fake_top = pathlib.Path('/fake/top')
111 self.fake_top.mkdir(parents=True)
112
113 self.soong_ui_dir = self.fake_top.joinpath('build/soong')
114 self.soong_ui_dir.mkdir(parents=True, exist_ok=True)
115
116 self.soong_ui = self.soong_ui_dir.joinpath('soong_ui.bash')
117 self.soong_ui.touch()
118
119 self.mock_os_environ.update({
120 'TARGET_RELEASE': 'release',
121 'TARGET_PRODUCT': 'product',
122 'TOP': str(self.fake_top),
123 })
124
125 self.mock_subprocess_run.return_value = 0
126
127
128class RunCommandIntegrationTest(ci_test_lib.TestCase):
129
130 def setUp(self):
131 self.temp_dir = ci_test_lib.TestTemporaryDirectory.create(self)
132
133 # Copy the Python executable from 'non-code' resources and make it
134 # executable for use by tests that launch a subprocess. Note that we don't
135 # use Python's native `sys.executable` property since that is not set when
136 # running via the embedded launcher.
137 base_name = 'py3-cmd'
138 dest_file = self.temp_dir.joinpath(base_name)
139 with resources.as_file(
140 resources.files('testdata').joinpath(base_name)
141 ) as p:
142 shutil.copy(p, dest_file)
143 dest_file.chmod(dest_file.stat().st_mode | stat.S_IEXEC)
144 self.python_executable = dest_file
145
146 self._managed_processes = []
147
148 def tearDown(self):
149 self._terminate_managed_processes()
150
151 def test_raises_on_nonzero_exit(self):
152 with self.assertRaises(Exception):
153 build_test_suites.run_command([
154 self.python_executable,
155 '-c',
156 textwrap.dedent(f"""\
157 import sys
158 sys.exit(1)
159 """),
160 ])
161
162 def test_streams_stdout(self):
163
164 def run_slow_command(stdout_file, marker):
165 with open(stdout_file, 'w') as f:
166 build_test_suites.run_command(
167 [
168 self.python_executable,
169 '-c',
170 textwrap.dedent(f"""\
171 import time
172
173 print('{marker}', end='', flush=True)
174
175 # Keep process alive until we check stdout.
176 time.sleep(10)
177 """),
178 ],
179 stdout=f,
180 )
181
182 marker = 'Spinach'
183 stdout_file = self.temp_dir.joinpath('stdout.txt')
184
185 p = self.start_process(target=run_slow_command, args=[stdout_file, marker])
186
187 self.assert_file_eventually_contains(stdout_file, marker)
188
189 def test_propagates_interruptions(self):
190
191 def run(pid_file):
192 build_test_suites.run_command([
193 self.python_executable,
194 '-c',
195 textwrap.dedent(f"""\
196 import os
197 import pathlib
198 import time
199
200 pathlib.Path('{pid_file}').write_text(str(os.getpid()))
201
202 # Keep the process alive for us to explicitly interrupt it.
203 time.sleep(10)
204 """),
205 ])
206
207 pid_file = self.temp_dir.joinpath('pid.txt')
208 p = self.start_process(target=run, args=[pid_file])
209 subprocess_pid = int(read_eventual_file_contents(pid_file))
210
211 os.kill(p.pid, signal.SIGINT)
212 p.join()
213
214 self.assert_process_eventually_dies(p.pid)
215 self.assert_process_eventually_dies(subprocess_pid)
216
217 def start_process(self, *args, **kwargs) -> multiprocessing.Process:
218 p = multiprocessing.Process(*args, **kwargs)
219 self._managed_processes.append(p)
220 p.start()
221 return p
222
223 def assert_process_eventually_dies(self, pid: int):
224 try:
225 wait_until(lambda: not ci_test_lib.process_alive(pid))
226 except TimeoutError as e:
227 self.fail(f'Process {pid} did not die after a while: {e}')
228
229 def assert_file_eventually_contains(self, file: pathlib.Path, substring: str):
230 wait_until(lambda: file.is_file() and file.stat().st_size > 0)
231 self.assertIn(substring, read_file_contents(file))
232
233 def _terminate_managed_processes(self):
234 for p in self._managed_processes:
235 if not p.is_alive():
236 continue
237
238 # We terminate the process with `SIGINT` since using `terminate` or
239 # `SIGKILL` doesn't kill any grandchild processes and we don't have
240 # `psutil` available to easily query all children.
241 os.kill(p.pid, signal.SIGINT)
242
243
Luca Farsi040fabe2024-05-22 17:21:47 -0700244class BuildPlannerTest(unittest.TestCase):
245
246 class TestOptimizedBuildTarget(optimized_targets.OptimizedBuildTarget):
247
Luca Farsi70a53bd2024-08-07 17:29:16 -0700248 def __init__(
Luca Farsid4e4b642024-09-10 16:37:51 -0700249 self, target, build_context, args, output_targets, packaging_commands
Luca Farsi70a53bd2024-08-07 17:29:16 -0700250 ):
251 super().__init__(target, build_context, args)
Luca Farsi040fabe2024-05-22 17:21:47 -0700252 self.output_targets = output_targets
Luca Farsid4e4b642024-09-10 16:37:51 -0700253 self.packaging_commands = packaging_commands
Luca Farsi040fabe2024-05-22 17:21:47 -0700254
Luca Farsi70a53bd2024-08-07 17:29:16 -0700255 def get_build_targets_impl(self):
Luca Farsi040fabe2024-05-22 17:21:47 -0700256 return self.output_targets
257
Luca Farsid4e4b642024-09-10 16:37:51 -0700258 def get_package_outputs_commands_impl(self):
259 return self.packaging_commands
Luca Farsi70a53bd2024-08-07 17:29:16 -0700260
261 def get_enabled_flag(self):
262 return f'{self.target}_enabled'
Luca Farsi040fabe2024-05-22 17:21:47 -0700263
264 def test_build_optimization_off_builds_everything(self):
265 build_targets = {'target_1', 'target_2'}
266 build_planner = self.create_build_planner(
267 build_context=self.create_build_context(optimized_build_enabled=False),
268 build_targets=build_targets,
269 )
270
271 build_plan = build_planner.create_build_plan()
272
273 self.assertSetEqual(build_targets, build_plan.build_targets)
274
275 def test_build_optimization_off_doesnt_package(self):
276 build_targets = {'target_1', 'target_2'}
277 build_planner = self.create_build_planner(
278 build_context=self.create_build_context(optimized_build_enabled=False),
279 build_targets=build_targets,
280 )
281
282 build_plan = build_planner.create_build_plan()
283
Luca Farsi8ea67422024-09-17 15:48:11 -0700284 for packaging_command in self.run_packaging_commands(build_plan):
285 self.assertEqual(len(packaging_command), 0)
Luca Farsi040fabe2024-05-22 17:21:47 -0700286
287 def test_build_optimization_on_optimizes_target(self):
288 build_targets = {'target_1', 'target_2'}
289 build_planner = self.create_build_planner(
290 build_targets=build_targets,
291 build_context=self.create_build_context(
Luca Farsib130e792024-08-22 12:04:41 -0700292 enabled_build_features=[{'name': self.get_target_flag('target_1')}]
Luca Farsi040fabe2024-05-22 17:21:47 -0700293 ),
294 )
295
296 build_plan = build_planner.create_build_plan()
297
298 expected_targets = {self.get_optimized_target_name('target_1'), 'target_2'}
299 self.assertSetEqual(expected_targets, build_plan.build_targets)
300
301 def test_build_optimization_on_packages_target(self):
302 build_targets = {'target_1', 'target_2'}
Luca Farsid4e4b642024-09-10 16:37:51 -0700303 optimized_target_name = self.get_optimized_target_name('target_1')
304 packaging_commands = [[f'packaging {optimized_target_name}']]
Luca Farsi040fabe2024-05-22 17:21:47 -0700305 build_planner = self.create_build_planner(
306 build_targets=build_targets,
307 build_context=self.create_build_context(
Luca Farsib130e792024-08-22 12:04:41 -0700308 enabled_build_features=[{'name': self.get_target_flag('target_1')}]
Luca Farsi040fabe2024-05-22 17:21:47 -0700309 ),
Luca Farsid4e4b642024-09-10 16:37:51 -0700310 packaging_commands=packaging_commands,
Luca Farsi040fabe2024-05-22 17:21:47 -0700311 )
312
313 build_plan = build_planner.create_build_plan()
314
Luca Farsi8ea67422024-09-17 15:48:11 -0700315 self.assertIn(packaging_commands, self.run_packaging_commands(build_plan))
Luca Farsi040fabe2024-05-22 17:21:47 -0700316
317 def test_individual_build_optimization_off_doesnt_optimize(self):
318 build_targets = {'target_1', 'target_2'}
319 build_planner = self.create_build_planner(
320 build_targets=build_targets,
321 )
322
323 build_plan = build_planner.create_build_plan()
324
325 self.assertSetEqual(build_targets, build_plan.build_targets)
326
327 def test_individual_build_optimization_off_doesnt_package(self):
328 build_targets = {'target_1', 'target_2'}
Luca Farsid4e4b642024-09-10 16:37:51 -0700329 packaging_commands = [['packaging command']]
Luca Farsi040fabe2024-05-22 17:21:47 -0700330 build_planner = self.create_build_planner(
331 build_targets=build_targets,
Luca Farsid4e4b642024-09-10 16:37:51 -0700332 packaging_commands=packaging_commands,
Luca Farsi040fabe2024-05-22 17:21:47 -0700333 )
334
335 build_plan = build_planner.create_build_plan()
336
Luca Farsi8ea67422024-09-17 15:48:11 -0700337 for packaging_command in self.run_packaging_commands(build_plan):
338 self.assertEqual(len(packaging_command), 0)
Luca Farsi040fabe2024-05-22 17:21:47 -0700339
Luca Farsib24c1c32024-08-01 14:47:10 -0700340 def test_target_output_used_target_built(self):
341 build_target = 'test_target'
342 build_planner = self.create_build_planner(
343 build_targets={build_target},
344 build_context=self.create_build_context(
345 test_context=self.get_test_context(build_target),
Luca Farsib130e792024-08-22 12:04:41 -0700346 enabled_build_features=[{'name': 'test_target_unused_exclusion'}],
Luca Farsib24c1c32024-08-01 14:47:10 -0700347 ),
348 )
349
350 build_plan = build_planner.create_build_plan()
351
352 self.assertSetEqual(build_plan.build_targets, {build_target})
353
354 def test_target_regex_used_target_built(self):
355 build_target = 'test_target'
356 test_context = self.get_test_context(build_target)
357 test_context['testInfos'][0]['extraOptions'] = [{
358 'key': 'additional-files-filter',
359 'values': [f'.*{build_target}.*\.zip'],
360 }]
361 build_planner = self.create_build_planner(
362 build_targets={build_target},
363 build_context=self.create_build_context(
364 test_context=test_context,
Luca Farsib130e792024-08-22 12:04:41 -0700365 enabled_build_features=[{'name': 'test_target_unused_exclusion'}],
Luca Farsib24c1c32024-08-01 14:47:10 -0700366 ),
367 )
368
369 build_plan = build_planner.create_build_plan()
370
371 self.assertSetEqual(build_plan.build_targets, {build_target})
372
373 def test_target_output_not_used_target_not_built(self):
374 build_target = 'test_target'
375 test_context = self.get_test_context(build_target)
376 test_context['testInfos'][0]['extraOptions'] = []
377 build_planner = self.create_build_planner(
378 build_targets={build_target},
379 build_context=self.create_build_context(
380 test_context=test_context,
Luca Farsib130e792024-08-22 12:04:41 -0700381 enabled_build_features=[{'name': 'test_target_unused_exclusion'}],
Luca Farsib24c1c32024-08-01 14:47:10 -0700382 ),
383 )
384
385 build_plan = build_planner.create_build_plan()
386
387 self.assertSetEqual(build_plan.build_targets, set())
388
Luca Farsi99d40d72024-08-06 13:35:28 -0700389 def test_target_regex_matching_not_too_broad(self):
390 build_target = 'test_target'
391 test_context = self.get_test_context(build_target)
392 test_context['testInfos'][0]['extraOptions'] = [{
393 'key': 'additional-files-filter',
394 'values': [f'.*a{build_target}.*\.zip'],
395 }]
396 build_planner = self.create_build_planner(
397 build_targets={build_target},
398 build_context=self.create_build_context(
399 test_context=test_context,
Luca Farsib130e792024-08-22 12:04:41 -0700400 enabled_build_features=[{'name': 'test_target_unused_exclusion'}],
Luca Farsi99d40d72024-08-06 13:35:28 -0700401 ),
402 )
403
404 build_plan = build_planner.create_build_plan()
405
406 self.assertSetEqual(build_plan.build_targets, set())
407
Luca Farsi040fabe2024-05-22 17:21:47 -0700408 def create_build_planner(
409 self,
410 build_targets: set[str],
Luca Farsib130e792024-08-22 12:04:41 -0700411 build_context: BuildContext = None,
Luca Farsi040fabe2024-05-22 17:21:47 -0700412 args: argparse.Namespace = None,
413 target_optimizations: dict[
414 str, optimized_targets.OptimizedBuildTarget
415 ] = None,
Luca Farsid4e4b642024-09-10 16:37:51 -0700416 packaging_commands: list[list[str]] = [],
Luca Farsi040fabe2024-05-22 17:21:47 -0700417 ) -> build_test_suites.BuildPlanner:
418 if not build_context:
419 build_context = self.create_build_context()
420 if not args:
421 args = self.create_args(extra_build_targets=build_targets)
422 if not target_optimizations:
423 target_optimizations = self.create_target_optimizations(
Luca Farsi70a53bd2024-08-07 17:29:16 -0700424 build_context,
425 build_targets,
Luca Farsid4e4b642024-09-10 16:37:51 -0700426 packaging_commands,
Luca Farsi040fabe2024-05-22 17:21:47 -0700427 )
428 return build_test_suites.BuildPlanner(
429 build_context, args, target_optimizations
430 )
431
432 def create_build_context(
433 self,
434 optimized_build_enabled: bool = True,
Luca Farsib130e792024-08-22 12:04:41 -0700435 enabled_build_features: list[dict[str, str]] = [],
Luca Farsi040fabe2024-05-22 17:21:47 -0700436 test_context: dict[str, any] = {},
Luca Farsib130e792024-08-22 12:04:41 -0700437 ) -> BuildContext:
438 build_context_dict = {}
439 build_context_dict['enabledBuildFeatures'] = enabled_build_features
Luca Farsi040fabe2024-05-22 17:21:47 -0700440 if optimized_build_enabled:
Luca Farsib130e792024-08-22 12:04:41 -0700441 build_context_dict['enabledBuildFeatures'].append(
442 {'name': 'optimized_build'}
443 )
444 build_context_dict['testContext'] = test_context
445 return BuildContext(build_context_dict)
Luca Farsi040fabe2024-05-22 17:21:47 -0700446
447 def create_args(
448 self, extra_build_targets: set[str] = set()
449 ) -> argparse.Namespace:
450 parser = argparse.ArgumentParser()
451 parser.add_argument('extra_targets', nargs='*')
452 return parser.parse_args(extra_build_targets)
453
454 def create_target_optimizations(
Luca Farsi70a53bd2024-08-07 17:29:16 -0700455 self,
Luca Farsib130e792024-08-22 12:04:41 -0700456 build_context: BuildContext,
Luca Farsi70a53bd2024-08-07 17:29:16 -0700457 build_targets: set[str],
Luca Farsid4e4b642024-09-10 16:37:51 -0700458 packaging_commands: list[list[str]] = [],
Luca Farsi040fabe2024-05-22 17:21:47 -0700459 ):
460 target_optimizations = dict()
461 for target in build_targets:
Luca Farsi70a53bd2024-08-07 17:29:16 -0700462 target_optimizations[target] = functools.partial(
463 self.TestOptimizedBuildTarget,
464 output_targets={self.get_optimized_target_name(target)},
Luca Farsid4e4b642024-09-10 16:37:51 -0700465 packaging_commands=packaging_commands,
Luca Farsi040fabe2024-05-22 17:21:47 -0700466 )
467
468 return target_optimizations
469
470 def get_target_flag(self, target: str):
471 return f'{target}_enabled'
472
473 def get_optimized_target_name(self, target: str):
474 return f'{target}_optimized'
475
Luca Farsib24c1c32024-08-01 14:47:10 -0700476 def get_test_context(self, target: str):
477 return {
478 'testInfos': [
479 {
480 'name': 'atp_test',
481 'target': 'test_target',
482 'branch': 'branch',
483 'extraOptions': [{
484 'key': 'additional-files-filter',
485 'values': [f'{target}.zip'],
486 }],
487 'command': '/tf/command',
488 'extraBuildTargets': [
489 'extra_build_target',
490 ],
491 },
492 ],
493 }
494
Luca Farsi8ea67422024-09-17 15:48:11 -0700495 def run_packaging_commands(self, build_plan: build_test_suites.BuildPlan):
496 return [
497 packaging_command_getter()
498 for packaging_command_getter in build_plan.packaging_commands_getters
499 ]
500
Luca Farsi040fabe2024-05-22 17:21:47 -0700501
Luca Farsidb136442024-03-26 10:55:21 -0700502def wait_until(
503 condition_function: Callable[[], bool],
504 timeout_secs: float = 3.0,
505 polling_interval_secs: float = 0.1,
506):
507 """Waits until a condition function returns True."""
508
509 start_time_secs = time.time()
510
511 while not condition_function():
512 if time.time() - start_time_secs > timeout_secs:
513 raise TimeoutError(
514 f'Condition not met within timeout: {timeout_secs} seconds'
515 )
516
517 time.sleep(polling_interval_secs)
518
519
520def read_file_contents(file: pathlib.Path) -> str:
521 with open(file, 'r') as f:
522 return f.read()
523
524
525def read_eventual_file_contents(file: pathlib.Path) -> str:
526 wait_until(lambda: file.is_file() and file.stat().st_size > 0)
527 return read_file_contents(file)
528
529
530if __name__ == '__main__':
531 ci_test_lib.main()