Skip to content
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 test to account for gutenberg_render_layout_support_flag() #6850

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

westonruter
Copy link
Member

Summary

In WordPress/gutenberg#37360 the gutenberg_render_layout_support_flag() function started injecting class names like wp-container-61e99d185b96f in the Columns block (as part of Gutenberg 12.3). This broke a unit test when for checking the expected output. So this PR just strips that class back out if it was added.

Checklist

  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@westonruter westonruter added the Testing Issues related with Unit, E2E, Smoke, and other testing requirements/needs label Jan 20, 2022
@westonruter westonruter added this to the v2.2.1 milestone Jan 20, 2022
@github-actions
Copy link
Contributor

Plugin builds for 825a555 are ready 🛎️!

@westonruter westonruter merged commit 6b06d30 into develop Jan 20, 2022
@westonruter westonruter deleted the fix/gutenberg-test-compat branch January 20, 2022 20:05
westonruter added a commit that referenced this pull request Jan 22, 2022
…r-amp-settings-panel

* 'develop' of github.com:ampproject/amp-wp:
  Render value as fallback instead of entire parent source object
  Fix logic inversion for sources.length
  Ensure `sources` is non-empty array
  Revert "Ensure error `sources` is never `null`"
  Ensure error `sources` is never `null`
  Ensure `sources` is iterated over only if it is non-empty array
  Fix test to account for gutenberg_render_layout_support_flag() (#6850)
  Support converting YouTube iframes with 100% width to amp-youtube with fixed-height (#6837)
  Update Gutenberg package dependencies
  Update Gutenberg package dependencies
  Allow composer normalize plugin to run
  Normalize Composer file
  Update Composer to use AMP toolbox v0.10.0
@westonruter westonruter added the Changelogged Whether the issue/PR has been added to release notes. label Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelogged Whether the issue/PR has been added to release notes. Testing Issues related with Unit, E2E, Smoke, and other testing requirements/needs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant