Skip to content

fix(react): properly check for custom elements to avoid errors in unit tests #24156

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

Merged

Conversation

anantsharma1310
Copy link
Contributor

@anantsharma1310 anantsharma1310 commented Nov 3, 2021

Fix for custom elements issue while running the unit tests in ReactJS.

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Build (npm run build) was run locally and any changes were pushed
  • Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

React unit tests were breaking due to Ion Tabs (Test env issue).

resolves #24149

What is the new behavior?

React unit tests can run without an issue.

Does this introduce a breaking change?

  • Yes
  • No

Other information

NA

Sorry, something went wrong.

@liamdebeasi liamdebeasi changed the title FIX: Ion tabs custom elements fix(react): properly check for custom elements to avoid errors in unit tests Dec 6, 2021
@liamdebeasi liamdebeasi merged commit 8f188ea into ionic-team:main Dec 6, 2021
@liamdebeasi
Copy link
Contributor

Merged. Thank you!

@anantsharma1310
Copy link
Contributor Author

Thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: react @ionic/react package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: "CustomElements not defined" while executing the unit tests.
2 participants