Finn Behrens | c25ce58 | 2020-11-23 15:15:33 +0100 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 2 | # SPDX-License-Identifier: GPL-2.0 |
| 3 | # |
| 4 | # A collection of tests for tools/testing/kunit/kunit.py |
| 5 | # |
| 6 | # Copyright (C) 2019, Google LLC. |
| 7 | # Author: Brendan Higgins <brendanhiggins@google.com> |
| 8 | |
| 9 | import unittest |
| 10 | from unittest import mock |
| 11 | |
| 12 | import tempfile, shutil # Handling test_tmpdir |
| 13 | |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 14 | import itertools |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 15 | import json |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 16 | import signal |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 17 | import os |
| 18 | |
| 19 | import kunit_config |
| 20 | import kunit_parser |
| 21 | import kunit_kernel |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 22 | import kunit_json |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 23 | import kunit |
| 24 | |
| 25 | test_tmpdir = '' |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 26 | abs_test_data_dir = '' |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 27 | |
| 28 | def setUpModule(): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 29 | global test_tmpdir, abs_test_data_dir |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 30 | test_tmpdir = tempfile.mkdtemp() |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 31 | abs_test_data_dir = os.path.abspath(os.path.join(os.path.dirname(__file__), 'test_data')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 32 | |
| 33 | def tearDownModule(): |
| 34 | shutil.rmtree(test_tmpdir) |
| 35 | |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 36 | def test_data_path(path): |
| 37 | return os.path.join(abs_test_data_dir, path) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 38 | |
| 39 | class KconfigTest(unittest.TestCase): |
| 40 | |
| 41 | def test_is_subset_of(self): |
| 42 | kconfig0 = kunit_config.Kconfig() |
| 43 | self.assertTrue(kconfig0.is_subset_of(kconfig0)) |
| 44 | |
| 45 | kconfig1 = kunit_config.Kconfig() |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 46 | kconfig1.add_entry(kunit_config.KconfigEntry('TEST', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 47 | self.assertTrue(kconfig1.is_subset_of(kconfig1)) |
| 48 | self.assertTrue(kconfig0.is_subset_of(kconfig1)) |
| 49 | self.assertFalse(kconfig1.is_subset_of(kconfig0)) |
| 50 | |
| 51 | def test_read_from_file(self): |
| 52 | kconfig = kunit_config.Kconfig() |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 53 | kconfig_path = test_data_path('test_read_from_file.kconfig') |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 54 | |
| 55 | kconfig.read_from_file(kconfig_path) |
| 56 | |
| 57 | expected_kconfig = kunit_config.Kconfig() |
| 58 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 59 | kunit_config.KconfigEntry('UML', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 60 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 61 | kunit_config.KconfigEntry('MMU', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 62 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 63 | kunit_config.KconfigEntry('TEST', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 64 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 65 | kunit_config.KconfigEntry('EXAMPLE_TEST', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 66 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 67 | kunit_config.KconfigEntry('MK8', 'n')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 68 | |
| 69 | self.assertEqual(kconfig.entries(), expected_kconfig.entries()) |
| 70 | |
| 71 | def test_write_to_file(self): |
| 72 | kconfig_path = os.path.join(test_tmpdir, '.config') |
| 73 | |
| 74 | expected_kconfig = kunit_config.Kconfig() |
| 75 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 76 | kunit_config.KconfigEntry('UML', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 77 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 78 | kunit_config.KconfigEntry('MMU', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 79 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 80 | kunit_config.KconfigEntry('TEST', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 81 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 82 | kunit_config.KconfigEntry('EXAMPLE_TEST', 'y')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 83 | expected_kconfig.add_entry( |
David Gow | 97752c3 | 2020-03-23 19:43:33 -0700 | [diff] [blame] | 84 | kunit_config.KconfigEntry('MK8', 'n')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 85 | |
| 86 | expected_kconfig.write_to_file(kconfig_path) |
| 87 | |
| 88 | actual_kconfig = kunit_config.Kconfig() |
| 89 | actual_kconfig.read_from_file(kconfig_path) |
| 90 | |
| 91 | self.assertEqual(actual_kconfig.entries(), |
| 92 | expected_kconfig.entries()) |
| 93 | |
| 94 | class KUnitParserTest(unittest.TestCase): |
| 95 | |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 96 | def assertContains(self, needle: str, haystack: kunit_parser.LineStream): |
| 97 | # Clone the iterator so we can print the contents on failure. |
| 98 | copy, backup = itertools.tee(haystack) |
| 99 | for line in copy: |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 100 | if needle in line: |
| 101 | return |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 102 | raise AssertionError(f'"{needle}" not found in {list(backup)}!') |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 103 | |
| 104 | def test_output_isolated_correctly(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 105 | log_path = test_data_path('test_output_isolated_correctly.log') |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 106 | with open(log_path) as file: |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 107 | result = kunit_parser.extract_tap_lines(file.readlines()) |
Daniel Latypov | 060352e | 2020-10-30 15:38:53 -0700 | [diff] [blame] | 108 | self.assertContains('TAP version 14', result) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 109 | self.assertContains(' # Subtest: example', result) |
| 110 | self.assertContains(' 1..2', result) |
| 111 | self.assertContains(' ok 1 - example_simple_test', result) |
| 112 | self.assertContains(' ok 2 - example_mock_test', result) |
| 113 | self.assertContains('ok 1 - example', result) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 114 | |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 115 | def test_output_with_prefix_isolated_correctly(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 116 | log_path = test_data_path('test_pound_sign.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 117 | with open(log_path) as file: |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 118 | result = kunit_parser.extract_tap_lines(file.readlines()) |
Daniel Latypov | 060352e | 2020-10-30 15:38:53 -0700 | [diff] [blame] | 119 | self.assertContains('TAP version 14', result) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 120 | self.assertContains(' # Subtest: kunit-resource-test', result) |
| 121 | self.assertContains(' 1..5', result) |
| 122 | self.assertContains(' ok 1 - kunit_resource_test_init_resources', result) |
| 123 | self.assertContains(' ok 2 - kunit_resource_test_alloc_resource', result) |
| 124 | self.assertContains(' ok 3 - kunit_resource_test_destroy_resource', result) |
| 125 | self.assertContains(' foo bar #', result) |
| 126 | self.assertContains(' ok 4 - kunit_resource_test_cleanup_resources', result) |
| 127 | self.assertContains(' ok 5 - kunit_resource_test_proper_free_ordering', result) |
| 128 | self.assertContains('ok 1 - kunit-resource-test', result) |
| 129 | self.assertContains(' foo bar # non-kunit output', result) |
| 130 | self.assertContains(' # Subtest: kunit-try-catch-test', result) |
| 131 | self.assertContains(' 1..2', result) |
| 132 | self.assertContains(' ok 1 - kunit_test_try_catch_successful_try_no_catch', |
| 133 | result) |
| 134 | self.assertContains(' ok 2 - kunit_test_try_catch_unsuccessful_try_does_catch', |
| 135 | result) |
| 136 | self.assertContains('ok 2 - kunit-try-catch-test', result) |
| 137 | self.assertContains(' # Subtest: string-stream-test', result) |
| 138 | self.assertContains(' 1..3', result) |
| 139 | self.assertContains(' ok 1 - string_stream_test_empty_on_creation', result) |
| 140 | self.assertContains(' ok 2 - string_stream_test_not_empty_after_add', result) |
| 141 | self.assertContains(' ok 3 - string_stream_test_get_string', result) |
| 142 | self.assertContains('ok 3 - string-stream-test', result) |
| 143 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 144 | def test_parse_successful_test_log(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 145 | all_passed_log = test_data_path('test_is_test_passed-all_passed.log') |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 146 | with open(all_passed_log) as file: |
| 147 | result = kunit_parser.parse_run_tests(file.readlines()) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 148 | self.assertEqual( |
| 149 | kunit_parser.TestStatus.SUCCESS, |
| 150 | result.status) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 151 | |
| 152 | def test_parse_failed_test_log(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 153 | failed_log = test_data_path('test_is_test_passed-failure.log') |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 154 | with open(failed_log) as file: |
| 155 | result = kunit_parser.parse_run_tests(file.readlines()) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 156 | self.assertEqual( |
| 157 | kunit_parser.TestStatus.FAILURE, |
| 158 | result.status) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 159 | |
Rae Moar | ed01ad3 | 2021-07-01 17:52:31 +0000 | [diff] [blame^] | 160 | def test_no_header(self): |
| 161 | empty_log = test_data_path('test_is_test_passed-no_tests_run_no_header.log') |
| 162 | with open(empty_log) as file: |
| 163 | result = kunit_parser.parse_run_tests( |
| 164 | kunit_parser.extract_tap_lines(file.readlines())) |
| 165 | self.assertEqual(0, len(result.suites)) |
| 166 | self.assertEqual( |
| 167 | kunit_parser.TestStatus.FAILURE_TO_PARSE_TESTS, |
| 168 | result.status) |
| 169 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 170 | def test_no_tests(self): |
Rae Moar | ed01ad3 | 2021-07-01 17:52:31 +0000 | [diff] [blame^] | 171 | empty_log = test_data_path('test_is_test_passed-no_tests_run_with_header.log') |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 172 | with open(empty_log) as file: |
| 173 | result = kunit_parser.parse_run_tests( |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 174 | kunit_parser.extract_tap_lines(file.readlines())) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 175 | self.assertEqual(0, len(result.suites)) |
| 176 | self.assertEqual( |
| 177 | kunit_parser.TestStatus.NO_TESTS, |
| 178 | result.status) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 179 | |
Uriel Guajardo | e173b8b | 2020-06-11 21:05:45 +0000 | [diff] [blame] | 180 | def test_no_kunit_output(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 181 | crash_log = test_data_path('test_insufficient_memory.log') |
Uriel Guajardo | e173b8b | 2020-06-11 21:05:45 +0000 | [diff] [blame] | 182 | print_mock = mock.patch('builtins.print').start() |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 183 | with open(crash_log) as file: |
| 184 | result = kunit_parser.parse_run_tests( |
Daniel Latypov | b29b14f | 2021-05-26 01:22:17 -0700 | [diff] [blame] | 185 | kunit_parser.extract_tap_lines(file.readlines())) |
Rae Moar | ed01ad3 | 2021-07-01 17:52:31 +0000 | [diff] [blame^] | 186 | print_mock.assert_any_call(StrContains('could not parse test results!')) |
Uriel Guajardo | e173b8b | 2020-06-11 21:05:45 +0000 | [diff] [blame] | 187 | print_mock.stop() |
| 188 | file.close() |
| 189 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 190 | def test_crashed_test(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 191 | crashed_log = test_data_path('test_is_test_passed-crash.log') |
Daniel Latypov | a3ece07 | 2020-12-02 11:08:23 -0800 | [diff] [blame] | 192 | with open(crashed_log) as file: |
| 193 | result = kunit_parser.parse_run_tests(file.readlines()) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 194 | self.assertEqual( |
| 195 | kunit_parser.TestStatus.TEST_CRASHED, |
| 196 | result.status) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 197 | |
David Gow | 5acaf60 | 2021-06-24 23:58:13 -0700 | [diff] [blame] | 198 | def test_skipped_test(self): |
| 199 | skipped_log = test_data_path('test_skip_tests.log') |
| 200 | file = open(skipped_log) |
| 201 | result = kunit_parser.parse_run_tests(file.readlines()) |
| 202 | |
| 203 | # A skipped test does not fail the whole suite. |
| 204 | self.assertEqual( |
| 205 | kunit_parser.TestStatus.SUCCESS, |
| 206 | result.status) |
| 207 | file.close() |
| 208 | |
| 209 | def test_skipped_all_tests(self): |
| 210 | skipped_log = test_data_path('test_skip_all_tests.log') |
| 211 | file = open(skipped_log) |
| 212 | result = kunit_parser.parse_run_tests(file.readlines()) |
| 213 | |
| 214 | self.assertEqual( |
| 215 | kunit_parser.TestStatus.SKIPPED, |
| 216 | result.status) |
| 217 | file.close() |
| 218 | |
| 219 | |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 220 | def test_ignores_prefix_printk_time(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 221 | prefix_log = test_data_path('test_config_printk_time.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 222 | with open(prefix_log) as file: |
| 223 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 224 | self.assertEqual( |
| 225 | kunit_parser.TestStatus.SUCCESS, |
| 226 | result.status) |
| 227 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 228 | |
| 229 | def test_ignores_multiple_prefixes(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 230 | prefix_log = test_data_path('test_multiple_prefixes.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 231 | with open(prefix_log) as file: |
| 232 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 233 | self.assertEqual( |
| 234 | kunit_parser.TestStatus.SUCCESS, |
| 235 | result.status) |
| 236 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 237 | |
| 238 | def test_prefix_mixed_kernel_output(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 239 | mixed_prefix_log = test_data_path('test_interrupted_tap_output.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 240 | with open(mixed_prefix_log) as file: |
| 241 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 242 | self.assertEqual( |
| 243 | kunit_parser.TestStatus.SUCCESS, |
| 244 | result.status) |
| 245 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 246 | |
| 247 | def test_prefix_poundsign(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 248 | pound_log = test_data_path('test_pound_sign.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 249 | with open(pound_log) as file: |
| 250 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 251 | self.assertEqual( |
| 252 | kunit_parser.TestStatus.SUCCESS, |
| 253 | result.status) |
| 254 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 255 | |
| 256 | def test_kernel_panic_end(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 257 | panic_log = test_data_path('test_kernel_panic_interrupt.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 258 | with open(panic_log) as file: |
| 259 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 260 | self.assertEqual( |
| 261 | kunit_parser.TestStatus.TEST_CRASHED, |
| 262 | result.status) |
| 263 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 264 | |
| 265 | def test_pound_no_prefix(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 266 | pound_log = test_data_path('test_pound_no_prefix.log') |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 267 | with open(pound_log) as file: |
| 268 | result = kunit_parser.parse_run_tests(file.readlines()) |
Brendan Higgins | 0d0d245 | 2020-10-21 13:39:14 -0700 | [diff] [blame] | 269 | self.assertEqual( |
| 270 | kunit_parser.TestStatus.SUCCESS, |
| 271 | result.status) |
| 272 | self.assertEqual('kunit-resource-test', result.suites[0].name) |
Heidi Fahim | afc63da | 2020-03-16 13:21:24 -0700 | [diff] [blame] | 273 | |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 274 | class LinuxSourceTreeTest(unittest.TestCase): |
| 275 | |
| 276 | def setUp(self): |
| 277 | mock.patch.object(signal, 'signal').start() |
| 278 | self.addCleanup(mock.patch.stopall) |
| 279 | |
| 280 | def test_invalid_kunitconfig(self): |
| 281 | with self.assertRaisesRegex(kunit_kernel.ConfigError, 'nonexistent.* does not exist'): |
| 282 | kunit_kernel.LinuxSourceTree('', kunitconfig_path='/nonexistent_file') |
| 283 | |
| 284 | def test_valid_kunitconfig(self): |
| 285 | with tempfile.NamedTemporaryFile('wt') as kunitconfig: |
| 286 | tree = kunit_kernel.LinuxSourceTree('', kunitconfig_path=kunitconfig.name) |
| 287 | |
Daniel Latypov | 9854781 | 2021-02-22 14:52:41 -0800 | [diff] [blame] | 288 | def test_dir_kunitconfig(self): |
| 289 | with tempfile.TemporaryDirectory('') as dir: |
| 290 | with open(os.path.join(dir, '.kunitconfig'), 'w') as f: |
| 291 | pass |
| 292 | tree = kunit_kernel.LinuxSourceTree('', kunitconfig_path=dir) |
| 293 | |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 294 | # TODO: add more test cases. |
| 295 | |
| 296 | |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 297 | class KUnitJsonTest(unittest.TestCase): |
| 298 | |
| 299 | def _json_for(self, log_file): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 300 | with open(test_data_path(log_file)) as file: |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 301 | test_result = kunit_parser.parse_run_tests(file) |
| 302 | json_obj = kunit_json.get_json_result( |
| 303 | test_result=test_result, |
| 304 | def_config='kunit_defconfig', |
| 305 | build_dir=None, |
| 306 | json_path='stdout') |
| 307 | return json.loads(json_obj) |
| 308 | |
| 309 | def test_failed_test_json(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 310 | result = self._json_for('test_is_test_passed-failure.log') |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 311 | self.assertEqual( |
| 312 | {'name': 'example_simple_test', 'status': 'FAIL'}, |
| 313 | result["sub_groups"][1]["test_cases"][0]) |
| 314 | |
| 315 | def test_crashed_test_json(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 316 | result = self._json_for('test_is_test_passed-crash.log') |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 317 | self.assertEqual( |
| 318 | {'name': 'example_simple_test', 'status': 'ERROR'}, |
| 319 | result["sub_groups"][1]["test_cases"][0]) |
| 320 | |
| 321 | def test_no_tests_json(self): |
Rae Moar | ed01ad3 | 2021-07-01 17:52:31 +0000 | [diff] [blame^] | 322 | result = self._json_for('test_is_test_passed-no_tests_run_with_header.log') |
Heidi Fahim | 21a6d17 | 2020-08-11 14:27:56 -0700 | [diff] [blame] | 323 | self.assertEqual(0, len(result['sub_groups'])) |
| 324 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 325 | class StrContains(str): |
| 326 | def __eq__(self, other): |
| 327 | return self in other |
| 328 | |
| 329 | class KUnitMainTest(unittest.TestCase): |
| 330 | def setUp(self): |
Daniel Latypov | cd4a9bc | 2020-12-02 11:08:24 -0800 | [diff] [blame] | 331 | path = test_data_path('test_is_test_passed-all_passed.log') |
Daniel Latypov | cfd607e | 2020-12-02 11:08:21 -0800 | [diff] [blame] | 332 | with open(path) as file: |
| 333 | all_passed_log = file.readlines() |
| 334 | |
| 335 | self.print_mock = mock.patch('builtins.print').start() |
| 336 | self.addCleanup(mock.patch.stopall) |
| 337 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 338 | self.linux_source_mock = mock.Mock() |
| 339 | self.linux_source_mock.build_reconfig = mock.Mock(return_value=True) |
Brendan Higgins | 87c9c16 | 2021-05-26 14:24:06 -0700 | [diff] [blame] | 340 | self.linux_source_mock.build_kernel = mock.Mock(return_value=True) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 341 | self.linux_source_mock.run_kernel = mock.Mock(return_value=all_passed_log) |
| 342 | |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 343 | def test_config_passes_args_pass(self): |
Brendan Higgins | d43c7fb | 2020-07-14 13:41:30 -0700 | [diff] [blame] | 344 | kunit.main(['config', '--build_dir=.kunit'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 345 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
| 346 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 0) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 347 | |
| 348 | def test_build_passes_args_pass(self): |
| 349 | kunit.main(['build'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 350 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 0) |
Brendan Higgins | 87c9c16 | 2021-05-26 14:24:06 -0700 | [diff] [blame] | 351 | self.linux_source_mock.build_kernel.assert_called_once_with(False, 8, '.kunit', None) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 352 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 0) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 353 | |
| 354 | def test_exec_passes_args_pass(self): |
| 355 | kunit.main(['exec'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 356 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 0) |
| 357 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 358 | self.linux_source_mock.run_kernel.assert_called_once_with( |
| 359 | build_dir='.kunit', filter_glob='', timeout=300) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 360 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
| 361 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 362 | def test_run_passes_args_pass(self): |
| 363 | kunit.main(['run'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 364 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
| 365 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) |
Heidi Fahim | 021ed9f | 2020-03-16 13:21:25 -0700 | [diff] [blame] | 366 | self.linux_source_mock.run_kernel.assert_called_once_with( |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 367 | build_dir='.kunit', filter_glob='', timeout=300) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 368 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
| 369 | |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 370 | def test_exec_passes_args_fail(self): |
| 371 | self.linux_source_mock.run_kernel = mock.Mock(return_value=[]) |
| 372 | with self.assertRaises(SystemExit) as e: |
| 373 | kunit.main(['exec'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 374 | self.assertEqual(e.exception.code, 1) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 375 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 376 | def test_run_passes_args_fail(self): |
| 377 | self.linux_source_mock.run_kernel = mock.Mock(return_value=[]) |
| 378 | with self.assertRaises(SystemExit) as e: |
| 379 | kunit.main(['run'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 380 | self.assertEqual(e.exception.code, 1) |
| 381 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
| 382 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 383 | self.print_mock.assert_any_call(StrContains(' 0 tests run')) |
| 384 | |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 385 | def test_exec_raw_output(self): |
| 386 | self.linux_source_mock.run_kernel = mock.Mock(return_value=[]) |
| 387 | kunit.main(['exec', '--raw_output'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 388 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) |
| 389 | for call in self.print_mock.call_args_list: |
| 390 | self.assertNotEqual(call, mock.call(StrContains('Testing complete.'))) |
| 391 | self.assertNotEqual(call, mock.call(StrContains(' 0 tests run'))) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 392 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 393 | def test_run_raw_output(self): |
| 394 | self.linux_source_mock.run_kernel = mock.Mock(return_value=[]) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 395 | kunit.main(['run', '--raw_output'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 396 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
| 397 | self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) |
| 398 | for call in self.print_mock.call_args_list: |
| 399 | self.assertNotEqual(call, mock.call(StrContains('Testing complete.'))) |
| 400 | self.assertNotEqual(call, mock.call(StrContains(' 0 tests run'))) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 401 | |
| 402 | def test_exec_timeout(self): |
| 403 | timeout = 3453 |
| 404 | kunit.main(['exec', '--timeout', str(timeout)], self.linux_source_mock) |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 405 | self.linux_source_mock.run_kernel.assert_called_once_with( |
| 406 | build_dir='.kunit', filter_glob='', timeout=timeout) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 407 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 408 | |
| 409 | def test_run_timeout(self): |
| 410 | timeout = 3453 |
| 411 | kunit.main(['run', '--timeout', str(timeout)], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 412 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
Heidi Fahim | 021ed9f | 2020-03-16 13:21:25 -0700 | [diff] [blame] | 413 | self.linux_source_mock.run_kernel.assert_called_once_with( |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 414 | build_dir='.kunit', filter_glob='', timeout=timeout) |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 415 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
| 416 | |
SeongJae Park | b1b3520 | 2019-12-20 05:14:08 +0000 | [diff] [blame] | 417 | def test_run_builddir(self): |
| 418 | build_dir = '.kunit' |
Brendan Higgins | d43c7fb | 2020-07-14 13:41:30 -0700 | [diff] [blame] | 419 | kunit.main(['run', '--build_dir=.kunit'], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 420 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
Heidi Fahim | 021ed9f | 2020-03-16 13:21:25 -0700 | [diff] [blame] | 421 | self.linux_source_mock.run_kernel.assert_called_once_with( |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 422 | build_dir=build_dir, filter_glob='', timeout=300) |
SeongJae Park | b1b3520 | 2019-12-20 05:14:08 +0000 | [diff] [blame] | 423 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
| 424 | |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 425 | def test_config_builddir(self): |
| 426 | build_dir = '.kunit' |
| 427 | kunit.main(['config', '--build_dir', build_dir], self.linux_source_mock) |
Daniel Latypov | 0b3e680 | 2020-12-02 11:08:22 -0800 | [diff] [blame] | 428 | self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 429 | |
| 430 | def test_build_builddir(self): |
| 431 | build_dir = '.kunit' |
| 432 | kunit.main(['build', '--build_dir', build_dir], self.linux_source_mock) |
Brendan Higgins | 87c9c16 | 2021-05-26 14:24:06 -0700 | [diff] [blame] | 433 | self.linux_source_mock.build_kernel.assert_called_once_with(False, 8, build_dir, None) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 434 | |
| 435 | def test_exec_builddir(self): |
| 436 | build_dir = '.kunit' |
| 437 | kunit.main(['exec', '--build_dir', build_dir], self.linux_source_mock) |
Daniel Latypov | d992880b | 2021-02-05 16:08:53 -0800 | [diff] [blame] | 438 | self.linux_source_mock.run_kernel.assert_called_once_with( |
| 439 | build_dir=build_dir, filter_glob='', timeout=300) |
David Gow | 45ba7a8 | 2020-04-30 21:27:01 -0700 | [diff] [blame] | 440 | self.print_mock.assert_any_call(StrContains('Testing complete.')) |
| 441 | |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 442 | @mock.patch.object(kunit_kernel, 'LinuxSourceTree') |
| 443 | def test_run_kunitconfig(self, mock_linux_init): |
| 444 | mock_linux_init.return_value = self.linux_source_mock |
| 445 | kunit.main(['run', '--kunitconfig=mykunitconfig']) |
| 446 | # Just verify that we parsed and initialized it correctly here. |
Brendan Higgins | 87c9c16 | 2021-05-26 14:24:06 -0700 | [diff] [blame] | 447 | mock_linux_init.assert_called_once_with('.kunit', |
| 448 | kunitconfig_path='mykunitconfig', |
| 449 | arch='um', |
| 450 | cross_compile=None, |
| 451 | qemu_config_path=None) |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 452 | |
| 453 | @mock.patch.object(kunit_kernel, 'LinuxSourceTree') |
| 454 | def test_config_kunitconfig(self, mock_linux_init): |
| 455 | mock_linux_init.return_value = self.linux_source_mock |
| 456 | kunit.main(['config', '--kunitconfig=mykunitconfig']) |
| 457 | # Just verify that we parsed and initialized it correctly here. |
Brendan Higgins | 87c9c16 | 2021-05-26 14:24:06 -0700 | [diff] [blame] | 458 | mock_linux_init.assert_called_once_with('.kunit', |
| 459 | kunitconfig_path='mykunitconfig', |
| 460 | arch='um', |
| 461 | cross_compile=None, |
| 462 | qemu_config_path=None) |
Daniel Latypov | 243180f | 2021-02-01 12:55:14 -0800 | [diff] [blame] | 463 | |
Felix Guo | 6ebf586 | 2019-09-23 02:02:43 -0700 | [diff] [blame] | 464 | if __name__ == '__main__': |
| 465 | unittest.main() |