Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add fuzzing #83

Merged
merged 5 commits into from
Oct 15, 2023
Merged

Add fuzzing #83

merged 5 commits into from
Oct 15, 2023

Conversation

martincostello
Copy link
Owner

Add some basic fuzzing of NuGetVersion.tryParse() with fast-check.

Add some basic fuzzing of `NuGetVersion.tryParse()` with fast-check.
@martincostello martincostello added enhancement New feature or request dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Oct 15, 2023
@github-actions
Copy link

Hi @martincostello 👋

It looks like this pull request makes changes which require the contents of dist to be updated, but these files have not been included with your changes.

Please run build.ps1 and commit the changes made to the dist directory to your branch and push them to this pull request.

@codecov
Copy link

codecov bot commented Oct 15, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (2af8663) 81.62% compared to head (b0b6945) 81.62%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   81.62%   81.62%           
=======================================
  Files           5        5           
  Lines         283      283           
  Branches       77       77           
=======================================
  Hits          231      231           
  Misses         40       40           
  Partials       12       12           
Flag Coverage Δ
linux 81.62% <ø> (ø)
macos 81.62% <ø> (ø)
windows ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Update the contents of dist.
Fix the constraints for the fuzz tests so not generate valid values.
Assert that no inputs throw an exception.
- Run prettier.
- Rename parameters.
@martincostello martincostello enabled auto-merge (squash) October 15, 2023 13:36
@martincostello martincostello merged commit 163e794 into main Oct 15, 2023
10 checks passed
@martincostello martincostello deleted the add-fast-check branch October 15, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant