From 0bffbe69b658dd964039f92fa630760466893bbc Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Sun, 7 Jun 2026 06:40:20 -0700 Subject: [PATCH] test(plugins): update list fixtures to 6-tuple (name,ver,desc,source,dir,key) Salvaged #41076 added a 'key' field to plugin entries for category-aware status matching; the existing list-test fixtures were still 5-tuples and broke with ValueError. Append key=name (flat plugins use name as key). --- tests/hermes_cli/test_plugins_cmd_list.py | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/hermes_cli/test_plugins_cmd_list.py b/tests/hermes_cli/test_plugins_cmd_list.py index 1d9051c2822f..5e8c061dab45 100644 --- a/tests/hermes_cli/test_plugins_cmd_list.py +++ b/tests/hermes_cli/test_plugins_cmd_list.py @@ -18,9 +18,9 @@ def _args(**kwargs): def test_filter_plugin_entries_enabled_only(): entries = [ - ("disk-cleanup", "2.0.0", "Bundled", "bundled", None), - ("web-search-plus", "2.2.0", "Search", "git", None), - ("old-plugin", "1.0.0", "Old", "user", None), + ("disk-cleanup", "2.0.0", "Bundled", "bundled", None, "disk-cleanup"), + ("web-search-plus", "2.2.0", "Search", "git", None, "web-search-plus"), + ("old-plugin", "1.0.0", "Old", "user", None, "old-plugin"), ] filtered = plugins_cmd._filter_plugin_entries( @@ -35,9 +35,9 @@ def test_filter_plugin_entries_enabled_only(): def test_filter_plugin_entries_no_bundled(): entries = [ - ("disk-cleanup", "2.0.0", "Bundled", "bundled", None), - ("drawthings-grpc", "0.3.0", "Draw Things", "user", None), - ("web-search-plus", "2.2.0", "Search", "git", None), + ("disk-cleanup", "2.0.0", "Bundled", "bundled", None, "disk-cleanup"), + ("drawthings-grpc", "0.3.0", "Draw Things", "user", None, "drawthings-grpc"), + ("web-search-plus", "2.2.0", "Search", "git", None, "web-search-plus"), ] filtered = plugins_cmd._filter_plugin_entries( @@ -52,8 +52,8 @@ def test_filter_plugin_entries_no_bundled(): def test_cmd_list_plain_compact_output(monkeypatch, capsys): entries = [ - ("disk-cleanup", "2.0.0", "Bundled", "bundled", None), - ("web-search-plus", "2.2.0", "Search", "git", None), + ("disk-cleanup", "2.0.0", "Bundled", "bundled", None, "disk-cleanup"), + ("web-search-plus", "2.2.0", "Search", "git", None, "web-search-plus"), ] monkeypatch.setattr(plugins_cmd, "_discover_all_plugins", lambda: entries) monkeypatch.setattr(plugins_cmd, "_get_enabled_set", lambda: {"web-search-plus"}) @@ -69,7 +69,7 @@ def test_cmd_list_plain_compact_output(monkeypatch, capsys): def test_cmd_list_json_output(monkeypatch, capsys): - entries = [("web-search-plus", "2.2.0", "Search", "git", None)] + entries = [("web-search-plus", "2.2.0", "Search", "git", None, "web-search-plus")] monkeypatch.setattr(plugins_cmd, "_discover_all_plugins", lambda: entries) monkeypatch.setattr(plugins_cmd, "_get_enabled_set", lambda: {"web-search-plus"}) monkeypatch.setattr(plugins_cmd, "_get_disabled_set", lambda: set())