Skip to content

Improve caching and logic in get_user_data_from_wp_global_styles. #2414

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.

Sign up for GitHub

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

Closed
spacedmonkey wants to merge 15 commits into from
Closed

Improve caching and logic in get_user_data_from_wp_global_styles. #2414

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
075d00a
Improve caching of global styles.
spacedmonkey Jun 13, 2022
2da31ee
Merge branch 'trunk' into fix/global-query
spacedmonkey Jun 17, 2022
83f9579
Add some unit tests.
spacedmonkey Jun 17, 2022
82ffd45
Improve caching.
spacedmonkey Jun 17, 2022
95667ce
Handle more edge cases.
spacedmonkey Jun 17, 2022
71bcb6c
Improve tests.
spacedmonkey Jun 17, 2022
478fa51
Merge branch 'trunk' into fix/global-query
spacedmonkey Jul 5, 2022
8bb1cd7
Add expiration back again.
spacedmonkey Jul 5, 2022
32d7306
Fix style.
spacedmonkey Jul 5, 2022
7482d90
Merge branch 'trunk' into fix/global-query
spacedmonkey Sep 9, 2022
12e6322
Fix test.
spacedmonkey Sep 9, 2022
67537a2
Apply suggestions from code review
spacedmonkey Sep 15, 2022
0389421
Merge branch 'trunk' into fix/global-query
spacedmonkey Sep 15, 2022
76fd86c
Formatting of tests.
spacedmonkey Sep 15, 2022
8e567de
Fix styling.
spacedmonkey Sep 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Formatting of tests.
  • Loading branch information
@spacedmonkey
spacedmonkey committed Sep 15, 2022
commit 76fd86cd60ac3d621828c25ec09d71e94f84e7f0
31 changes: 19 additions & 12 deletions tests/phpunit/tests/theme/wpThemeJsonResolver.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -352,25 +352,26 @@ function test_merges_child_theme_json_into_parent_theme_json() {
* @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
*/
function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries() {
WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme() );
$theme = wp_get_theme();
WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
add_filter( 'query', array( $this, 'filter_db_query' ) );
$query_count = count( $this->queries );
for ( $i = 0; $i < 3; $i++ ) {
WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme() );
WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
WP_Theme_JSON_Resolver::clean_cached_data();
}
$query_count = count( $this->queries ) - $query_count;
$this->assertEquals( 0, $query_count, 'Unexpected SQL queries detected for the wp_global_style post type' );

$user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme() );
$user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
$this->assertEmpty( $user_cpt );

$user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme(), true );
$user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, true );
$this->assertNotEmpty( $user_cpt );

$query_count = count( $this->queries );
for ( $i = 0; $i < 3; $i ++ ) {
$new_user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme() );
$new_user_cpt = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
WP_Theme_JSON_Resolver::clean_cached_data();
$this->assertSameSets( $user_cpt, $new_user_cpt );
}
Expand All @@ -381,43 +382,49 @@ function test_get_user_data_from_wp_global_styles_does_not_use_uncached_queries(

/**
* @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
*
* @ticket 55392
*/
function test_get_user_data_from_wp_global_styles_does_exist() {
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme(), true );
$theme = wp_get_theme();
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, true );
$this->assertIsArray( $post1 );
$this->assertArrayHasKey( 'ID', $post1 );
wp_delete_post( $post1['ID'], true );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme(), true );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, true );
$this->assertIsArray( $post2 );
$this->assertArrayHasKey( 'ID', $post2 );
}

/**
* @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
*
* @ticket 55392
*/
function test_get_user_data_from_wp_global_styles_create_post() {
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme( 'testing' ) );
$theme = wp_get_theme( 'testing' );
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
$this->assertIsArray( $post1 );
$this->assertSameSets( array(), $post1 );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme( 'testing' ) );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme );
$this->assertIsArray( $post2 );
$this->assertSameSets( array(), $post2 );
$post3 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme( 'testing' ), true );
$post3 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, true );
$this->assertIsArray( $post3 );
$this->assertArrayHasKey( 'ID', $post3 );
}

/**
* @covers WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles
*
* @ticket 55392
*/
function test_get_user_data_from_wp_global_styles_filter_state() {
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme( 'foo' ), true, array( 'publish' ) );
$theme = wp_get_theme( 'foo' );
$post1 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, true, array( 'publish' ) );
$this->assertIsArray( $post1 );
$this->assertArrayHasKey( 'ID', $post1 );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( wp_get_theme( 'foo' ), false, array( 'draft' ) );
$post2 = WP_Theme_JSON_Resolver::get_user_data_from_wp_global_styles( $theme, false, array( 'draft' ) );
$this->assertIsArray( $post2 );
$this->assertSameSets( array(), $post2 );
}
Expand Down