Changeset 899 in tests

Timestamp:

07/10/2012 09:23:19 PM (12 years ago)

Author:

nacin

Message:

Admin bar: When calling add_node() for an existing node, all properties should be merged into the existing arguments, including individual meta arguments. see #WP21117.

File:

• trunk/tests/test_includes_admin_bar.php (modified) (3 diffs)

trunk/tests/test_includes_admin_bar.php

@@ -7 +7 @@
  */
 class TestWPAdminBar extends WP_UnitTestCase {
+
+    static function setUpBeforeClass() {
+        WP_UnitTestCase::setUpBeforeClass();
+        require_once ABSPATH . WPINC . '/class-wp-admin-bar.php';
+    }
+
     function setUp() {
         parent::setUp();
@@ -24 +30 @@
         register_post_type( 'content', array( 'show_in_admin_bar' => true ) );
 
-        require_once ABSPATH . WPINC . '/class-wp-admin-bar.php';
         $admin_bar = new WP_Admin_Bar;
 
@@ -36 +41 @@
     }
 
+    /**
+     * @ticket 21117
+     */
+    function test_merging_existing_meta_values() {
+        $admin_bar = new WP_Admin_Bar;
+
+        $admin_bar->add_node( array(
+            'id' => 'test-node',
+            'meta' => array( 'class' => 'test-class' ),
+        ) );
+        $node = $admin_bar->get_node( 'test-node' );
+        $this->assertEquals( array( 'class' => 'test-class' ), $node->meta );
+
+        $admin_bar->add_node( array(
+            'id' => 'test-node',
+            'meta' => array( 'some-meta' => 'value' ),
+        ) );
+
+        $node = $admin_bar->get_node( 'test-node' );
+        $this->assertEquals( array( 'class' => 'test-class', 'some-meta' => 'value' ), $node->meta );
+    }
 }