Skip to content

test(e2e): add missing webhook tests #437

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

sahithvibudhi
Copy link

Description

Add missing webhook e2e tests

Related Issues

#274

How can it be tested?

npm run test

Check list before submitting

  • [x ] This PR is wrote in a clear language and correctly labeled
  • [ x] I created the related changeset for my changes with npx changeset
  • [x ] I have performed a self-review of my code (no debugs, no commented code, good naming, etc.)
  • [ x] I wrote the relative tests
  • [ x] I created a PR for the documentation if necessary and attached the link to this PR

Copy link

codecov bot commented Jun 23, 2025

Bundle Report

Bundle size has no change ✅

@sahithvibudhi
Copy link
Author

@brunobuddy can you review this when you get a chance?

Copy link

codecov bot commented Jun 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.44%. Comparing base (902c104) to head (184bef4).
Report is 12 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #437   +/-   ##
=======================================
  Coverage   89.44%   89.44%           
=======================================
  Files          58       58           
  Lines        1668     1668           
  Branches      385      376    -9     
=======================================
  Hits         1492     1492           
- Misses        155      170   +15     
+ Partials       21        6   -15     

☔ 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.

@sahithvibudhi
Copy link
Author

Since this change doesn't affect the published package, I’ve added an empty changeset to comply with contribution guidelines.

@sahithvibudhi
Copy link
Author

just circling back on this PR! curious if there’s been any internal discussion or leaning on whether it’s a candidate for merge? Happy to make changes if needed.

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.

1 participant