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

2026-07-06T20:20:09Z

funman300 commented

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

Continues #118 after PR #124. The four remaining oversized plugin files become module directories with their trailing #[cfg(test)] blocks extracted verbatim into sibling tests.rs files (the replay_overlay/ pattern).

File	Before	mod.rs	tests.rs	Tests
hud_plugin	3,598	2,725	872	44
settings_plugin	3,512	2,857	654	25
game_plugin	2,562	1,310	1,251	39
input_plugin	2,540	1,832	707	31

Note: the issue suggested one file per PR, but these four are the identical mechanical transform already validated on card_plugin, so they're batched — each file is still an independent rename + block move in the diff.

git mv preserves history (53–82% rename similarity)
No runtime code changes; all 139 tests preserved
cargo clippy --workspace -- -D warnings clean, cargo test --workspace green

Issue #118 stays open for the deeper runtime splits along system boundaries.

Refs #118

🤖 Generated with Claude Code

Continues #118 after PR #124. The four remaining oversized plugin files become module directories with their trailing `#[cfg(test)]` blocks extracted verbatim into sibling `tests.rs` files (the `replay_overlay/` pattern). | File | Before | mod.rs | tests.rs | Tests | |---|---|---|---|---| | hud_plugin | 3,598 | 2,725 | 872 | 44 | | settings_plugin | 3,512 | 2,857 | 654 | 25 | | game_plugin | 2,562 | 1,310 | 1,251 | 39 | | input_plugin | 2,540 | 1,832 | 707 | 31 | Note: the issue suggested one file per PR, but these four are the identical mechanical transform already validated on card_plugin, so they're batched — each file is still an independent rename + block move in the diff. - `git mv` preserves history (53–82% rename similarity) - No runtime code changes; all 139 tests preserved - `cargo clippy --workspace -- -D warnings` clean, `cargo test --workspace` green Issue #118 stays open for the deeper runtime splits along system boundaries. Refs #118 🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

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

Continues #118 after card_plugin (PR #124): the four remaining
oversized plugin files become module directories with their trailing
#[cfg(test)] blocks extracted verbatim into sibling tests.rs files,
following the replay_overlay/ pattern.

  hud_plugin:      3,598 -> 2,725 + 872   (44 tests)
  settings_plugin: 3,512 -> 2,857 + 654   (25 tests)
  game_plugin:     2,562 -> 1,310 + 1,251 (39 tests)
  input_plugin:    2,540 -> 1,832 + 707   (31 tests)

Pure mechanical moves via git mv — no runtime code changes; all 139
tests preserved and passing.

Refs #118

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

funman300 merged commit b81b314197 into master

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

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.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: funman300/Ferrous-Solitaire#125