Skip to content

✅ test(parser): add insta snapshot coverage#153

Merged
zrr1999 merged 2 commits into
mainfrom
codex/add-insta-snapshot-tests
Jun 17, 2026
Merged

✅ test(parser): add insta snapshot coverage#153
zrr1999 merged 2 commits into
mainfrom
codex/add-insta-snapshot-tests

Conversation

@zrr1999

@zrr1999 zrr1999 commented Jun 16, 2026

Copy link
Copy Markdown
Member

Summary

  • Enable insta as a workspace testing dependency.
  • Add insta to sporec-parser dev-dependencies.
  • Add a parser AST snapshot test that captures core language surface syntax and commits the generated baseline snapshot.

Validation

  • INSTA_UPDATE=always cargo test -p sporec-parser core_surface_ast_snapshot
  • cargo fmt --all -- --check
  • cargo test -p sporec-parser
  • cargo clippy -p sporec-parser --all-targets -- -D warnings
  • GitHub Actions passed Rust checks on head 97db8c7: test, msrv, prek, and wheel jobs.

Notes

  • The previous PR metadata used a non-template Impact heading and a [codex] title prefix, which caused the PR-only CI checks to fail.
  • No code changes were needed for this CI fix.

@zrr1999 zrr1999 changed the title [codex] Add parser insta snapshot coverage ✅ test(parser): add insta snapshot coverage Jun 17, 2026
@zrr1999
zrr1999 marked this pull request as ready for review June 17, 2026 03:26
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@zrr1999
zrr1999 merged commit 4a386b7 into main Jun 17, 2026
12 checks passed
@zrr1999
zrr1999 deleted the codex/add-insta-snapshot-tests branch June 17, 2026 08:44
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