Skip to content

Conversation

@jsardev
Copy link
Owner

@jsardev jsardev commented Jan 27, 2026

Summary

  • migrate all tests to vitest
  • remove all old testing utilities and libraries
  • fix reaptcha/example build issues

Notes

Tests before:

index › should pass id
index › should pass className
index › should have default className
index › should execute render prop if passed
index › should render recaptcha
index › should render recaptcha on delayed grecaptcha load
index › should render invisible recaptcha
index › should render invisible recaptcha in dark mode
index › should render recaptcha with hl
index › should render recaptcha explicitly
index › should not render recaptcha when grecaptcha not available
index › should reset recaptcha
index › should execute recaptcha
index › should getResponse from recaptcha
index › should inject script
index › should not inject script
index › should not inject script if one with src https://google.com/recaptcha already present
index › should not inject script if one with src https://www.google.com/recaptcha already present
index › should not inject script if one with src https://recaptcha.net/recaptcha already present
index › should not inject script if one with src https://www.recaptcha.net/recaptcha already present
index › should not inject script if one with src https://gstatic.com/recaptcha already present
index › should not inject script if one with src https://www.gstatic.com/recaptcha already present
index › should inject script with hl parameter if specified
index › should inject script without hl parameter if not specified
index › should inject only one script on multiple instances
index › should call onLoad
index › should call onRender
index › should call onVerify
index › should call onExpire
index › should call onError
index › should throw error on double render
index › should throw error when no grecaptcha available on render
index › should throw error when recaptcha not rendered on reset
index › should throw error when trying to execute while not in invisible mode
index › should throw error when trying to execute while not rendered
index › should throw error when trying to getResponse while not rendered
index › should not throw error on unmount when not rendered
index › should derive invisible state from size prop
index › should rerender recaptcha when changing sitekey prop
index › should rerender recaptcha when changing theme prop
index › should rerender recaptcha when changing size prop
index › should rerender recaptcha when changing badge prop
index › should rerender recaptcha when changing tabindex prop
index › should rerender recaptcha when changing hl prop
index › should rerender recaptcha when changing isolated prop
utils › injectScript › should inject script
utils › injectScript › should not throw when document head not present
utils › isAnyScriptPresent › should return true
utils › isAnyScriptPresent › should return false

@codecov-commenter
Copy link

codecov-commenter commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (2f8a947) to head (8d4af1b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##           master      #271        +/-   ##
=============================================
+ Coverage        0   100.00%   +100.00%     
=============================================
  Files           0         3         +3     
  Lines           0       100       +100     
  Branches        0        23        +23     
=============================================
+ Hits            0       100       +100     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jsardev jsardev force-pushed the refactor/migrate-to-vitest branch 7 times, most recently from 61d4c7d to 85190bd Compare January 28, 2026 12:06
fix: fix coverage
@jsardev jsardev force-pushed the refactor/migrate-to-vitest branch from 85190bd to 8d4af1b Compare January 28, 2026 12:10
@changeset-bot
Copy link

changeset-bot bot commented Jan 28, 2026

⚠️ No Changeset found

Latest commit: 8d4af1b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@jsardev jsardev merged commit 1fb6515 into master Jan 28, 2026
3 checks passed
@jsardev jsardev deleted the refactor/migrate-to-vitest branch January 28, 2026 12:34
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.

3 participants