From 7254c777a7af8f4af5fe2912c63978a1ce3642d1 Mon Sep 17 00:00:00 2001 From: Simon Pinfold Date: Tue, 18 Aug 2026 12:26:43 -0700 Subject: [PATCH] Name the two whitespace positions the vocabulary test covers The loop's two values differ by where the space sits, which is the whole point of covering both; naming them puts that in the failure message instead of a comment above the loop. --- tests-unit/assets_test/test_semantics_reset.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests-unit/assets_test/test_semantics_reset.py b/tests-unit/assets_test/test_semantics_reset.py index dce153964..3c72e48ff 100644 --- a/tests-unit/assets_test/test_semantics_reset.py +++ b/tests-unit/assets_test/test_semantics_reset.py @@ -392,7 +392,7 @@ class TestTagReprojection: tags={"model_type:loras": "automatic"}, ) - for registered in (" loras", "loras "): + for position, registered in (("leading", " loras"), ("trailing", "loras ")): with patch( "app.assets.services.path_utils.get_comfy_models_folders", return_value=[ @@ -407,7 +407,7 @@ class TestTagReprojection: reproject_derived_state() assert "model_type:loras" not in _tags(session, "ref-1"), ( - f"a category registered as {registered!r} must not smuggle an " + f"{position} whitespace in {registered!r} must not smuggle an " "entry past the vocabulary and leave the stale tag in place" )