From nobody Sun Feb 8 20:28:59 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 599C4C001B0 for ; Fri, 7 Jul 2023 21:10:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232438AbjGGVKU (ORCPT ); Fri, 7 Jul 2023 17:10:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60268 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232744AbjGGVKM (ORCPT ); Fri, 7 Jul 2023 17:10:12 -0400 Received: from mail-yw1-x1149.google.com (mail-yw1-x1149.google.com [IPv6:2607:f8b0:4864:20::1149]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 55240212D for ; Fri, 7 Jul 2023 14:10:03 -0700 (PDT) Received: by mail-yw1-x1149.google.com with SMTP id 00721157ae682-573cacf4804so24655657b3.1 for ; Fri, 07 Jul 2023 14:10:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20221208; t=1688764202; x=1691356202; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=cZZVn4GL1xsWAqTIUq7UDdp+D3mHVFXeX/1OGD7LRzY=; b=p1TlXHxG39+pTa25N6I6Um//Q9VKJlZx+s6CFd1KFhnubnaL8Et64lOA1nBIz7BkO4 giAWq6eVV3mEtHPiJ1pn6hRvlYkZGlfUHHSgdf2KLjiJHndCz4UVw2bJnMp+kKoinrpl VKNoFHlev/N1tRxGbq6QQZY3tcUaLl4HSlZpHJPSwwA7flRDbFaR8fReHAzNSnFCl98m 0G7240FunmPmFKL47vba0L5GRjcLP0JmAXnDA6MFFEPQnSmrwf6vFA44RMf1UGbmsZbJ GsGF2R+/4FD/p3lu1HoE4ULjmo98CJUCQUOQtI44ZZhvRoyCF93m83uNICculgRnD6fa iBwQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688764202; x=1691356202; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=cZZVn4GL1xsWAqTIUq7UDdp+D3mHVFXeX/1OGD7LRzY=; b=bn1kk33vzga8JkLz0J/mU7lCvrI8N10oI7WWLvy0SCZ9vuCHr/L8aeukrxmp0/Ts7L pr1L7oAqOjLzCxoIrq8r31iSjrAj2+zidkPDxyFZjojq0Vf/RbapcPBkcRKGqreBr6k+ jxn6mFlc76rvqmE3ZaX5Kq+bzH0/WuanuyBuBMsBzr3mz8GKtWsHc4HXHd+U5dZzmRMt xY+tiFICXcO1ZaZ9ZTLRpp8HifYFeHiPz4q2MF58MSQgyv6KcnhXVp/JYsImipsbo3+0 r0M//oQTFu2/Ab+7DkBQuAhtkvOz3iKFOrwdPl93k14XOUwgLtxEpX5w4QdAiBzcd1Un 5qbQ== X-Gm-Message-State: ABy/qLZgmpRD+DKUbRZA1s6VOiHwXZ1HKV32qCaRcgXshjFaXM0piHlm dQLXsSul9G2JZ7ygbX0wW9kyrzZTjA== X-Google-Smtp-Source: APBJJlHQg+UDXLchFCf9netJSWZyJGw19qlrWpeY2nX4RLLm6P9XQVBlwld8VyD2pQwCH83H4vCqpmxJkg== X-Received: from rmoar-specialist.c.googlers.com ([fda3:e722:ac3:cc00:2b:7d90:c0a8:45d3]) (user=rmoar job=sendgmr) by 2002:a25:d6:0:b0:c6e:269e:953a with SMTP id 205-20020a2500d6000000b00c6e269e953amr12102yba.3.1688764202530; Fri, 07 Jul 2023 14:10:02 -0700 (PDT) Date: Fri, 7 Jul 2023 21:09:43 +0000 In-Reply-To: <20230707210947.1208717-1-rmoar@google.com> Mime-Version: 1.0 References: <20230707210947.1208717-1-rmoar@google.com> X-Mailer: git-send-email 2.41.0.255.g8b1d071c50-goog Message-ID: <20230707210947.1208717-6-rmoar@google.com> Subject: [RFC v2 5/9] kunit: tool: Add command line interface to filter and report attributes From: Rae Moar To: shuah@kernel.org, davidgow@google.com, dlatypov@google.com, brendan.higgins@linux.dev Cc: linux-kselftest@vger.kernel.org, kunit-dev@googlegroups.com, linux-kernel@vger.kernel.org, keescook@chromium.org, linux-hardening@vger.kernel.org, jstultz@google.com, tglx@linutronix.de, sboyd@kernel.org, Rae Moar Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add ability to kunit.py to filter attributes and report a list of tests including attributes without running tests. Add flag "--filter" to input filters on test attributes. Tests will be filtered out if they do not match all inputted filters. Example: --filter speed=3Dslow (This filter would run only the tests that a= re marked as slow) Filters have operations: <, >, <=3D, >=3D, !=3D, and =3D. But note that the characters < and > are often interpreted by the shell, so they may need to be quoted or escaped. Example: --filter "speed>slow" or --filter speed\>slow (This filter would run only the tests that have the speed faster than slow. Additionally, multiple filters can be used. Example: --filter "speed=3Dslow, module!=3Dexample" (This filter would run only the tests that have the speed slow and are not in the "example" module) Note if the user wants to skip filtered tests instead of not running/showing them use the "--filter_skip" flag instead. Expose the output of kunit.action=3Dlist option with flag "--list_tests" to output a list of tests. Additionally, add flag "--list_tests_attr" to output a list of tests and their attributes. These flags are useful to see tests and test attributes without needing to run tests. Example of the output of "--list_tests_attr": example example.test_1 example.test_2 # example.test_2.speed: slow This output includes a suite, example, with two test cases, test_1 and test_2. And in this instance test_2 has been marked as slow. Signed-off-by: Rae Moar --- Changes since v1: - Change method for inputting filters to allow for spaces in filtering values - Add option to skip filtered tests instead of not run or show them with the --filter_skip flag - Separate the new feature to list tests and their attributes into both --list_tests (lists just tests) and --list_tests_attr (lists all) tools/testing/kunit/kunit.py | 80 ++++++++++++++++++++++++-- tools/testing/kunit/kunit_kernel.py | 6 +- tools/testing/kunit/kunit_tool_test.py | 39 ++++++------- 3 files changed, 96 insertions(+), 29 deletions(-) diff --git a/tools/testing/kunit/kunit.py b/tools/testing/kunit/kunit.py index 3905c43369c3..6104e622ce20 100755 --- a/tools/testing/kunit/kunit.py +++ b/tools/testing/kunit/kunit.py @@ -55,8 +55,12 @@ class KunitExecRequest(KunitParseRequest): build_dir: str timeout: int filter_glob: str + filter: str + filter_skip: str kernel_args: Optional[List[str]] run_isolated: Optional[str] + list_tests: bool + list_tests_attr: bool =20 @dataclass class KunitRequest(KunitExecRequest, KunitBuildRequest): @@ -102,19 +106,39 @@ def config_and_build_tests(linux: kunit_kernel.LinuxS= ourceTree, =20 def _list_tests(linux: kunit_kernel.LinuxSourceTree, request: KunitExecReq= uest) -> List[str]: args =3D ['kunit.action=3Dlist'] + + if request.kernel_args: + args.extend(request.kernel_args) + + output =3D linux.run_kernel(args=3Dargs, + timeout=3Drequest.timeout, + filter_glob=3Drequest.filter_glob, + filter=3Drequest.filter, + build_dir=3Drequest.build_dir) + lines =3D kunit_parser.extract_tap_lines(output) + # Hack! Drop the dummy TAP version header that the executor prints out. + lines.pop() + + # Filter out any extraneous non-test output that might have gotten mixed = in. + return [l for l in output if re.match(r'^[^\s.]+\.[^\s.]+$', l)] + +def _list_tests_attr(linux: kunit_kernel.LinuxSourceTree, request: KunitEx= ecRequest) -> Iterable[str]: + args =3D ['kunit.action=3Dlist_attr'] + if request.kernel_args: args.extend(request.kernel_args) =20 output =3D linux.run_kernel(args=3Dargs, timeout=3Drequest.timeout, filter_glob=3Drequest.filter_glob, + filter=3Drequest.filter, build_dir=3Drequest.build_dir) lines =3D kunit_parser.extract_tap_lines(output) # Hack! Drop the dummy TAP version header that the executor prints out. lines.pop() =20 # Filter out any extraneous non-test output that might have gotten mixed = in. - return [l for l in lines if re.match(r'^[^\s.]+\.[^\s.]+$', l)] + return lines =20 def _suites_from_test_list(tests: List[str]) -> List[str]: """Extracts all the suites from an ordered list of tests.""" @@ -128,10 +152,18 @@ def _suites_from_test_list(tests: List[str]) -> List[= str]: suites.append(suite) return suites =20 - - def exec_tests(linux: kunit_kernel.LinuxSourceTree, request: KunitExecRequ= est) -> KunitResult: filter_globs =3D [request.filter_glob] + if request.list_tests: + output =3D _list_tests(linux, request) + for line in output: + print(line.rstrip()) + return KunitResult(status=3DKunitStatus.SUCCESS, elapsed_time=3D0.0) + if request.list_tests_attr: + attr_output =3D _list_tests_attr(linux, request) + for line in attr_output: + print(line.rstrip()) + return KunitResult(status=3DKunitStatus.SUCCESS, elapsed_time=3D0.0) if request.run_isolated: tests =3D _list_tests(linux, request) if request.run_isolated =3D=3D 'test': @@ -145,6 +177,17 @@ def exec_tests(linux: kunit_kernel.LinuxSourceTree, re= quest: KunitExecRequest) - =20 metadata =3D kunit_json.Metadata(arch=3Dlinux.arch(), build_dir=3Drequest= .build_dir, def_config=3D'kunit_defconfig') =20 + filter =3D request.filter + if request.filter_skip: + args =3D ['kunit.filter_action=3Dskip'] + filter =3D request.filter_skip + if request.kernel_args: + args.extend(request.kernel_args) + elif request.kernel_args: + args =3D request.kernel_args + else: + args =3D None + test_counts =3D kunit_parser.TestCounts() exec_time =3D 0.0 for i, filter_glob in enumerate(filter_globs): @@ -152,9 +195,10 @@ def exec_tests(linux: kunit_kernel.LinuxSourceTree, re= quest: KunitExecRequest) - =20 test_start =3D time.time() run_result =3D linux.run_kernel( - args=3Drequest.kernel_args, + args=3Dargs, timeout=3Drequest.timeout, filter_glob=3Dfilter_glob, + filter=3Dfilter, build_dir=3Drequest.build_dir) =20 _, test_result =3D parse_tests(request, metadata, run_result) @@ -341,6 +385,18 @@ def add_exec_opts(parser: argparse.ArgumentParser) -> = None: nargs=3D'?', default=3D'', metavar=3D'filter_glob') + parser.add_argument('--filter', + help=3D'Filter KUnit tests with attributes, ' + 'filtered tests will not run, ' + 'e.g. speed=3Dfast or speed=3D>low', + type=3Dstr, + default=3D'') + parser.add_argument('--filter_skip', + help=3D'Filter KUnit tests run with attributes, ' + 'filtered tests will be skipped, ' + 'e.g. speed=3Dfast or speed=3D>low', + type=3Dstr, + default=3D'') parser.add_argument('--kernel_args', help=3D'Kernel command-line parameters. Maybe be repeated', action=3D'append', metavar=3D'') @@ -350,6 +406,10 @@ def add_exec_opts(parser: argparse.ArgumentParser) -> = None: 'what ran before it.', type=3Dstr, choices=3D['suite', 'test']) + parser.add_argument('--list_tests', help=3D'If set, list all tests', + action=3D'store_true') + parser.add_argument('--list_tests_attr', help=3D'If set, list all tests a= nd attributes.', + action=3D'store_true') =20 def add_parse_opts(parser: argparse.ArgumentParser) -> None: parser.add_argument('--raw_output', help=3D'If set don\'t parse output fr= om kernel. ' @@ -398,8 +458,12 @@ def run_handler(cli_args: argparse.Namespace) -> None: json=3Dcli_args.json, timeout=3Dcli_args.timeout, filter_glob=3Dcli_args.filter_glob, + filter=3Dcli_args.filter, + filter_skip=3Dcli_args.filter_skip, kernel_args=3Dcli_args.kernel_args, - run_isolated=3Dcli_args.run_isolated) + run_isolated=3Dcli_args.run_isolated, + list_tests=3Dcli_args.list_tests, + list_tests_attr=3Dcli_args.list_tests_attr) result =3D run_tests(linux, request) if result.status !=3D KunitStatus.SUCCESS: sys.exit(1) @@ -441,8 +505,12 @@ def exec_handler(cli_args: argparse.Namespace) -> None: json=3Dcli_args.json, timeout=3Dcli_args.timeout, filter_glob=3Dcli_args.filter_glob, + filter=3Dcli_args.filter, + filter_skip=3Dcli_args.filter_skip, kernel_args=3Dcli_args.kernel_args, - run_isolated=3Dcli_args.run_isolated) + run_isolated=3Dcli_args.run_isolated, + list_tests=3Dcli_args.list_tests, + list_tests_attr=3Dcli_args.list_tests_attr) result =3D exec_tests(linux, exec_request) stdout.print_with_timestamp(( 'Elapsed time: %.3fs\n') % (result.elapsed_time)) diff --git a/tools/testing/kunit/kunit_kernel.py b/tools/testing/kunit/kuni= t_kernel.py index 7f648802caf6..281f062a4767 100644 --- a/tools/testing/kunit/kunit_kernel.py +++ b/tools/testing/kunit/kunit_kernel.py @@ -330,11 +330,13 @@ class LinuxSourceTree: return False return self.validate_config(build_dir) =20 - def run_kernel(self, args: Optional[List[str]]=3DNone, build_dir: str=3D'= ', filter_glob: str=3D'', timeout: Optional[int]=3DNone) -> Iterator[str]: + def run_kernel(self, args: Optional[List[str]]=3DNone, build_dir: str=3D'= ', filter_glob: str=3D'', filter: str=3D'', timeout: Optional[int]=3DNone) = -> Iterator[str]: if not args: args =3D [] if filter_glob: - args.append('kunit.filter_glob=3D'+filter_glob) + args.append('kunit.filter_glob=3D' + filter_glob) + if filter: + args.append('kunit.filter=3D"' + filter + '"') args.append('kunit.enable=3D1') =20 process =3D self._ops.start(args, build_dir) diff --git a/tools/testing/kunit/kunit_tool_test.py b/tools/testing/kunit/k= unit_tool_test.py index be35999bb84f..85a1fb72735e 100755 --- a/tools/testing/kunit/kunit_tool_test.py +++ b/tools/testing/kunit/kunit_tool_test.py @@ -597,7 +597,7 @@ class KUnitMainTest(unittest.TestCase): self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 0) self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', timeout=3D300) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', filter=3D'', timeo= ut=3D300) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_run_passes_args_pass(self): @@ -605,7 +605,7 @@ class KUnitMainTest(unittest.TestCase): self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) self.assertEqual(self.linux_source_mock.run_kernel.call_count, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', timeout=3D300) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', filter=3D'', timeo= ut=3D300) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_exec_passes_args_fail(self): @@ -629,7 +629,7 @@ class KUnitMainTest(unittest.TestCase): kunit.main(['run']) self.assertEqual(e.exception.code, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', timeout=3D300) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', filter=3D'', timeo= ut=3D300) self.print_mock.assert_any_call(StrContains(' 0 tests run!')) =20 def test_exec_raw_output(self): @@ -670,13 +670,13 @@ class KUnitMainTest(unittest.TestCase): self.linux_source_mock.run_kernel =3D mock.Mock(return_value=3D[]) kunit.main(['run', '--raw_output', 'filter_glob']) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'filter_glob', timeout= =3D300) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'filter_glob', filter= =3D'', timeout=3D300) =20 def test_exec_timeout(self): timeout =3D 3453 kunit.main(['exec', '--timeout', str(timeout)]) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', timeout=3Dtimeout) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', filter=3D'', timeo= ut=3Dtimeout) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_run_timeout(self): @@ -684,7 +684,7 @@ class KUnitMainTest(unittest.TestCase): kunit.main(['run', '--timeout', str(timeout)]) self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', timeout=3Dtimeout) + args=3DNone, build_dir=3D'.kunit', filter_glob=3D'', filter=3D'', timeo= ut=3Dtimeout) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_run_builddir(self): @@ -692,7 +692,7 @@ class KUnitMainTest(unittest.TestCase): kunit.main(['run', '--build_dir=3D.kunit']) self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3Dbuild_dir, filter_glob=3D'', timeout=3D300) + args=3DNone, build_dir=3Dbuild_dir, filter_glob=3D'', filter=3D'', time= out=3D300) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_config_builddir(self): @@ -710,7 +710,7 @@ class KUnitMainTest(unittest.TestCase): build_dir =3D '.kunit' kunit.main(['exec', '--build_dir', build_dir]) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3DNone, build_dir=3Dbuild_dir, filter_glob=3D'', timeout=3D300) + args=3DNone, build_dir=3Dbuild_dir, filter_glob=3D'', filter=3D'', time= out=3D300) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_run_kunitconfig(self): @@ -786,7 +786,7 @@ class KUnitMainTest(unittest.TestCase): kunit.main(['run', '--kernel_args=3Da=3D1', '--kernel_args=3Db=3D2']) self.assertEqual(self.linux_source_mock.build_reconfig.call_count, 1) self.linux_source_mock.run_kernel.assert_called_once_with( - args=3D['a=3D1','b=3D2'], build_dir=3D'.kunit', filter_glob=3D'', = timeout=3D300) + args=3D['a=3D1','b=3D2'], build_dir=3D'.kunit', filter_glob=3D'', = filter=3D'', timeout=3D300) self.print_mock.assert_any_call(StrContains('Testing complete.')) =20 def test_list_tests(self): @@ -794,13 +794,11 @@ class KUnitMainTest(unittest.TestCase): self.linux_source_mock.run_kernel.return_value =3D ['TAP version 14', 'i= nit: random output'] + want =20 got =3D kunit._list_tests(self.linux_source_mock, - kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*', None,= 'suite')) - + kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*', '', '= ', None, 'suite', False, False)) self.assertEqual(got, want) # Should respect the user's filter glob when listing tests. self.linux_source_mock.run_kernel.assert_called_once_with( - args=3D['kunit.action=3Dlist'], build_dir=3D'.kunit', filter_glob=3D'su= ite*', timeout=3D300) - + args=3D['kunit.action=3Dlist'], build_dir=3D'.kunit', filter_glob=3D'su= ite*', filter=3D'', timeout=3D300) =20 @mock.patch.object(kunit, '_list_tests') def test_run_isolated_by_suite(self, mock_tests): @@ -809,10 +807,10 @@ class KUnitMainTest(unittest.TestCase): =20 # Should respect the user's filter glob when listing tests. mock_tests.assert_called_once_with(mock.ANY, - kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*.test*',= None, 'suite')) + kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*.test*',= '', '', None, 'suite', False, False)) self.linux_source_mock.run_kernel.assert_has_calls([ - mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test*= ', timeout=3D300), - mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite2.test= *', timeout=3D300), + mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test*= ', filter=3D'', timeout=3D300), + mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite2.test= *', filter=3D'', timeout=3D300), ]) =20 @mock.patch.object(kunit, '_list_tests') @@ -822,13 +820,12 @@ class KUnitMainTest(unittest.TestCase): =20 # Should respect the user's filter glob when listing tests. mock_tests.assert_called_once_with(mock.ANY, - kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*', None,= 'test')) + kunit.KunitExecRequest(None, None, '.kunit', 300, 'suite*', '', '= ', None, 'test', False, False)) self.linux_source_mock.run_kernel.assert_has_calls([ - mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test1= ', timeout=3D300), - mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test2= ', timeout=3D300), - mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite2.test= 1', timeout=3D300), + mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test1= ', filter=3D'', timeout=3D300), + mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite.test2= ', filter=3D'', timeout=3D300), + mock.call(args=3DNone, build_dir=3D'.kunit', filter_glob=3D'suite2.test= 1', filter=3D'', timeout=3D300), ]) =20 - if __name__ =3D=3D '__main__': unittest.main() --=20 2.41.0.255.g8b1d071c50-goog