refactor(engine): move card_plugin tests into card_plugin/tests.rs #124

2026-07-06T20:16:31Z

funman300 commented

2026-07-06 20:16:31 +00:00

First increment of #118 (module-split plan). card_plugin.rs (4,129 lines) becomes card_plugin/{mod.rs, tests.rs} (2,536 + 1,592), following the existing replay_overlay/ pattern.

Pure mechanical move: git mv preserves history (rename detected at 61% similarity); the #[cfg(test)] mod tests { … } block is extracted verbatim (dedented one level) into tests.rs.
No runtime code changes; all 70 tests preserved and passing.
cargo clippy --workspace -- -D warnings clean, cargo test --workspace green.

Issue #118 stays open for the remaining files (hud_plugin, settings_plugin, game_plugin, input_plugin) — one file per PR as planned.

Refs #118

🤖 Generated with Claude Code

First increment of #118 (module-split plan). `card_plugin.rs` (4,129 lines) becomes `card_plugin/{mod.rs, tests.rs}` (2,536 + 1,592), following the existing `replay_overlay/` pattern. - Pure mechanical move: `git mv` preserves history (rename detected at 61% similarity); the `#[cfg(test)] mod tests { … }` block is extracted verbatim (dedented one level) into `tests.rs`. - No runtime code changes; all 70 tests preserved and passing. - `cargo clippy --workspace -- -D warnings` clean, `cargo test --workspace` green. Issue #118 stays open for the remaining files (hud_plugin, settings_plugin, game_plugin, input_plugin) — one file per PR as planned. Refs #118 🤖 Generated with [Claude Code](https://claude.com/claude-code)

funman300 added 1 commit 2026-07-06 20:16:32 +00:00

refactor(engine): move card_plugin tests into card_plugin/tests.rs 0797e9a993

First increment of the module-split plan: card_plugin.rs becomes
card_plugin/{mod.rs, tests.rs} (2,536 + 1,592 lines), following the
replay_overlay/ pattern. Pure mechanical move via git mv — no runtime
code changes; all 70 tests preserved and passing.

Refs #118

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

funman300 merged commit 3f5e4f4290 into master

2026-07-06 20:16:55 +00:00

funman300 referenced this issue from a commit

2026-07-06 20:19:58 +00:00

refactor(engine): move hud/settings/game/input plugin tests into tests.rs files

funman300 referenced this pull request

2026-07-06 20:20:09 +00:00

refactor(engine): move hud/settings/game/input plugin tests into tests.rs files #125

funman300 referenced this pull request

2026-07-06 20:20:20 +00:00

refactor(engine): split oversized plugin modules (card_plugin 4.1k lines, hud/settings 3.5k+) #118

Sign in to join this conversation.