Changeset 51415 for trunk/tests/phpunit/tests/cache.php

Timestamp:

07/13/2021 10:43:42 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Tests: Clean up skipping conditions and requirements for various tests.

This improves the consistency of test skipping and ensures that:

• The @requires annotations use the right condition and format, and are on the right level (class vs. function).
• Inline conditions with a markTestSkipped() call are only used when annotations cannot be used.
• All markTestSkipped() calls contain a verbose explanation of why the test is being skipped.

Props jrf, hellofromTonya.
Fixes #53009.

File:

• trunk/tests/phpunit/tests/cache.php (modified) (1 diff)

trunk/tests/phpunit/tests/cache.php

@@ -252 +252 @@
     function test_switch_to_blog() {
         if ( ! method_exists( $this->cache, 'switch_to_blog' ) ) {
-            return;
+            $this->markTestSkipped( 'This test requires a switch_to_blog() method on the cache object.' );
         }