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

🐛 Also produce null-prototype at root level of generated object when requested to #4502

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

dubzzz
Copy link
Owner

@dubzzz dubzzz commented Dec 1, 2023

Our current implementation of fc.object with withNullPrototype: true was not producing any null-prototype instances at the root. With that fix, it will now produce some.

As the change fixes a slightly minor bug that could cause replays to break (the version including it, would not replay to the same values), it might not be included into a patch but wait for a minor).

Category:

  • ✨ Introduce new features
  • 📝 Add or update documentation
  • ✅ Add or update tests
  • 🐛 Fix a bug
  • 🏷️ Add or update types
  • ⚡️ Improve performance
  • Other(s): ...

Potential impacts:

  • Generated values
  • Shrink values
  • Performance
  • Typings
  • Other(s): ...

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…n requested to

Our current implementation of `fc.object` with `withNullPrototype: true` was not producing any null-prototype instances at the root. With that fix, it will now produce some.

As the change fixes a slightly minor bug that could cause replays to break (the version including it, would not replay to the same values), it might not be included into a patch but wait for a minor).
Copy link

codesandbox-ci bot commented Dec 1, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 953ea45:

Sandbox Source
@fast-check/examples Configuration

Copy link

codecov bot commented Dec 1, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (98da675) 93.57% compared to head (953ea45) 93.57%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4502   +/-   ##
=======================================
  Coverage   93.57%   93.57%           
=======================================
  Files         207      207           
  Lines        5013     5013           
  Branches     1368     1368           
=======================================
  Hits         4691     4691           
  Misses        322      322           
Flag Coverage Δ
unit-tests 93.57% <ø> (ø)
unit-tests-18.x-Linux 93.57% <ø> (ø)
unit-tests-20.x-Linux 93.57% <ø> (ø)
unit-tests-latest-Linux 93.57% <ø> (ø)

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.

@dubzzz dubzzz merged commit bf0933a into main Dec 1, 2023
@dubzzz dubzzz deleted the fix-obj-null-proto branch December 1, 2023 23:05
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.

None yet

1 participant