Skip to content

test: added 3 new test cases in the new describe named edge case - #1034

Merged
sonukapoor merged 1 commit into
OWASP:mainfrom
Vansh1419:test/baselineTestCoverage
Aug 21, 2026
Merged

sonukapoor merged 1 commit into
OWASP:mainfrom
Vansh1419:test/baselineTestCoverage

Conversation

@Vansh1419

Copy link
Copy Markdown
Contributor

What changed and why

Added 3 unit test in test/baseline.test.ts under new describe named edge cases

Closes #1030

Before

Test Suites: 6 failed, 108 passed, 114 total
Tests:       29 failed, 1417 passed, 1446 total
Snapshots:   0 total
Time:        3.115 s, estimated 4 s

After

Test Suites: 6 failed, 108 passed, 114 total
Tests:       29 failed, 1420 passed, 1449 total
Snapshots:   0 total
Time:        2.906 s, estimated 3 s

@Vansh1419
Vansh1419 requested a review from sonukapoor as a code owner August 21, 2026 07:26

@sonukapoor sonukapoor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good edge case coverage - all-new, partial match, and version-mismatch are exactly the three scenarios worth testing here. The Baseline type import was missing too, nice catch. Merging.

@sonukapoor
sonukapoor merged commit f20858e into OWASP:main Aug 21, 2026
6 checks passed
@sonukapoor

Copy link
Copy Markdown
Collaborator

Merged - thank you @Vansh1419!

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.

test: add edge-case coverage to tests/baseline.test.ts

2 participants