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

Timestamp:

11/03/2019 10:12:32 PM (7 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
          */