-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fix failing UI tests after React Native upgrade in Gutenberg #21143
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
Fix failing UI tests after React Native upgrade in Gutenberg #21143
Conversation
|
| App Name | Jetpack Alpha |
|
| Configuration | Release-Alpha | |
| Build Number | pr21143-4530832 | |
| Version | 22.8 | |
| Bundle ID | com.jetpack.alpha | |
| Commit | 4530832 | |
| App Center Build | jetpack-installable-builds #5459 |
|
| App Name | WordPress Alpha |
|
| Configuration | Release-Alpha | |
| Build Number | pr21143-4530832 | |
| Version | 22.8 | |
| Bundle ID | org.wordpress.alpha | |
| Commit | 4530832 | |
| App Center Build | WPiOS - One-Offs #6431 |
|
Thanks @mokagio for the fix 🙇 ! I'm a bit hesitant about this workaround because typing should be a testable action. However, I tried the approaches shared by @jostnes here but none have worked when running the tests locally 🤔. Hence, I agree with going with this approach for now, especially for the sake of unblocking the React Native upgrade. We could review later on a better approach. |
fluiddot
left a comment
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 🎊 !
- Tests are passing in CI ✅
- Running editor tests locally in Xcode succeed ✅


Implements a working albeit not elegant workaround to the issue of UI tests not being able to get a handle on the keyboard when attempting to insert text in the title and paragraph blocks in the Gutenberg editor built and integrated via XCFramework.
As you can see, CI is green here, while it's red in the base branch.
Based on #21134. In that PR, @jostnes made a few suggestions of approaches to try. I haven't had a chance to do so yet. If you trust I'll followup on the approaches, I'd rather merge with the tests green as is sooner rather than later. 😅
Regression Notes
PR submission checklist:
RELEASE-NOTES.txtif necessary. N.A.UI changes testing checklist: Not a UI PR.