WordPress.org

Make WordPress Core

Changeset 40524


Ignore:
Timestamp:
04/22/2017 07:42:12 PM (21 months 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:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/db.php

    r39626 r40524  
    8888        $flag = setlocale( LC_ALL, 'ru_RU.utf8', 'rus', 'fr_FR.utf8', 'fr_FR', 'de_DE.utf8', 'de_DE', 'es_ES.utf8', 'es_ES' );
    8989        if ( false === $flag )
    90             $this->markTestSkipped( 'No European languages available for testing' );
     90            $this->fail( 'No European languages available for testing' );
    9191
    9292        // Try an update query
  • trunk/tests/phpunit/tests/http/base.php

    r39174 r40524  
    385385    function test_ssl() {
    386386        if ( ! wp_http_supports( array( 'ssl' ) ) )
    387             $this->markTestSkipped( 'This install of PHP does not support SSL' );
     387            $this->fail( 'This install of PHP does not support SSL' );
    388388
    389389        $res = wp_remote_get( 'https://wordpress.org/' );
  • trunk/tests/phpunit/tests/theme.php

    r39919 r40524  
    190190        $latest_default_theme = WP_Theme::get_core_default_theme();
    191191        if ( ! $latest_default_theme->exists() || 'twenty' !== substr( $latest_default_theme->get_stylesheet(), 0, 6 ) ) {
    192             $this->markTestSkipped( 'No Twenty* series default themes are installed' );
     192            $this->fail( 'No Twenty* series default themes are installed' );
    193193        }
    194194        $this->assertContains( $latest_default_theme->get_stylesheet(), $this->default_themes );
Note: See TracChangeset for help on using the changeset viewer.