Changeset 55236 for trunk/tests/phpunit/tests/theme/wpTheme.php

Timestamp:

02/06/2023 07:57:41 PM (4 years ago)

Author:

flixos90

Message:

Themes: Add caching to WP_Theme::is_block_theme().

This changeset adds a block_theme entry in the theme cache data, similar to the existing entries headers, errors, stylesheet, and template.

Props spacedmonkey, costdev, joemcgill, flixos90, mukesh27, adamsilverstein.
Fixes #57114.

File:

• trunk/tests/phpunit/tests/theme/wpTheme.php (modified) (1 diff)

trunk/tests/phpunit/tests/theme/wpTheme.php

@@ -277 +277 @@
 
         /**
+         * @ticket 57114
+         *
+         * @covers WP_Theme::is_block_theme
+         *
+         * @dataProvider data_is_block_theme
+         */
+        public function test_is_block_theme_property( $theme_dir, $expected ) {
+                $theme = new WP_Theme( $theme_dir, $this->theme_root );
+                $theme->is_block_theme();
+                $reflection          = new ReflectionClass( $theme );
+                $reflection_property = $reflection->getProperty( 'block_theme' );
+                $reflection_property->setAccessible( true );
+
+                $this->assertSame( $expected, $reflection_property->getValue( $theme ) );
+        }
+
+        /**
+         * @ticket 57114
+         *
+         * @covers WP_Theme::is_block_theme
+         * @covers WP_Theme::cache_get
+         */
+        public function test_is_block_theme_check_cache() {
+                $filter = new MockAction();
+                add_filter( 'theme_file_path', array( $filter, 'filter' ) );
+
+                $theme1 = new WP_Theme( 'block-theme', $this->theme_root );
+                // First run.
+                $this->assertTrue( $theme1->is_block_theme(), 'is_block_theme should return true on first run' );
+
+                $theme2 = new WP_Theme( 'block-theme', $this->theme_root );
+                // Second run.
+                $this->assertTrue( $theme2->is_block_theme(), 'is_block_theme should return true on second run' );
+                $this->assertCount( 0, $filter->get_events(), 'Should only be 0, as second run should be cached' );
+        }
+
+        /**
+         * @ticket 57114
+         *
+         * @covers WP_Theme::is_block_theme
+         * @covers WP_Theme::cache_delete
+         */
+        public function test_is_block_theme_delete_cache() {
+                $filter = new MockAction();
+                add_filter( 'theme_file_path', array( $filter, 'filter' ) );
+
+                $theme = new WP_Theme( 'block-theme', $this->theme_root );
+                // First run.
+                $this->assertTrue( $theme->is_block_theme(), 'is_block_theme should return true on first run' );
+                // Clear cache.
+                $theme->cache_delete();
+                // Second run.
+                $this->assertTrue( $theme->is_block_theme(), 'is_block_theme should return true on second run' );
+                $this->assertCount( 2, $filter->get_events(), 'Should only be 4, as second run should not be cached' );
+        }
+
+        /**
          * Test get_files for an existing theme.
          *