Skip to content

tests: fix tests - #14

Open
lecaros wants to merge 1 commit into
masterfrom
lecaros-fix-tests
Open

tests: fix tests#14
lecaros wants to merge 1 commit into
masterfrom
lecaros-fix-tests

Conversation

@lecaros

@lecaros lecaros commented Jul 21, 2026

Copy link
Copy Markdown

Fix broken test suite

The test suite was failing for two reasons: it couldn't build on Apple Silicon, and the golden outputs were out of date and non-reproducible after the gofakeit v6 upgrade.

Changes:

  • Removed the bou.ke/monkey dependency (amd64-only, blocked tests from building on ARM). It was only used to freeze time.Now in tests — the generators already accept a timestamp parameter, so tests now pass the fixed time directly.
  • Made test output deterministic again by seeding gofakeit's global faker (gofakeit.Seed(11)) in addition to math/rand. Since v6, gofakeit uses its own randomly-seeded source, so seeding math/rand alone no longer produced stable output.
  • Regenerated all golden // Output: blocks to match the current gofakeit version.
  • Updated CLAUDE.md test notes accordingly.

go test ./... now passes on both arm64 and amd64, and go vet is clean. No changes to the CLI or generated log formats.

Signed-off-by: lecaros <lecaros@chronosphere.io>
@lecaros
lecaros requested a review from cosmo0920 July 21, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant