Add missing @covers annotations

phpunit.xml.dist

@@ -3,6 +3,7 @@
 		backupGlobals="false"
 		colors="true"
 		beStrictAboutTestsThatDoNotTestAnything="true"
+		forceCoversAnnotation="true"
 		>
 	<testsuites>
 		<!-- Default test suite to run all tests. -->

tests/phpunit/multisite.xml

@@ -3,6 +3,7 @@
 		backupGlobals="false"
 		colors="true"
 		beStrictAboutTestsThatDoNotTestAnything="true"
+		forceCoversAnnotation="true"
 		>
 	<php>
 		<const name="WP_TESTS_MULTISITE" value="1" />

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 @@ function test_simple_action() {
 		$this->assertSame( array( '' ), $args );
 	}
 
+	/**
+	 * @covers ::remove_action
+	 */
 	function test_remove_action() {
 		$a   = new MockAction();
 		$tag = __FUNCTION__;
@@ -43,6 +49,9 @@ function test_remove_action() {
 
 	}
 
+	/**
+	 * @covers ::has_action
+	 */
 	function test_has_action() {
 		$tag  = __FUNCTION__;
 		$func = __FUNCTION__ . '_func';
@@ -57,7 +66,11 @@ function test_has_action() {
 		$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 @@ function test_multiple_actions() {
 		$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 @@ function test_action_args_1() {
 		$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 @@ function test_action_args_2() {
 	 *
 	 * @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 @@ function test_action_args_3() {
 	 * 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 @@ function test_action_priority() {
 		$this->assertSame( $expected, $a->get_events() );
 	}
 
+	/**
+	 * @covers ::did_action
+	 */
 	function test_did_action() {
 		$tag1 = 'action1';
 		$tag2 = 'action2';
@@ -222,6 +252,9 @@ function test_did_action() {
 
 	}
 
+	/**
+	 * @covers ::do_action
+	 */
 	function test_all_action() {
 		$a    = new MockAction();
 		$tag1 = __FUNCTION__ . '_1';
@@ -246,6 +279,9 @@ function test_all_action() {
 
 	}
 
+	/**
+	 * @covers ::remove_action
+	 */
 	function test_remove_all_action() {
 		$a   = new MockAction();
 		$tag = __FUNCTION__;
@@ -266,6 +302,9 @@ function test_remove_all_action() {
 		$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 @@ function test_action_ref_array() {
 
 	/**
 	 * @ticket 11241
+	 *
+	 * @covers ::do_action
 	 */
 	function test_action_keyed_array() {
 		$a = new MockAction();
@@ -309,6 +350,9 @@ function test_action_keyed_array() {
 
 	}
 
+	/**
+	 * @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 @@ function action_self_removal() {
 
 	/**
 	 * @ticket 17817
+	 *
+	 * @covers ::do_action
 	 */
 	function test_action_recursion() {
 		$tag = __FUNCTION__;
@@ -336,6 +382,9 @@ function test_action_recursion() {
 		$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 @@ function action_that_causes_recursion( $tag ) {
 	/**
 	 * @ticket 9968
 	 * @ticket 17817
+	 *
+	 * @covers ::remove_action
+	 * @covers ::add_action
 	 */
 	function test_action_callback_manipulation_while_running() {
 		$tag = __FUNCTION__;
@@ -383,6 +435,8 @@ function action_that_manipulates_a_running_hook( $tag, $mocks ) {
 	 *
 	 * 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 @@ function test_remove_anonymous_callback() {
 	 * 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 @@ function test_array_access_of_wp_filter_global() {
 	 * 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 @@ function test_current_action() {
 
 	/**
 	 * @ticket 14994
+	 *
+	 * @covers ::doing_filter
 	 */
 	function test_doing_filter() {
 		global $wp_current_filter;
@@ -472,6 +534,8 @@ function test_doing_filter() {
 
 	/**
 	 * @ticket 14994
+	 *
+	 * @covers ::doing_filter
 	 */
 	function test_doing_action() {
 		global $wp_current_filter;
@@ -491,6 +555,8 @@ function test_doing_action() {
 
 	/**
 	 * @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 @@ function apply_testing_nested_filter() {
 	/**
 	 * @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 @@ public static function deprecated_action_callback( $p ) {
 	/**
 	 * @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' ) );

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__;

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';

tests/phpunit/tests/admin/includesComment.php

@@ -47,6 +47,8 @@ public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
 
 	/**
 	 * Verify that both the comment date and author must match for a comment to exist.
+	 *
+	 * @covers ::comment_exists
 	 */
 	public function test_must_match_date_and_author() {
 		$this->assertNull( comment_exists( 1, '2004-01-02 12:00:00' ) );
@@ -55,27 +57,35 @@ public function test_must_match_date_and_author() {
 
 	/**
 	 * @ticket 33871
+	 *
+	 * @covers ::comment_exists
 	 */
 	public function test_default_value_of_timezone_should_be_blog() {
 		$this->assertEquals( self::$post_id, comment_exists( 1, '2014-05-06 12:00:00' ) );
 	}
 
 	/**
 	 * @ticket 33871
+	 *
+	 * @covers ::comment_exists
 	 */
 	public function test_should_respect_timezone_blog() {
 		$this->assertEquals( self::$post_id, comment_exists( 1, '2014-05-06 12:00:00', 'blog' ) );
 	}
 
 	/**
 	 * @ticket 33871
+	 *
+	 * @covers ::comment_exists
 	 */
 	public function test_should_respect_timezone_gmt() {
 		$this->assertEquals( self::$post_id, comment_exists( 1, '2014-05-06 07:00:00', 'gmt' ) );
 	}
 
 	/**
 	 * @ticket 33871
+	 *
+	 * @covers ::comment_exists
 	 */
 	public function test_invalid_timezone_should_fall_back_on_blog() {
 		$this->assertEquals( self::$post_id, comment_exists( 1, '2014-05-06 12:00:00', 'not_a_valid_value' ) );