Changeset 26370

Timestamp:

11/25/2013 10:49:51 PM (11 years ago)

Author:

wonderboymusic

Message:

When setting WP_TESTS_FORCE_KNOWN_BUGS to true, it is preferable that some forced tests are still skipped when they call classes or functions that do not exist, producing fatal errors.

Fixes #26248.

Location:

trunk/tests/phpunit/tests

Files:

• dependencies/scripts.php (modified) (2 diffs)
• export/class-wp-export-query.php (modified) (1 diff)
• export/functions.export.php (modified) (1 diff)
• export/writers.php (modified) (1 diff)
• formatting/MapDeep.php (modified) (1 diff)
• iterators.php (modified) (1 diff)

trunk/tests/phpunit/tests/dependencies/scripts.php

@@ -98 +98 @@
      */
     function test_json_encode_should_not_encode_special_literal_values() {
+        if ( ! class_exists( 'WP_JS_Literal' ) ) {
+            $this->markTestSkipped( "WP_JS_Literal class doesn't exist" );
+        }
+
         $literal = new WP_JS_Literal( 'baba()' );
         $this->assertEquals( '{"x":baba()}', WP_JS_Literal::json_encode( array( 'x' => $literal ), array( $literal ) ) );
@@ -106 +110 @@
      */
     function test_json_encode_should_not_encode_special_literal_values_with_dependencies() {
+        if ( ! class_exists( 'WP_JS_Literal' ) ) {
+            $this->markTestSkipped( "WP_JS_Literal class doesn't exist" );
+        }
+        
         $literal = new WP_JS_Literal( 'baba()', array( 'dep0', 'dep1' ) );
         $this->assertEquals( '{"x":baba()}', WP_JS_Literal::json_encode( array( 'x' => $literal ), array( $literal ) ) );

trunk/tests/phpunit/tests/export/class-wp-export-query.php

@@ -8 +8 @@
  */
 class Test_WP_Export_Query extends WP_UnitTestCase {
+    function setUp() {
+        if ( ! class_exists( 'WP_Export_Query' ) ) {
+            $this->markTestSkipped( "WP_Export_Query class doesn't exist" );
+        }
+
+        parent::setUp();
+    }
+
     function test_WP_Export_Query_should_be_initialized_with_an_array() {
         $export = new WP_Export_Query( array( 'author' => 'all' ) );

trunk/tests/phpunit/tests/export/functions.export.php

@@ -8 +8 @@
  */
 class Test_WP_Export_Functions extends WP_UnitTestCase {
+    function setUp() {
+        if ( ! function_exists( 'wp_export' ) ) {
+            $this->markTestSkipped( "wp_export function doesn't exist" );
+        }
+
+        parent::setUp();
+    }
+    
     function test_wp_export_returns_wp_error_if_the_writer_throws_Export_exception() {
         $this->assertTrue( is_wp_error( wp_export( array( 'writer' => 'Test_WP_Export_Stub_Writer_Throws_Export_Exception' ) ) ) );

trunk/tests/phpunit/tests/export/writers.php

@@ -9 +9 @@
 class Test_WP_Export_Writers extends WP_UnitTestCase {
     function test_export_returner_returns_all_the_return_values() {
+        if ( ! class_exists( 'WP_Export_Returner' ) ) {
+            $this->markTestSkipped( "WP_Export_Returner class doesn't exist" );
+        }
         $returner = new WP_Export_Returner( $this->get_x_formatter() );
         $this->assertEquals( 'xxx' , $returner->export() );

trunk/tests/phpunit/tests/formatting/MapDeep.php

@@ -5 +5 @@
  */
 class Tests_Formatting_MapDeep extends WP_UnitTestCase {
+    function setUp() {
+        if ( ! function_exists( 'map_deep' ) ) {
+            $this->markTestSkipped( "map_deep function doesn't exist" );
+        }
+
+        parent::setUp();
+    }
+
     function test_map_deep_with_any_function_over_empty_array_should_return_empty_array() {
         $this->assertEquals( array(), map_deep( array( $this, 'return_baba' ), array() ) );

trunk/tests/phpunit/tests/iterators.php

@@ -5 +5 @@
  */
 class Test_WP_Post_IDs_Iterator extends WP_UnitTestCase {
+    function setUp() {
+        if ( ! class_exists( 'WP_Post_IDs_Iterator' ) ) {
+            $this->markTestSkipped( "WP_Post_IDs_Iterator class doesn't exist" );
+        }
+
+        parent::setUp();
+    }
+
     function test_create() {
         new WP_Post_IDs_Iterator( array( 1, 2, 3 ) );