Changeset 40524

Timestamp:

04/22/2017 07:42:12 PM (8 years ago)

Author:

johnbillion

Message:

Build/Test Tools: Remove more skipped tests that should actually be failures if their conditions aren't satisfied.

See #40533

Location:

trunk/tests/phpunit/tests

Files:

• db.php (modified) (1 diff)
• http/base.php (modified) (1 diff)
• theme.php (modified) (1 diff)

trunk/tests/phpunit/tests/db.php

@@ -88 +88 @@
         $flag = setlocale( LC_ALL, 'ru_RU.utf8', 'rus', 'fr_FR.utf8', 'fr_FR', 'de_DE.utf8', 'de_DE', 'es_ES.utf8', 'es_ES' );
         if ( false === $flag )
-            $this->markTestSkipped( 'No European languages available for testing' );
+            $this->fail( 'No European languages available for testing' );
 
         // Try an update query

trunk/tests/phpunit/tests/http/base.php

@@ -385 +385 @@
     function test_ssl() {
         if ( ! wp_http_supports( array( 'ssl' ) ) )
-            $this->markTestSkipped( 'This install of PHP does not support SSL' );
+            $this->fail( 'This install of PHP does not support SSL' );
 
         $res = wp_remote_get( 'https://wordpress.org/' );

trunk/tests/phpunit/tests/theme.php

@@ -190 +190 @@
         $latest_default_theme = WP_Theme::get_core_default_theme();
         if ( ! $latest_default_theme->exists() || 'twenty' !== substr( $latest_default_theme->get_stylesheet(), 0, 6 ) ) {
-            $this->markTestSkipped( 'No Twenty* series default themes are installed' ); 
+            $this->fail( 'No Twenty* series default themes are installed' ); 
         }
         $this->assertContains( $latest_default_theme->get_stylesheet(), $this->default_themes );