Skip to content

test(version): cover carries, glued pre-release words and || bounds - #156

Open
roxblnfk wants to merge 1 commit into
feat/own-constraint-matcherfrom
test/pr155-gaps
Open

roxblnfk wants to merge 1 commit into
feat/own-constraint-matcherfrom
test/pr155-gaps

Conversation

@roxblnfk

Copy link
Copy Markdown
Member

What was changed

  • Tests only, src untouched: carrying an incremented part of nines in ^, ~ and wildcards (9.*, ~9.9, ^0.99, 1.99.*, 1.19.*), a pre-release word glued to a binary's version (tool 1.2.3beta2, tool 2.0.0alpha), pre-release bounds next to || without spaces (^1.0-beta||^2.0, ^1.0||^2.0-beta.1), and the lowest stability when the least stable bound comes last (>=1.0-RC1 || ^2.0-alpha.1).

Why?

A mutation review of #155 found these branches unprotected. Every new test passes on the current code and fails on its mutant: the carry digits in Range::increment(), the letter check in BinaryVersion's lookahead, | in Constraint's bound and single-term checks, and picking the first bound instead of the lowest.

Checklist

  • Tested
    • Unit tests added
    • Full suite run locally

Pin the branches a mutation review found unprotected: carrying an
incremented part of nines (`9.*`, `~9.9`, `^0.99`, `1.99.*`), a
pre-release word glued to a binary's version (`1.2.3beta2`), pre-release
bounds next to `||` without spaces, and the lowest stability when the
least stable bound comes last.

Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the tests label Oct 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant