Skip to content

test: catch 2 surviving mutants in bazel lexer#63

Merged
avrabe merged 1 commit intomainfrom
fix/mutation-testing
Mar 21, 2026
Merged

test: catch 2 surviving mutants in bazel lexer#63
avrabe merged 1 commit intomainfrom
fix/mutation-testing

Conversation

@avrabe
Copy link
Copy Markdown
Contributor

@avrabe avrabe commented Mar 21, 2026

0 missed mutants.

🤖 Generated with Claude Code

Catches both surviving mutants from cargo-mutants:
- lex_carriage_return_newlines: verifies \r\n produces Newline tokens
- lex_dot_token: verifies . produces Dot token

0 missed mutants remaining.

Refs: TEST-006

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@avrabe avrabe merged commit 39853d2 into main Mar 21, 2026
@avrabe avrabe deleted the fix/mutation-testing branch March 21, 2026 17:32
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Rivet Criterion Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 72473cc Previous: 39853d2 Ratio
store_lookup/100 2140 ns/iter (± 9) 1682 ns/iter (± 11) 1.27
store_lookup/1000 26716 ns/iter (± 182) 19329 ns/iter (± 71) 1.38
traceability_matrix/1000 59397 ns/iter (± 226) 41626 ns/iter (± 95) 1.43
diff/1000 694246 ns/iter (± 6896) 578432 ns/iter (± 28231) 1.20
query/100 801 ns/iter (± 2) 664 ns/iter (± 1) 1.21
query/1000 7466 ns/iter (± 40) 5566 ns/iter (± 24) 1.34

This comment was automatically generated by workflow using github-action-benchmark.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 21, 2026

Codecov Report

❌ Patch coverage is 91.30435% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rivet-core/src/bazel.rs 91.30% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

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