diff --git a/crates/benilla-app/src/ui_script/group_loot_tests.rs b/crates/benilla-app/src/ui_script/group_loot_tests.rs index 5339a002..62c75032 100644 --- a/crates/benilla-app/src/ui_script/group_loot_tests.rs +++ b/crates/benilla-app/src/ui_script/group_loot_tests.rs @@ -385,6 +385,20 @@ fn nothing_repaints_a_frame_that_opened_before_its_snapshot() { let mut s = setup(); load_group_loot(&s); + // What the label reads **untouched**, which is not nothing: `LootFrame.xml` authors + // `text="Item Name"` on `$parentName` as a design-time placeholder, and every copy of that + // file has it — the realm's `patch-4`, `patch.MPQ` and stock `interface.MPQ` alike. So an + // unpainted name is the placeholder, and "stays blank" means **unchanged**, not nil. Read + // rather than written here, so the realm owns its own placeholder. + let unpainted = s + .eval::>("return GroupLootFrame1Name:GetText()") + .unwrap(); + assert!( + unpainted.is_some(), + "the reference authors a placeholder on this label; if that ever stops being true this \ + test's premise is what changed" + ); + // The order the app must never produce: the event first, against a model with no such roll. s.fire_event( "START_LOOT_ROLL", @@ -397,7 +411,7 @@ fn nothing_repaints_a_frame_that_opened_before_its_snapshot() { assert_eq!( s.eval::>("return GroupLootFrame1Name:GetText()") .unwrap(), - None, + unpainted, "nothing to paint yet — this is the reported state" ); @@ -408,8 +422,8 @@ fn nothing_repaints_a_frame_that_opened_before_its_snapshot() { assert_eq!( s.eval::>("return GroupLootFrame1Name:GetText()") .unwrap(), - None, - "the name stays blank: the reference has no repaint path" + unpainted, + "the name is unchanged: the reference has no repaint path" ); assert!( !s.eval::("return GroupLootFrame1Decoration:IsShown()") @@ -429,7 +443,7 @@ fn nothing_repaints_a_frame_that_opened_before_its_snapshot() { assert_eq!( s.eval::>("return GroupLootFrame1Name:GetText()") .unwrap(), - None + unpainted ); }