Problem
`mirrorstack_test.go` defines an `assertPanics(t, fn, msg)` helper. Its docstring claims it consolidates the `recover()` pattern across test files.
In practice the helper is only usable within `package mirrorstack`. Other test packages still inline the same recover pattern:
- `internal/registry/registry_test.go` — 6 inline `recover()` blocks
- A few call sites in `mirrorstack_test.go` itself
Fix
Create `internal/testutil/panic.go` exposing `AssertPanics(t *testing.T, fn func(), msgFmt string, args ...any)`. Migrate inline `recover()` blocks. Update or remove the overclaim in the current `assertPanics` docstring.
Effort
~30 minutes. Purely test-code refactor, no behavior change.
Flagged during /simplify on PR #87.
Problem
`mirrorstack_test.go` defines an `assertPanics(t, fn, msg)` helper. Its docstring claims it consolidates the `recover()` pattern across test files.
In practice the helper is only usable within `package mirrorstack`. Other test packages still inline the same recover pattern:
Fix
Create `internal/testutil/panic.go` exposing `AssertPanics(t *testing.T, fn func(), msgFmt string, args ...any)`. Migrate inline `recover()` blocks. Update or remove the overclaim in the current `assertPanics` docstring.
Effort
~30 minutes. Purely test-code refactor, no behavior change.
Flagged during /simplify on PR #87.