-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[E2E Tests]: Fix WP editor metabox test #32915
Conversation
Size Change: 0 B Total Size: 1.04 MB ℹ️ View Unchanged
|
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.
Thank you for investigating this test failure. I think is fine to update the test with how it works, but we need to figure out what has changed in the WordPress core that triggered this change in behavior.
@youknowriad or @noisysocks, any ideas?
So it looks like the wrapping |
443ed94
to
d08438e
Compare
Description
toMatchSnapshot
withtoMatchInlineSnapshot
to make tests more readable.I did manual testing and always saw wrapping
<p>
when getting value from TinyMCE editor.How has this been tested?
Types of changes
Bugfix
Checklist:
*.native.js
files for terms that need renaming or removal).