Tests: Explicitly mark empty REST API tests as not performing any assertions.

WordPress core test suite uses PHPUnit's `beStrictAboutTestsThatDoNotTestAnything` option set to true, which marks a test as risky when no assertions are performed.

REST API test classes have some empty tests for non-implemented methods because these test classes extend the abstract `WP_Test_REST_Controller_Testcase` class, which requires several methods to be implemented that don't necessarily make sense for all REST API routes.

As these tests are intentionally empty, they were previously marked as skipped, so that they are not reported as risky.

This commit aims to further reduce noise in the test suite and effectively ignores these empty tests altogether, which seems like a more appropriate option at this time.

The `@doesNotPerformAssertions` annotation can be reconsidered in the future when the tests are either removed as unnecessary or updated to actually perform assertions related to their behavior.

Follow-up to [40534], [41176], [41228], [53921].

See #40538, #41463, #55652.

git-svn-id: https://develop.svn.wordpress.org/trunk@54058 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov
2022-09-01 22:54:19 +00:00
parent c4594ec05b
commit 32ead26653
18 changed files with 265 additions and 87 deletions

View File

@@ -99,58 +99,58 @@ class Tests_REST_WpRestEditSiteExportController extends WP_Test_REST_Controller_
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_context_param() {
$this->markTestSkipped( 'Controller does not use get_context_param().' );
// Controller does not use get_context_param().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_get_item() {
$this->markTestSkipped( 'Controller does not implement get_item().' );
// Controller does not implement get_item().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_get_items() {
$this->markTestSkipped( 'Controller does not implement get_items().' );
// Controller does not implement get_items().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_create_item() {
$this->markTestSkipped( 'Controller does not implement create_item().' );
// Controller does not implement create_item().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_update_item() {
$this->markTestSkipped( 'Controller does not implement update_item().' );
// Controller does not implement update_item().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_delete_item() {
$this->markTestSkipped( 'Controller does not implement delete_item().' );
// Controller does not implement delete_item().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_prepare_item() {
$this->markTestSkipped( 'Controller does not implement prepare_item().' );
// Controller does not implement prepare_item().
}
/**
* @ticket 54448
* @doesNotPerformAssertions
*/
public function test_get_item_schema() {
$this->markTestSkipped( 'Controller does not implement get_item_schema().' );
// Controller does not implement get_item_schema().
}
}