Skip to content

Test that add_theme_supports are loaded for themes without theme.json#34998

Merged
oandregal merged 2 commits intotrunkfrom
add/tests-to-resolver
Sep 21, 2021
Merged

Test that add_theme_supports are loaded for themes without theme.json#34998
oandregal merged 2 commits intotrunkfrom
add/tests-to-resolver

Conversation

@oandregal
Copy link
Copy Markdown
Member

Adds tests for #34955

How to test

Make sure the test passes in the current trunk:

  • npm run test-unit-php -- /var/www/html/wp-content/plugins/gutenberg/phpunit/class-wp-theme-json-resolver-test.php

Make sure the test would have failed before #34955:

  • git checkout -b test/before 4474d01bc88d7ee4edadbf3227852e43e141fb56 => checks out commit prior
  • npm run test-unit-php -- /var/www/html/wp-content/plugins/gutenberg/phpunit/class-wp-theme-json-resolver-test.php => the expected result is that the test fails.

@oandregal oandregal self-assigned this Sep 21, 2021
@oandregal oandregal added the Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json label Sep 21, 2021
Copy link
Copy Markdown
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@Mamaduka
Copy link
Copy Markdown
Member

I think the "React Native E2E Tests (Android)" check got stuck, and there's no way to cancel it.

@oandregal oandregal merged commit a2169df into trunk Sep 21, 2021
@oandregal oandregal deleted the add/tests-to-resolver branch September 21, 2021 14:37
@github-actions github-actions bot added this to the Gutenberg 11.6 milestone Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants