Changeset 34816

Timestamp:

10/04/2015 04:43:46 AM (9 years ago)

Author:

DrewAPicture

Message:

Widgets: Add tests for dynamic_sidebar() where sidebars were registered with 1) no supplied ID, 2) a numeric ID, 3) a string ID.

See #17078.

File:

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

trunk/tests/phpunit/tests/widgets.php

@@ -91 +91 @@
         unregister_sidebar( $sidebar_id );
         $this->assertArrayNotHasKey( 'wp-unit-test', $wp_registered_sidebars );
+    }
+
+    /**
+     * Utility hook callback used to store a sidebar ID mid-function.
+     */
+    function retrieve_sidebar_id( $index ) {
+        $this->sidebar_index = $index;
+    }
+
+    /**
+     * @group sidebar
+     */
+    function test_dynamic_sidebar_using_sidebar_registered_with_no_id() {
+        global $wp_registered_sidebars;
+
+        $this->setExpectedIncorrectUsage( 'register_sidebar' );
+
+        // Register a couple of sidebars for fun.
+        register_sidebar();
+        register_sidebar();
+
+        $derived_sidebar_id = "sidebar-3"; // Number of sidebars in the global + 1.
+
+        add_action( 'dynamic_sidebar_before', array( $this, 'retrieve_sidebar_id' ) );
+
+        dynamic_sidebar( 3 );
+
+        $this->assertSame( $derived_sidebar_id, $this->sidebar_index );
+    }
+
+    /**
+     * @group sidebar
+     */
+    function test_dynamic_sidebar_numeric_id() {
+        $sidebar_id = 2;
+        register_sidebar( array( 'id' => $sidebar_id ) );
+
+        add_action( 'dynamic_sidebar_before', array( $this, 'retrieve_sidebar_id' ) );
+
+        dynamic_sidebar( $sidebar_id );
+
+        $this->assertSame( "sidebar-{$sidebar_id}", $this->sidebar_index );
+    }
+
+    /**
+     * @group sidebar
+     */
+    function test_dynamic_sidebar_string_id() {
+        $sidebar_id = 'wp-unit-tests';
+        register_sidebar( array( 'id' => $sidebar_id ) );
+
+        add_action( 'dynamic_sidebar_before', array( $this, 'retrieve_sidebar_id' ) );
+
+        dynamic_sidebar( $sidebar_id );
+
+        $this->assertSame( $sidebar_id, $this->sidebar_index );
     }