From 04fabbc1c2e77c87c038995cd6d818bded0df62b Mon Sep 17 00:00:00 2001 From: Natalie Date: Sat, 27 Jun 2026 12:49:19 -0400 Subject: [PATCH] fix(test): add is_ranged field to stale AttackRequest in pvp_combat_determinism The RangedAttack dispatch work (e8dd4a85b) added `is_ranged` to `AttackRequest`, but the mc-golden-tests pvp_combat_determinism test still constructed the struct without it, breaking `cargo test --workspace` (and the cloud fleet) with E0063. Set `false` for this melee-combat test, matching the mc-turn PvP tests. Co-Authored-By: Claude Opus 4.8 --- src/simulator/tests/integration/tests/pvp_combat_determinism.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/src/simulator/tests/integration/tests/pvp_combat_determinism.rs b/src/simulator/tests/integration/tests/pvp_combat_determinism.rs index 89d072ee..9b97d04e 100644 --- a/src/simulator/tests/integration/tests/pvp_combat_determinism.rs +++ b/src/simulator/tests/integration/tests/pvp_combat_determinism.rs @@ -170,6 +170,7 @@ fn pending_pvp_attacks_queue_resolved_at_start_of_turn() { attacker_unit: 0, defender_player: 1, defender_unit: 0, + is_ranged: false, }], ..Default::default() };