Commit 922d1dde authored by David Gow's avatar David Gow Committed by Shuah Khan
Browse files

kunit: tool: Accept --raw_output=full as an alias of 'all' 

I can never remember whether --raw_output takes 'all' or 'full'. No
reason we can't support both.

For the record, 'all' is the recommended, documented option.

Link: https://lore.kernel.org/r/20250730031624.1911689-1-davidgow@google.com


Signed-off-by: default avatarDavid Gow <davidgow@google.com>
Reviewed-by: default avatarRae Moar <rmoar@google.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent bd80c4d6

tools/testing/kunit/kunit.py

+2 −2
Original line number Diff line number Diff line
@@ -228,7 +228,7 @@ def parse_tests(request: KunitParseRequest, metadata: kunit_json.Metadata, input 
		fake_test.counts.passed = 1



		output: Iterable[str] = input_data

		if request.raw_output == 'all':

		if request.raw_output == 'all' or request.raw_output == 'full':

			pass

		elif request.raw_output == 'kunit':

			output = kunit_parser.extract_tap_lines(output)
@@ -425,7 +425,7 @@ def add_parse_opts(parser: argparse.ArgumentParser) -> None: 
	parser.add_argument('--raw_output', help='If set don\'t parse output from kernel. '

			    'By default, filters to just KUnit output. Use '

			    '--raw_output=all to show everything',

			     type=str, nargs='?', const='all', default=None, choices=['all', 'kunit'])

			     type=str, nargs='?', const='all', default=None, choices=['all', 'full', 'kunit'])

	parser.add_argument('--json',

			    nargs='?',

			    help='Prints parsed test results as JSON to stdout or a file if '