Changeset 9700

Timestamp:

04/06/2015 03:16:07 PM (10 years ago)

Author:

boonebgorges

Message:

Add some tests for trailingslashit() behavior in bp_core_new_subnav_item().

See #6353.

File:

• trunk/tests/phpunit/testcases/core/nav.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/nav.php

@@ -342 +342 @@
         $this->set_current_user( $old_current_user );
     }
+
+    /**
+     * @group bp_core_new_subnav_item()
+     */
+    public function test_bp_core_new_subnav_item_should_trailingslash_link_when_link_is_autogenerated_using_slug() {
+        bp_core_new_subnav_item( array(
+            'name' => 'bar',
+            'slug' => 'bar',
+            'parent_slug' => 'foo',
+            'parent_url' => bp_get_root_domain() . 'foo/',
+            'screen_function' => 'foo',
+        ) );
+
+        $expected = bp_get_root_domain() . 'foo/bar/';
+        $this->assertSame( $expected, buddypress()->bp_options_nav['foo']['bar']['link'] );
+    }
+
+    /**
+     * @group bp_core_new_subnav_item()
+     */
+    public function test_bp_core_new_subnav_item_should_trailingslash_link_when_link_is_autogenerated_not_using_slug() {
+        bp_core_new_nav_item( array(
+            'name' => 'foo',
+            'slug' => 'foo-parent',
+            'link' => bp_get_root_domain() . 'foo-parent/',
+            'default_subnav_slug' => 'bar',
+            'screen_function' => 'foo',
+        ) );
+
+        bp_core_new_subnav_item( array(
+            'name' => 'bar',
+            'slug' => 'bar',
+            'parent_slug' => 'foo-parent',
+            'parent_url' => bp_get_root_domain() . 'foo-parent/',
+            'screen_function' => 'bar',
+        ) );
+
+        $expected = bp_get_root_domain() . 'foo-parent/';
+        $this->assertSame( $expected, buddypress()->bp_options_nav['foo-parent']['bar']['link'] );
+    }
 }