tecnickcom / gogen / 28867038265

100%

push

github

fix(testutil): harden timestamp regexes, output capture, and error mocks

- ReplaceDateTime: bound the match to the RFC3339 grammar (date, "T", time, optional fractional seconds and timezone offset) so it no longer greedily consumes across newlines or depends on a trailing quote; the previous `[^"]*` tail could silently clobber unrelated content in multi-line or unquoted input.
- ReplaceUnixTimestamp: match a word-bounded 19-digit integer (`\b[0-9]{19}\b`) so numbers longer than 19 digits are no longer half-replaced into corrupted values; document that it is a digit-count heuristic that also matches unrelated 19-digit identifiers.
- CaptureOutput: save and restore the standard logger's real destination via log.Writer() (was unconditionally reset to stderr), remove the no-op sync.WaitGroup, close both pipe ends on every exit path to avoid a goroutine/FD leak on panic, and document that it mutates process-global state and must not be used under t.Parallel().
- NewErrorCloser: return the concrete *ErrorCloser with a prebuilt error and embed io.Reader instead of *bytes.Reader, so the exported surface is exactly io.ReadCloser.
- tests: assert stdout, stderr, and global-state restoration in CaptureOutput; assert ErrorCloser.Read returns io.EOF; make the time helpers table-driven with regression cases for the multi-line datetime and 20-digit timestamp footguns.

Keeps pkg/testutil at 100% statement coverage with a clean golangci-lint run.

22 of 22 new or added lines in 2 files covered. (100.0%)

13857 of 13861 relevant lines covered (99.97%)

900.08 hits per line