From 35ce94a2f8ae37bd74b10bcc86c75a7ab2e205d1 Mon Sep 17 00:00:00 2001
From: teknium1 <127238744+teknium1@users.noreply.github.com>
Date: Wed, 13 May 2026 22:53:50 -0700
Subject: [PATCH] fix(tests): correct skin engine test API call
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The salvaged regression test called skin.get_spinner_list() which
doesn't exist on SkinConfig. Replace with direct dict access on
skin.spinner — same intent (verify default empty spinner is preserved
when user override is invalid).
---
 tests/hermes_cli/test_skin_engine.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/hermes_cli/test_skin_engine.py b/tests/hermes_cli/test_skin_engine.py
index 9da6df2bc28..1ed7e35323b 100644
--- a/tests/hermes_cli/test_skin_engine.py
+++ b/tests/hermes_cli/test_skin_engine.py
@@ -226,7 +226,7 @@ class TestUserSkins:
         assert skin.name == "broken"
         assert skin.get_color("banner_title") == "#FFD700"
         assert skin.get_branding("agent_name") == "Hermes Agent"
-        assert skin.get_spinner_list("waiting_faces") == []
+        assert skin.spinner.get("waiting_faces", []) == []
         assert skin.tool_emojis == {}
         assert skin.tool_prefix == "!"