From 00c3b7f83da4c6819d84186c7f621c3df6a35ddf Mon Sep 17 00:00:00 2001 From: Ben Copeland Date: Wed, 18 Feb 2026 11:59:09 +0000 Subject: [PATCH] results/hardware: pass all filter options to cmd_tests and cmd_builds The builds_and_tests_options decorator adds filter options (start_date, end_date, compiler, config, hardware, test_path, git_branch, compatible, min_duration, max_duration) but the boots, builds, and tests commands did not accept or forward them, causing a TypeError on invocation. Add all missing parameters to the function signatures and pass them through to cmd_tests and cmd_builds. Signed-off-by: Ben Copeland --- kcidev/subcommands/results/hardware.py | 112 +++++++++++++++++++++++-- 1 file changed, 106 insertions(+), 6 deletions(-) diff --git a/kcidev/subcommands/results/hardware.py b/kcidev/subcommands/results/hardware.py index e3a2cce..8fbb6fa 100644 --- a/kcidev/subcommands/results/hardware.py +++ b/kcidev/subcommands/results/hardware.py @@ -57,24 +57,124 @@ def summary(name, origin, use_json): @hardware_common_opt @results_display_options @builds_and_tests_options -def boots(name, origin, use_json, download_logs, status, filter, count): +def boots( + name, + origin, + use_json, + download_logs, + status, + filter, + start_date, + end_date, + compiler, + config, + hardware, + test_path, + git_branch, + compatible, + min_duration, + max_duration, + count, +): data = dashboard_fetch_hardware_boots(name, origin, use_json) - cmd_tests(data["boots"], name, download_logs, status, filter, count, use_json) + cmd_tests( + data["boots"], + name, + download_logs, + status, + filter, + start_date, + end_date, + compiler, + config, + hardware, + test_path, + git_branch, + compatible, + min_duration, + max_duration, + count, + use_json, + ) @hardware.command() @hardware_common_opt @results_display_options @builds_and_tests_options -def builds(name, origin, use_json, download_logs, status, filter, count): +def builds( + name, + origin, + use_json, + download_logs, + status, + filter, + start_date, + end_date, + compiler, + config, + hardware, + test_path, + git_branch, + compatible, + min_duration, + max_duration, + count, +): data = dashboard_fetch_hardware_builds(name, origin, use_json) - cmd_builds(data, name, download_logs, status, count, use_json) + cmd_builds( + data, + name, + download_logs, + status, + compiler, + config, + git_branch, + count, + use_json, + ) @hardware.command() @hardware_common_opt @results_display_options @builds_and_tests_options -def tests(name, origin, use_json, download_logs, status, filter, count): +def tests( + name, + origin, + use_json, + download_logs, + status, + filter, + start_date, + end_date, + compiler, + config, + hardware, + test_path, + git_branch, + compatible, + min_duration, + max_duration, + count, +): data = dashboard_fetch_hardware_tests(name, origin, use_json) - cmd_tests(data["tests"], name, download_logs, status, filter, count, use_json) + cmd_tests( + data["tests"], + name, + download_logs, + status, + filter, + start_date, + end_date, + compiler, + config, + hardware, + test_path, + git_branch, + compatible, + min_duration, + max_duration, + count, + use_json, + )