Changeset 9180 for trunk/tests/phpunit/testcases/core/functions.php

Timestamp:

11/25/2014 06:52:58 PM (10 years ago)

Author:

boonebgorges

Message:

Don't remove Register and Activate pages from bp_core_get_directory_page_ids().

These two pages do not correspond to components, so they should be excluded
from bp_is_active() checks.

Props imath.
Fixes #6043.

File:

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

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

@@ -369 +369 @@
 
     /**
+     * @group bp_core_get_directory_page_ids
+     */
+    public function test_bp_core_get_directory_page_ids_should_contain_register_and_activet_pages_when_registration_is_open() {
+        add_filter( 'bp_get_signup_allowed', '__return_true', 999 );
+
+        $ac = buddypress()->active_components;
+        bp_core_add_page_mappings( array_keys( $ac ) );
+
+        $page_ids = bp_core_get_directory_page_ids();
+        $page_names = array_keys( $page_ids );
+
+        $this->assertContains( 'register', $page_names );
+        $this->assertContains( 'activate', $page_names );
+
+        remove_filter( 'bp_get_signup_allowed', '__return_true', 999 );
+    }
+
+    /**
+     * @group bp_core_get_directory_page_ids
+     */
+    public function test_bp_core_get_directory_page_ids_should_not_contain_register_and_activet_pages_when_registration_is_closed() {
+
+        // Make sure the pages exist, to verify they're filtered out.
+        add_filter( 'bp_get_signup_allowed', '__return_true', 999 );
+        $ac = buddypress()->active_components;
+        bp_core_add_page_mappings( array_keys( $ac ) );
+        remove_filter( 'bp_get_signup_allowed', '__return_true', 999 );
+
+        add_filter( 'bp_get_signup_allowed', '__return_false', 999 );
+        $page_ids = bp_core_get_directory_page_ids();
+        remove_filter( 'bp_get_signup_allowed', '__return_false', 999 );
+
+        $page_names = array_keys( $page_ids );
+
+        $this->assertNotContains( 'register', $page_names );
+        $this->assertNotContains( 'activate', $page_names );
+    }
+
+    /**
      * @group bp_core_get_directory_pages
      */