Changeset 46642 for trunk/tests/phpunit/tests/adminbar.php

Timestamp:

11/03/2019 10:12:32 PM (5 years ago)

Author:

whyisjake

Message:

Toolbar: Use add_node() instead of add_menu() in core.

This patch replaces all references to the add_menu() method with the add_node() one. (Also some code structure modifications for wp_admin_bar_appearance_menu().)

Fixes: #19647
Props: linuxologos, paulschreiber, morganestes, akibjorklund, nacin, whyisjake.

File:

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

trunk/tests/phpunit/tests/adminbar.php

@@ -347 +347 @@
 
     /**
+     * @ticket 19647
+     */
+    public function test_deprecated_menu_methods() {
+        $admin_bar = new WP_Admin_Bar();
+        $node_data = array( 'id' => 'test-node-19647' );
+
+        // Test add_menu() deprecation.
+        $this->setExpectedDeprecated( 'WP_Admin_Bar::add_menu' );
+        $admin_bar->add_menu( $node_data );
+        $node_test = $admin_bar->get_node( 'test-node-19647' );
+        $this->assertNotNull( $node_test );
+
+        // Test remove_menu() deprecation.
+        $this->setExpectedDeprecated( 'WP_Admin_Bar::remove_menu' );
+        $admin_bar->remove_menu( 'test-node-19647' );
+        $node_test = $admin_bar->get_node( 'test-node-19647' );
+        $this->assertNull( $node_test );
+    }
+
+    /**
      * @ticket 22247
      */