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

Timestamp:

09/01/2016 03:44:48 AM (10 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;