-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
test: Ignore local environment setup #55172
Conversation
Size Change: 0 B Total Size: 1.65 MB ℹ️ View Unchanged
|
Flaky tests detected in 4e111a0. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6469759333
|
Ah, good catch. Makes sense that the local environment setup only applies when using the development version of the RN bundle 👍 . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎊 !
Local environment setup configuration is intended for the development workflow. Applying it to the testing environment produced unexpected outcomes where one's local configuration can cause tests to fail unexpectedly.
3ac8200
to
4e111a0
Compare
What?
Do not apply local environment configuration added in #54957 to non-dev environments.
Why?
Local environment setup configuration is intended for the development
workflow. Applying it to the testing environment produced unexpected
outcomes where one's local configuration can cause tests to fail.
How?
Add a
__DEV__
conditional to the logic applying thesetup-local.js
configuration file.
Testing Instructions
Create a local file:
packages/react-native-editor/src/setup-local.js
Add the following example configuration to the file to clear out the Demo editor's initial HTML.
Example configuration
Verify the tests should pass on both platforms when running the following
commands.
Testing Instructions for Keyboard
n/a
Screenshots or screencast
n/a