mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-04 20:19:47 +08:00
97752c39bd
Rework kunit_tool in order to allow .kunitconfig files to better enforce that disabled items in .kunitconfig are disabled in the generated .config. Previously, kunit_tool simply enforced that any line present in .kunitconfig was also present in .config, but this could cause problems if a config option was disabled in .kunitconfig, but not listed in .config due to (for example) having disabled dependencies. To fix this, re-work the parser to track config names and values, and require values to match unless they are explicitly disabled with the "CONFIG_x is not set" comment (or by setting its value to 'n'). Those "disabled" values will pass validation if omitted from the .config, but not if they have a different value. Signed-off-by: David Gow <davidgow@google.com> Reviewed-by: Brendan Higgins <brendanhiggins@google.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
287 lines
10 KiB
Python
Executable File
287 lines
10 KiB
Python
Executable File
#!/usr/bin/python3
|
|
# SPDX-License-Identifier: GPL-2.0
|
|
#
|
|
# A collection of tests for tools/testing/kunit/kunit.py
|
|
#
|
|
# Copyright (C) 2019, Google LLC.
|
|
# Author: Brendan Higgins <brendanhiggins@google.com>
|
|
|
|
import unittest
|
|
from unittest import mock
|
|
|
|
import tempfile, shutil # Handling test_tmpdir
|
|
|
|
import os
|
|
|
|
import kunit_config
|
|
import kunit_parser
|
|
import kunit_kernel
|
|
import kunit
|
|
|
|
test_tmpdir = ''
|
|
|
|
def setUpModule():
|
|
global test_tmpdir
|
|
test_tmpdir = tempfile.mkdtemp()
|
|
|
|
def tearDownModule():
|
|
shutil.rmtree(test_tmpdir)
|
|
|
|
def get_absolute_path(path):
|
|
return os.path.join(os.path.dirname(__file__), path)
|
|
|
|
class KconfigTest(unittest.TestCase):
|
|
|
|
def test_is_subset_of(self):
|
|
kconfig0 = kunit_config.Kconfig()
|
|
self.assertTrue(kconfig0.is_subset_of(kconfig0))
|
|
|
|
kconfig1 = kunit_config.Kconfig()
|
|
kconfig1.add_entry(kunit_config.KconfigEntry('TEST', 'y'))
|
|
self.assertTrue(kconfig1.is_subset_of(kconfig1))
|
|
self.assertTrue(kconfig0.is_subset_of(kconfig1))
|
|
self.assertFalse(kconfig1.is_subset_of(kconfig0))
|
|
|
|
def test_read_from_file(self):
|
|
kconfig = kunit_config.Kconfig()
|
|
kconfig_path = get_absolute_path(
|
|
'test_data/test_read_from_file.kconfig')
|
|
|
|
kconfig.read_from_file(kconfig_path)
|
|
|
|
expected_kconfig = kunit_config.Kconfig()
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('UML', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('MMU', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('TEST', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('EXAMPLE_TEST', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('MK8', 'n'))
|
|
|
|
self.assertEqual(kconfig.entries(), expected_kconfig.entries())
|
|
|
|
def test_write_to_file(self):
|
|
kconfig_path = os.path.join(test_tmpdir, '.config')
|
|
|
|
expected_kconfig = kunit_config.Kconfig()
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('UML', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('MMU', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('TEST', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('EXAMPLE_TEST', 'y'))
|
|
expected_kconfig.add_entry(
|
|
kunit_config.KconfigEntry('MK8', 'n'))
|
|
|
|
expected_kconfig.write_to_file(kconfig_path)
|
|
|
|
actual_kconfig = kunit_config.Kconfig()
|
|
actual_kconfig.read_from_file(kconfig_path)
|
|
|
|
self.assertEqual(actual_kconfig.entries(),
|
|
expected_kconfig.entries())
|
|
|
|
class KUnitParserTest(unittest.TestCase):
|
|
|
|
def assertContains(self, needle, haystack):
|
|
for line in haystack:
|
|
if needle in line:
|
|
return
|
|
raise AssertionError('"' +
|
|
str(needle) + '" not found in "' + str(haystack) + '"!')
|
|
|
|
def test_output_isolated_correctly(self):
|
|
log_path = get_absolute_path(
|
|
'test_data/test_output_isolated_correctly.log')
|
|
file = open(log_path)
|
|
result = kunit_parser.isolate_kunit_output(file.readlines())
|
|
self.assertContains('TAP version 14\n', result)
|
|
self.assertContains(' # Subtest: example', result)
|
|
self.assertContains(' 1..2', result)
|
|
self.assertContains(' ok 1 - example_simple_test', result)
|
|
self.assertContains(' ok 2 - example_mock_test', result)
|
|
self.assertContains('ok 1 - example', result)
|
|
file.close()
|
|
|
|
def test_output_with_prefix_isolated_correctly(self):
|
|
log_path = get_absolute_path(
|
|
'test_data/test_pound_sign.log')
|
|
with open(log_path) as file:
|
|
result = kunit_parser.isolate_kunit_output(file.readlines())
|
|
self.assertContains('TAP version 14\n', result)
|
|
self.assertContains(' # Subtest: kunit-resource-test', result)
|
|
self.assertContains(' 1..5', result)
|
|
self.assertContains(' ok 1 - kunit_resource_test_init_resources', result)
|
|
self.assertContains(' ok 2 - kunit_resource_test_alloc_resource', result)
|
|
self.assertContains(' ok 3 - kunit_resource_test_destroy_resource', result)
|
|
self.assertContains(' foo bar #', result)
|
|
self.assertContains(' ok 4 - kunit_resource_test_cleanup_resources', result)
|
|
self.assertContains(' ok 5 - kunit_resource_test_proper_free_ordering', result)
|
|
self.assertContains('ok 1 - kunit-resource-test', result)
|
|
self.assertContains(' foo bar # non-kunit output', result)
|
|
self.assertContains(' # Subtest: kunit-try-catch-test', result)
|
|
self.assertContains(' 1..2', result)
|
|
self.assertContains(' ok 1 - kunit_test_try_catch_successful_try_no_catch',
|
|
result)
|
|
self.assertContains(' ok 2 - kunit_test_try_catch_unsuccessful_try_does_catch',
|
|
result)
|
|
self.assertContains('ok 2 - kunit-try-catch-test', result)
|
|
self.assertContains(' # Subtest: string-stream-test', result)
|
|
self.assertContains(' 1..3', result)
|
|
self.assertContains(' ok 1 - string_stream_test_empty_on_creation', result)
|
|
self.assertContains(' ok 2 - string_stream_test_not_empty_after_add', result)
|
|
self.assertContains(' ok 3 - string_stream_test_get_string', result)
|
|
self.assertContains('ok 3 - string-stream-test', result)
|
|
|
|
def test_parse_successful_test_log(self):
|
|
all_passed_log = get_absolute_path(
|
|
'test_data/test_is_test_passed-all_passed.log')
|
|
file = open(all_passed_log)
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual(
|
|
kunit_parser.TestStatus.SUCCESS,
|
|
result.status)
|
|
file.close()
|
|
|
|
def test_parse_failed_test_log(self):
|
|
failed_log = get_absolute_path(
|
|
'test_data/test_is_test_passed-failure.log')
|
|
file = open(failed_log)
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual(
|
|
kunit_parser.TestStatus.FAILURE,
|
|
result.status)
|
|
file.close()
|
|
|
|
def test_no_tests(self):
|
|
empty_log = get_absolute_path(
|
|
'test_data/test_is_test_passed-no_tests_run.log')
|
|
file = open(empty_log)
|
|
result = kunit_parser.parse_run_tests(
|
|
kunit_parser.isolate_kunit_output(file.readlines()))
|
|
self.assertEqual(0, len(result.suites))
|
|
self.assertEqual(
|
|
kunit_parser.TestStatus.NO_TESTS,
|
|
result.status)
|
|
file.close()
|
|
|
|
def test_crashed_test(self):
|
|
crashed_log = get_absolute_path(
|
|
'test_data/test_is_test_passed-crash.log')
|
|
file = open(crashed_log)
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual(
|
|
kunit_parser.TestStatus.TEST_CRASHED,
|
|
result.status)
|
|
file.close()
|
|
|
|
def test_ignores_prefix_printk_time(self):
|
|
prefix_log = get_absolute_path(
|
|
'test_data/test_config_printk_time.log')
|
|
with open(prefix_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
def test_ignores_multiple_prefixes(self):
|
|
prefix_log = get_absolute_path(
|
|
'test_data/test_multiple_prefixes.log')
|
|
with open(prefix_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
def test_prefix_mixed_kernel_output(self):
|
|
mixed_prefix_log = get_absolute_path(
|
|
'test_data/test_interrupted_tap_output.log')
|
|
with open(mixed_prefix_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
def test_prefix_poundsign(self):
|
|
pound_log = get_absolute_path('test_data/test_pound_sign.log')
|
|
with open(pound_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
def test_kernel_panic_end(self):
|
|
panic_log = get_absolute_path('test_data/test_kernel_panic_interrupt.log')
|
|
with open(panic_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
def test_pound_no_prefix(self):
|
|
pound_log = get_absolute_path('test_data/test_pound_no_prefix.log')
|
|
with open(pound_log) as file:
|
|
result = kunit_parser.parse_run_tests(file.readlines())
|
|
self.assertEqual('kunit-resource-test', result.suites[0].name)
|
|
|
|
class StrContains(str):
|
|
def __eq__(self, other):
|
|
return self in other
|
|
|
|
class KUnitMainTest(unittest.TestCase):
|
|
def setUp(self):
|
|
path = get_absolute_path('test_data/test_is_test_passed-all_passed.log')
|
|
file = open(path)
|
|
all_passed_log = file.readlines()
|
|
self.print_patch = mock.patch('builtins.print')
|
|
self.print_mock = self.print_patch.start()
|
|
self.linux_source_mock = mock.Mock()
|
|
self.linux_source_mock.build_reconfig = mock.Mock(return_value=True)
|
|
self.linux_source_mock.build_um_kernel = mock.Mock(return_value=True)
|
|
self.linux_source_mock.run_kernel = mock.Mock(return_value=all_passed_log)
|
|
|
|
def tearDown(self):
|
|
self.print_patch.stop()
|
|
pass
|
|
|
|
def test_run_passes_args_pass(self):
|
|
kunit.main(['run'], self.linux_source_mock)
|
|
assert self.linux_source_mock.build_reconfig.call_count == 1
|
|
assert self.linux_source_mock.run_kernel.call_count == 1
|
|
self.linux_source_mock.run_kernel.assert_called_once_with(
|
|
build_dir='', timeout=300)
|
|
self.print_mock.assert_any_call(StrContains('Testing complete.'))
|
|
|
|
def test_run_passes_args_fail(self):
|
|
self.linux_source_mock.run_kernel = mock.Mock(return_value=[])
|
|
with self.assertRaises(SystemExit) as e:
|
|
kunit.main(['run'], self.linux_source_mock)
|
|
assert type(e.exception) == SystemExit
|
|
assert e.exception.code == 1
|
|
assert self.linux_source_mock.build_reconfig.call_count == 1
|
|
assert self.linux_source_mock.run_kernel.call_count == 1
|
|
self.print_mock.assert_any_call(StrContains(' 0 tests run'))
|
|
|
|
def test_run_raw_output(self):
|
|
self.linux_source_mock.run_kernel = mock.Mock(return_value=[])
|
|
with self.assertRaises(SystemExit) as e:
|
|
kunit.main(['run', '--raw_output'], self.linux_source_mock)
|
|
assert type(e.exception) == SystemExit
|
|
assert e.exception.code == 1
|
|
assert self.linux_source_mock.build_reconfig.call_count == 1
|
|
assert self.linux_source_mock.run_kernel.call_count == 1
|
|
|
|
def test_run_timeout(self):
|
|
timeout = 3453
|
|
kunit.main(['run', '--timeout', str(timeout)], self.linux_source_mock)
|
|
assert self.linux_source_mock.build_reconfig.call_count == 1
|
|
self.linux_source_mock.run_kernel.assert_called_once_with(
|
|
build_dir='', timeout=timeout)
|
|
self.print_mock.assert_any_call(StrContains('Testing complete.'))
|
|
|
|
def test_run_builddir(self):
|
|
build_dir = '.kunit'
|
|
kunit.main(['run', '--build_dir', build_dir], self.linux_source_mock)
|
|
assert self.linux_source_mock.build_reconfig.call_count == 1
|
|
self.linux_source_mock.run_kernel.assert_called_once_with(
|
|
build_dir=build_dir, timeout=300)
|
|
self.print_mock.assert_any_call(StrContains('Testing complete.'))
|
|
|
|
if __name__ == '__main__':
|
|
unittest.main()
|