Changeset 56042 for trunk/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php

Timestamp:

06/26/2023 07:55:28 PM (3 years ago)

Author:

flixos90

Message:

General: Introduce WP_DEVELOPMENT_MODE constant to signify context-specific development mode.

In recent releases, WordPress core added several instances of cache usage around specific files. While those caches are safe to use in a production context, in development certain nuances apply for whether or not those caches make sense to use. Initially, WP_DEBUG was used as a temporary workaround, but it was clear that a more granular method to signify a specific development mode was required: For example, caches around theme.json should be disabled when working on a theme as otherwise it would disrupt the theme developer's workflow, but when working on a plugin or WordPress core, this consideration does not apply.

This changeset introduces a WP_DEVELOPMENT_MODE constant which, for now, can be set to either "core", "plugin", "theme", or an empty string, the latter of which means no development mode, which is also the default. A new function wp_get_development_mode() is the recommended way to retrieve that configuration value.

With the new function available, this changeset replaces all existing instances of the aforementioned WP_DEBUG workaround to use wp_get_development_mode() with a more specific check.

Props azaozz, sergeybiryukov, peterwilsoncc, spacedmonkey.
Fixes #57487.

File:

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

trunk/tests/phpunit/tests/theme/wpGetGlobalStylesheet.php

@@ -222 +222 @@
 
     /**
+     * Tests that the function relies on the development mode for whether to use caching.
+     *
+     * @ticket 57487
+     */
+    public function test_caching_is_used_when_developing_theme() {
+        global $_wp_tests_development_mode;
+
+        $this->maybe_switch_theme( 'block-theme' );
+
+        // Store CSS in cache.
+        $css = '.my-class { display: block; }';
+        wp_cache_set( 'wp_get_global_stylesheet', $css, 'theme_json' );
+
+        // By default, caching should be used, so the above value will be returned.
+        $_wp_tests_development_mode = '';
+        $this->assertSame( $css, wp_get_global_stylesheet(), 'Caching was not used despite development mode disabled' );
+
+        // When the development mode is set to 'theme', caching should not be used.
+        $_wp_tests_development_mode = 'theme';
+        $this->assertNotSame( $css, wp_get_global_stylesheet(), 'Caching was used despite theme development mode' );
+    }
+
+    /**
      * Adds the 'editor-font-sizes' theme support with custom font sizes.
      *