Skip to content

test(review): cover empty Mistral key CLI failure - #42

Merged
pierrick-fonquerne merged 1 commit into
mainfrom
chore/ai-review-empty-key-integration
Aug 13, 2026
Merged

test(review): cover empty Mistral key CLI failure#42
pierrick-fonquerne merged 1 commit into
mainfrom
chore/ai-review-empty-key-integration

Conversation

@pierrick-fonquerne

Copy link
Copy Markdown
Contributor

What changed

Adds a local integration test that runs the review executable with a blank Mistral key and asserts the explicit failure contract.

Why

Prevents a regression where an invalid review configuration could be accepted by the executable path.

Validation

  • cargo test --manifest-path tools/ai-review/Cargo.toml --test empty_mistral_key
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace --all-features --no-fail-fast
  • cargo build --release

@pierrick-fonquerne
pierrick-fonquerne marked this pull request as ready for review August 13, 2026 22:30
@pierrick-fonquerne
pierrick-fonquerne merged commit 579ef55 into main Aug 13, 2026
5 checks passed
@pierrick-fonquerne
pierrick-fonquerne deleted the chore/ai-review-empty-key-integration branch August 13, 2026 22:30
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