Changeset 55567

Timestamp:

03/20/2023 06:20:05 PM (2 years ago)

Author:

hellofromTonya

Message:

Tests: Add test class for wp_enqueue_stored_styles().

[54214] added the wp_enqueue_stored_styles() tests and clean_up_global_scope() reset global method to Tests_Theme_wpGetGlobalStylesheet.

This changeset relocates those tests a new test class specifically for wp_enqueue_stored_styles() and removes Tests_Theme_wpGetGlobalStylesheet::clean_up_global_scope() method.

Why not relocate the clean_up_global_scope() method to the new test class?
The test class extends from WP_Theme_UnitTestCase which includes this method.

Follow-up to [54214], [54703].

Props costdev.
See #57841.

Location:

trunk/tests/phpunit/tests/theme

Files:

• wpEnqueueStoredStyles.php (added)
• wpGetGlobalStylesheet.php (modified) (2 diffs)

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

@@ -55 +55 @@
     }
 
-    /**
-     * Cleans up global scope.
-     *
-     * @global WP_Styles $wp_styles
-     */
-    public function clean_up_global_scope() {
-        global $wp_styles;
-        parent::clean_up_global_scope();
-        $wp_styles = null;
-    }
-
     public function filter_set_theme_root() {
         return $this->theme_root;
@@ -211 +200 @@
     }
 
-    /**
-     * Tests that stored CSS is enqueued.
-     *
-     * @ticket 56467
-     *
-     * @covers ::wp_enqueue_stored_styles
-     */
-    public function test_should_enqueue_stored_styles() {
-        $core_styles_to_enqueue = array(
-            array(
-                'selector'     => '.saruman',
-                'declarations' => array(
-                    'color'        => 'white',
-                    'height'       => '100px',
-                    'border-style' => 'solid',
-                ),
-            ),
-        );
-
-        // Enqueues a block supports (core styles).
-        wp_style_engine_get_stylesheet_from_css_rules(
-            $core_styles_to_enqueue,
-            array(
-                'context' => 'block-supports',
-            )
-        );
-
-        $my_styles_to_enqueue = array(
-            array(
-                'selector'     => '.gandalf',
-                'declarations' => array(
-                    'color'        => 'grey',
-                    'height'       => '90px',
-                    'border-style' => 'dotted',
-                ),
-            ),
-        );
-
-        // Enqueues some other styles.
-        wp_style_engine_get_stylesheet_from_css_rules(
-            $my_styles_to_enqueue,
-            array(
-                'context' => 'my-styles',
-            )
-        );
-
-        wp_enqueue_stored_styles( array( 'prettify' => false ) );
-
-        $this->assertSame(
-            array( '.saruman{color:white;height:100px;border-style:solid;}' ),
-            wp_styles()->registered['core-block-supports']->extra['after'],
-            'Registered styles with handle of "core-block-supports" do not match expected value from Style Engine store.'
-        );
-
-        $this->assertSame(
-            array( '.gandalf{color:grey;height:90px;border-style:dotted;}' ),
-            wp_styles()->registered['wp-style-engine-my-styles']->extra['after'],
-            'Registered styles with handle of "wp-style-engine-my-styles" do not match expected value from the Style Engine store.'
-        );
-    }
     /**
      * Tests that switching themes recalculates the stylesheet.