Changeset 11055 for trunk/tests/phpunit/includes/testcase.php

Timestamp:

09/01/2016 03:44:48 AM (8 years ago)

Author:

boonebgorges

Message:

Tests: Set up page mappings in setUp() rather than setUpBeforeClass().

Since ​https://core.trac.wordpress.org/changeset/38398, wp_posts data
that has been committed will be deleted by WP_UnitTestCase after each
test. So we need to ensure that the data exists before each test.

Fixes #7241.

File:

• trunk/tests/phpunit/includes/testcase.php (modified) (2 diffs)

trunk/tests/phpunit/includes/testcase.php

@@ -27 +27 @@
 
     public static function setUpBeforeClass() {
+
+        // Fake WP mail globals, to avoid errors
+        add_filter( 'wp_mail', array( 'BP_UnitTestCase', 'setUp_wp_mail' ) );
+        add_filter( 'wp_mail_from', array( 'BP_UnitTestCase', 'tearDown_wp_mail' ) );
+    }
+
+    public function setUp() {
+        parent::setUp();
 
         /*
@@ -34 +42 @@
         bp_core_add_page_mappings( bp_get_option( 'bp-active-components' ), 'delete' );
 
-        // Fake WP mail globals, to avoid errors
-        add_filter( 'wp_mail', array( 'BP_UnitTestCase', 'setUp_wp_mail' ) );
-        add_filter( 'wp_mail_from', array( 'BP_UnitTestCase', 'tearDown_wp_mail' ) );
-    }
-
-    public function setUp() {
-        parent::setUp();
 
         $this->factory = new BP_UnitTest_Factory;