From 0e473343d048f6a45812fc68dc7dd560654dd0e5 Mon Sep 17 00:00:00 2001 From: hz Date: Fri, 24 Jul 2026 23:17:59 -0500 Subject: [PATCH] Fix stale web_responsive_test icon assertion It checked for a "DynamicQuestionsIcon" mode card that no longer exists in study_selector.gd (that mode moved to its own nav item and screen a while back), so it always failed. CCS Cases is now also UAT-gated and won't be present in the default test environment either, so point the check at Question Bank, the one mode card that's always rendered. --- tests/web_responsive_test.gd | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/web_responsive_test.gd b/tests/web_responsive_test.gd index e7a46b1..22e4065 100644 --- a/tests/web_responsive_test.gd +++ b/tests/web_responsive_test.gd @@ -28,10 +28,10 @@ func _run() -> void: for row in selector.responsive_card_rows: _assert(row.vertical, "phone-width card content should stack instead of clipping") _assert(selector.mode_view.find_children("*", "RichTextLabel", true, false).is_empty(), "study modes should not render the legacy rich-text simulation note") - var dynamic_icon := selector.mode_view.find_child("DynamicQuestionsIcon", true, false) as TextureRect - _assert(dynamic_icon != null, "dynamic questions should expose its mode icon") - if dynamic_icon != null: - _assert(dynamic_icon.texture.resource_path == "res://assets/icons/lucide/activity.svg", "dynamic questions should use a web-safe vector icon") + var question_bank_icon := selector.mode_view.find_child("QuestionBankIcon", true, false) as TextureRect + _assert(question_bank_icon != null, "question bank should expose its mode icon") + if question_bank_icon != null: + _assert(question_bank_icon.texture.resource_path == "res://assets/icons/lucide/clipboard-list.svg", "question bank should use a web-safe vector icon") selector._show_stage("question_bank") await process_frame _assert(selector.question_bank_launch_view.visible and not selector.mode_view.visible, "direct question bank navigation should show only the focused launch state")