Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix: Handle DOMException error in Ava tests #3370

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

Conversation

Prasanth2706
Copy link

This PR fixes an issue where DOMException was not correctly recognized as an error in the test.

  • Updated the test case to ensure DOMException is properly caught.
  • Verified the fix by running npm test and confirming all tests pass.
  • No breaking changes introduced.

@sholladay
Copy link

I was hoping this was a fix for #3367. But this PR doesn't actually change any behavior, right?

@novemberborn
Copy link
Member

@Prasanth2706 I don't think you pushed up all the changes, there's some submodule showing in the PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants