Skip to content

Conversation

@westonruter
Copy link
Member

@westonruter westonruter commented Jun 10, 2021

Summary

This fixes tests which were broken with Gutenberg v10.8.0 (and WP Core 5.8-beta1) due to WordPress/gutenberg#32370 and WordPress/wordpress-develop#1275, accounting for Custom Logo now being stored in an option in addition to a theme mod.

Also fixes test_wrap_widget_callbacks after changes to default widgets registered. See WordPress/wordpress-develop@8add05f in https://core.trac.wordpress.org/ticket/53324.

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 this to the v2.1.3 milestone Jun 10, 2021
@westonruter westonruter enabled auto-merge June 10, 2021 04:23
@westonruter westonruter disabled auto-merge June 10, 2021 04:23
@westonruter westonruter enabled auto-merge (squash) June 10, 2021 04:24
@westonruter westonruter requested a review from pierlon June 10, 2021 04:24
@github-actions
Copy link
Contributor

github-actions bot commented Jun 10, 2021

Plugin builds for 8b58a9b are ready 🛎️!

@westonruter westonruter changed the title Fix test_amp_get_schemaorg_metadata to account for site logo in option Fix test_amp_get_schemaorg_metadata and test_wrap_widget_callbacks in Gutenberg v10.8 and WP 5.8 Jun 10, 2021
@westonruter westonruter disabled auto-merge June 10, 2021 04:44
@westonruter westonruter force-pushed the fix/test_amp_get_schemaorg_metadata branch from 34b794f to 8b58a9b Compare June 10, 2021 05:47
@westonruter westonruter merged commit 8fa2fab into develop Jun 10, 2021
@westonruter westonruter deleted the fix/test_amp_get_schemaorg_metadata branch June 10, 2021 05:54
westonruter added a commit that referenced this pull request Jun 10, 2021
@westonruter westonruter self-assigned this Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants