Skip to content

Commit 440b192

Browse files
committed
Fix test_analysis_suggestions for pmui < 0.10 fallback
When footer_actions is unavailable, follow-up icons share footer_objects with analysis suggestions, so skip the separation assertion.
1 parent e839a90 commit 440b192

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

lumen/tests/ai/test_ui.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1972,8 +1972,10 @@ async def test_analysis_suggestions_still_use_default_group_name(explorer_ui):
19721972
followup_icons = [f for f in footer_actions if getattr(f, 'name', '') == "FollowUp"]
19731973
assert len(followup_icons) == 1
19741974

1975-
# Analysis suggestions (if any) still go in footer_objects under "Suggestions"
1976-
# The key point: follow-up and analysis suggestions use separate containers
1977-
footer = last_msg.footer_objects or []
1978-
followup_in_footer = [f for f in footer if getattr(f, 'name', '') == "FollowUp"]
1979-
assert len(followup_in_footer) == 0
1975+
# When footer_actions is available (pmui >= 0.10), follow-up icons live there
1976+
# and footer_objects is reserved for analysis suggestions.
1977+
# When footer_actions is unavailable, both share footer_objects.
1978+
if hasattr(last_msg, "footer_actions"):
1979+
footer = last_msg.footer_objects or []
1980+
followup_in_footer = [f for f in footer if getattr(f, 'name', '') == "FollowUp"]
1981+
assert len(followup_in_footer) == 0

0 commit comments

Comments
 (0)