Skip to content

Add fixture-based and live integration tests#40

Merged
jdidion merged 1 commit intomainfrom
integration-tests
Apr 17, 2026
Merged

Add fixture-based and live integration tests#40
jdidion merged 1 commit intomainfrom
integration-tests

Conversation

@jdidion
Copy link
Copy Markdown
Owner

@jdidion jdidion commented Apr 17, 2026

Note: This MR was largely generated by Claude and has not been completely reviewed by me (the human). You should feel free to defer your review until this warning has been removed.

Summary

Large-scale integration tests using real API response data.

Fixture tests (10, run in CI)

Wiremock serves real JSON from GitHub/GitLab APIs:

  • ripgrep#2900 (S): PR metadata, 2 commits, 1 file with patches, 3 review comments
  • ruff#16000 (M): 13 files, 3 threaded review comments
  • gitlab-org/gitlab!229380 (M): 8 changed files, 2 discussion threads with replies

Live smoke tests (3, --ignored)

Hit real APIs to verify end-to-end:

Run with: cargo test -- --ignored

Test plan

  • 527 tests pass (10 new fixture + existing)
  • 3 live smoke tests correctly ignored in normal runs

🤖 Generated with Claude Code

10 fixture tests using real API responses served via wiremock:
- GitHub: ripgrep#2900 (PR metadata, commits, files, comments)
- GitHub: ruff#16000 (medium PR with 13 files, threaded comments)
- GitLab: gitlab-org/gitlab!229380 (MR metadata, commits, changes, discussions)

3 live smoke tests (gated behind #[ignore]):
- GitHub: ripgrep#2900 (small), rust-lang/rust#128440 (XL, 666 files)
- GitLab: gitlab-org/gitlab!229380

Fixtures stored in tests/fixtures/ as real JSON from GitHub/GitLab APIs.
Run live tests with: cargo test -- --ignored

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jdidion jdidion merged commit 226041e into main Apr 17, 2026
1 check passed
@jdidion jdidion deleted the integration-tests branch April 17, 2026 12:06
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