Changeset 11052 for trunk/tests/phpunit/testcases/core/caps.php

Timestamp:

09/01/2016 12:50:06 AM (9 years ago)

Author:

dcavins

Message:

Improve bp_current_user_can().

• Deprecate $args['blog_id’] in favor of $args['site_id’].
• Improve the structure of the passed $args array in the

bp_current_user_can filter.

Props r-a-y, dcavins.

Fixes #7192.

File:

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

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

@@ -47 +47 @@
     }
 
-    /**
-     * @group bp_xprofile_change_field_visibility
-     */
-    public function test_bp_current_user_can_should_pass_null_in_args_parameter_if_empty() {
-        $u = $this->factory->user->create();
-        $this->set_current_user( $u );
-
-        /**
-         * Fake bp_get_the_profile_field_id() to pretend we're in the field loop and
-         * to avoid notices when checking 'bp_xprofile_change_field_visibility' cap
-         */
-        $GLOBALS['field'] = new stdClass;
-        $GLOBALS['field']->id = 1;
-
-        // Capture the cap's $args
-        add_filter( 'bp_xprofile_map_meta_caps', array( $this, 'check_cap_args' ), 10, 4 );
-
-        // Use a cap check that depends on a null value for a cap's args
-        bp_current_user_can( 'bp_xprofile_change_field_visibility' );
-
-        // Assert!
-        $this->assertEquals( null, $this->test_args[0] );
-
-        // Reset
-        remove_filter( 'bp_xprofile_map_meta_caps', array( $this, 'check_cap_args' ), 10, 4 );
-        unset( $GLOBALS['field'], $this->test_args );
-    }
-
     public function grant_cap_foo( $allcaps, $caps ) {
         if ( bp_is_root_blog() ) {