From 815a492fbdcce55a03eba40d1fee0fff5712c61b Mon Sep 17 00:00:00 2001 From: Natalie Date: Sun, 10 May 2026 20:44:06 -0700 Subject: [PATCH] =?UTF-8?q?fix(@projects/@magic-civilization):=20?= =?UTF-8?q?=F0=9F=90=9B=20remove=20move=20not-yet-implemented=20test=20stu?= =?UTF-8?q?b?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Lilith Autocommit --- .../crates/mc-player-api/src/dispatch.rs | 26 +++---------------- 1 file changed, 4 insertions(+), 22 deletions(-) diff --git a/src/simulator/crates/mc-player-api/src/dispatch.rs b/src/simulator/crates/mc-player-api/src/dispatch.rs index ea915b7f..5fe2cc07 100644 --- a/src/simulator/crates/mc-player-api/src/dispatch.rs +++ b/src/simulator/crates/mc-player-api/src/dispatch.rs @@ -549,28 +549,10 @@ mod tests { assert!(state.pending_pvp_attacks.is_empty()); } - #[test] - fn move_returns_not_yet_implemented_with_tracked_breadcrumb() { - let mut state = empty_state_with_one_unit(1); - let err = apply_action( - &mut state, - 0, - &PlayerAction::Move { - unit_id: "1".into(), - to: [2, 2], - }, - ) - .unwrap_err(); - match err { - ActionError::NotYetImplemented { message } => { - assert!( - message.contains("Move") && message.contains("p2-67"), - "breadcrumb missing: {message}" - ); - } - other => panic!("expected NotYetImplemented, got {other:?}"), - } - } + // Move dispatch landed in Wave 2 (trust-the-caller v1) — see + // `move_to_empty_hex_updates_unit_position_and_emits_unit_moved` and the + // related tests for the new behaviour. The old "returns NotYetImplemented" + // assertion was removed when the body switched from stub to live. #[test] fn city_op_returns_not_yet_implemented() {