diff --git a/tests/phpunit/tests/actions.php b/tests/phpunit/tests/actions.php index a511890bb7..ceacc72d7c 100644 --- a/tests/phpunit/tests/actions.php +++ b/tests/phpunit/tests/actions.php @@ -7,6 +7,9 @@ */ class Tests_Actions extends WP_UnitTestCase { + /** + * @covers ::do_action + */ function test_simple_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -24,6 +27,9 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( array( '' ), $args ); } + /** + * @covers ::remove_action + */ function test_remove_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -43,6 +49,9 @@ class Tests_Actions extends WP_UnitTestCase { } + /** + * @covers ::has_action + */ function test_has_action() { $tag = __FUNCTION__; $func = __FUNCTION__ . '_func'; @@ -57,7 +66,11 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertFalse( has_action( $tag ) ); } - // One tag with multiple actions. + /** + * One tag with multiple actions. + * + * @covers ::do_action + */ function test_multiple_actions() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -74,6 +87,11 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( 1, $a2->get_call_count() ); } + /** + * One tag with multiple actions. + * + * @covers ::do_action + */ function test_action_args_1() { $a = new MockAction(); $tag = __FUNCTION__; @@ -89,6 +107,11 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( array( $val ), array_pop( $argsvar ) ); } + /** + * One tag with multiple actions. + * + * @covers ::do_action + */ function test_action_args_2() { $a1 = new MockAction(); $a2 = new MockAction(); @@ -120,6 +143,8 @@ class Tests_Actions extends WP_UnitTestCase { * * @see https://core.trac.wordpress.org/ticket/17817#comment:72 * @ticket 17817 + * + * @covers ::do_action */ function test_action_args_3() { $a1 = new MockAction(); @@ -157,6 +182,8 @@ class Tests_Actions extends WP_UnitTestCase { * Tests PHP 4 notation for calling actions while passing in an object by reference. * * @ticket 48312 + * + * @covers ::do_action */ function test_action_args_with_php4_syntax() { $a = new MockAction(); @@ -201,6 +228,9 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( $expected, $a->get_events() ); } + /** + * @covers ::did_action + */ function test_did_action() { $tag1 = 'action1'; $tag2 = 'action2'; @@ -222,6 +252,9 @@ class Tests_Actions extends WP_UnitTestCase { } + /** + * @covers ::do_action + */ function test_all_action() { $a = new MockAction(); $tag1 = __FUNCTION__ . '_1'; @@ -246,6 +279,9 @@ class Tests_Actions extends WP_UnitTestCase { } + /** + * @covers ::remove_action + */ function test_remove_all_action() { $a = new MockAction(); $tag = __FUNCTION__; @@ -266,6 +302,9 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( array( $tag ), $a->get_tags() ); } + /** + * @covers ::do_action_ref_array + */ function test_action_ref_array() { $obj = new stdClass(); $a = new MockAction(); @@ -284,6 +323,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 11241 + * + * @covers ::do_action */ function test_action_keyed_array() { $a = new MockAction(); @@ -309,6 +350,9 @@ class Tests_Actions extends WP_UnitTestCase { } + /** + * @covers ::remove_action + */ function test_action_self_removal() { add_action( 'test_action_self_removal', array( $this, 'action_self_removal' ) ); do_action( 'test_action_self_removal' ); @@ -321,6 +365,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 17817 + * + * @covers ::do_action */ function test_action_recursion() { $tag = __FUNCTION__; @@ -336,6 +382,9 @@ class Tests_Actions extends WP_UnitTestCase { $this->assertSame( 2, $b->get_call_count(), 'recursive actions should call callbacks with later priority' ); } + /** + * @covers ::do_action + */ function action_that_causes_recursion( $tag ) { static $recursing = false; if ( ! $recursing ) { @@ -348,6 +397,9 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 9968 * @ticket 17817 + * + * @covers ::remove_action + * @covers ::add_action */ function test_action_callback_manipulation_while_running() { $tag = __FUNCTION__; @@ -383,6 +435,8 @@ class Tests_Actions extends WP_UnitTestCase { * * This specificaly addresses the concern raised at * https://core.trac.wordpress.org/ticket/17817#comment:52 + * + * @covers ::remove_filter */ function test_remove_anonymous_callback() { $tag = __FUNCTION__; @@ -416,6 +470,10 @@ class Tests_Actions extends WP_UnitTestCase { * Test the ArrayAccess methods of WP_Hook * * @ticket 17817 + * + * @covers WP_Hook::offsetGet + * @covers WP_Hook::offsetSet + * @covers WP_Hook::offsetUnset */ function test_array_access_of_wp_filter_global() { global $wp_filter; @@ -442,6 +500,8 @@ class Tests_Actions extends WP_UnitTestCase { * Make sure current_action() behaves as current_filter() * * @ticket 14994 + * + * @covers ::current_action */ function test_current_action() { global $wp_current_filter; @@ -453,6 +513,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_filter() { global $wp_current_filter; @@ -472,6 +534,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_action() { global $wp_current_filter; @@ -491,6 +555,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 14994 + * + * @covers ::doing_filter */ function test_doing_filter_real() { $this->assertFalse( doing_filter() ); // No filter is passed in, and no filter is being processed. @@ -541,6 +607,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 10441 * @expectedDeprecated tests_do_action_deprecated + * + * @covers ::do_action_deprecated */ public function test_do_action_deprecated() { $p = new WP_Post( (object) array( 'post_title' => 'Foo' ) ); @@ -559,6 +627,8 @@ class Tests_Actions extends WP_UnitTestCase { /** * @ticket 10441 * @expectedDeprecated tests_do_action_deprecated + * + * @covers ::do_action_deprecated */ public function test_do_action_deprecated_with_multiple_params() { $p1 = new WP_Post( (object) array( 'post_title' => 'Foo1' ) ); diff --git a/tests/phpunit/tests/actions/callbacks.php b/tests/phpunit/tests/actions/callbacks.php index fc58c33906..32d1c733d9 100644 --- a/tests/phpunit/tests/actions/callbacks.php +++ b/tests/phpunit/tests/actions/callbacks.php @@ -7,6 +7,8 @@ class Tests_Actions_Callbacks extends WP_UnitTestCase { /** * @ticket 23265 + * + * @covers ::add_action */ function test_callback_representations() { $tag = __FUNCTION__; diff --git a/tests/phpunit/tests/actions/closures.php b/tests/phpunit/tests/actions/closures.php index bd8b88d97f..833367714b 100644 --- a/tests/phpunit/tests/actions/closures.php +++ b/tests/phpunit/tests/actions/closures.php @@ -9,6 +9,10 @@ class Tests_Actions_Closures extends WP_UnitTestCase { /** * @ticket 10493 + * + * @covers ::add_action + * @covers ::has_action + * @covers ::do_action */ function test_action_closure() { $tag = 'test_action_closure';