Skip to content

chore: expand range parsing option coverage - #921

Merged
reggi merged 3 commits into
reggi/chore-adds-testsfrom
reggi/chore-expand-range-options
Oct 13, 2026
Merged

reggi merged 3 commits into
reggi/chore-adds-testsfrom
reggi/chore-expand-range-options

Conversation

@reggi

@reggi reggi commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #917, targeting reggi/chore-adds-tests so these tests can be merged into that PR independently of the performance fix.

Changes

  • Materialize the operator/precision fixtures as explicit default-options rows, retaining the original fixture cases.
  • Run every base row with default options, loose, includePrerelease, and both, using explicit expected-result overrides (1,392 fixtures total).
  • Preserve prerelease options when comparing canonical ranges and assert that all four option combinations cover the same input list.

Only test/fixtures/range-parse.js and test/classes/range.js are changed.

Validation

  • npm test passed, including coverage checks, repository-wide lint, and template-oss-check.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e40646ba-5ae5-433c-abed-2cdf7f08793f
@reggi
reggi requested a review from a team as a code owner October 13, 2026 19:56
reggi and others added 2 commits October 13, 2026 16:17
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e40646ba-5ae5-433c-abed-2cdf7f08793f
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e40646ba-5ae5-433c-abed-2cdf7f08793f
@reggi
reggi merged commit 6e8d7d1 into reggi/chore-adds-tests Oct 13, 2026
63 of 64 checks passed
@reggi
reggi deleted the reggi/chore-expand-range-options branch October 13, 2026 20:54
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.

2 participants