Skip to content

[Refactor] Deduplicate test helpers#45

Merged
samzong merged 1 commit into
mainfrom
refactor/test-helper-cleanup
Jun 9, 2026
Merged

[Refactor] Deduplicate test helpers#45
samzong merged 1 commit into
mainfrom
refactor/test-helper-cleanup

Conversation

@samzong

@samzong samzong commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add shared JSON golden helpers in internal/testutil for formatted golden assertions and sorted rawir.RawModule output.
  • Use those helpers in backend and normalize golden tests.
  • Add local render test fixture helpers for temporary Go module setup and generated output reads.

Verification

  • python3 /Users/x/.agents/skills/codex-review/scripts/run_codex_review.py --uncommitted
  • make check

Compatibility

None. This is test-only cleanup and does not change generated command shape, catalog schema, auth/body/output behavior, or docs.

Checklist

  • Tests or focused verification cover the changed surface.
  • No user-facing behavior changes.
  • Generated output under internal/generated/, .cache/, and ad-hoc skills/<cli-name>/ directories is not committed.
  • Commits are signed off when this is ready to merge.

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong samzong merged commit ba11daf into main Jun 9, 2026
3 checks passed
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