Changeset 5896 for trunk/tests/phpunit/testcases/replies/template/reply.php

Timestamp:

08/08/2015 07:13:21 AM (9 years ago)

Author:

netweb

Message:

Tests: If we add filters in tests, we need to also remove them for WordPress < 4.0 compatability

Props thebrandonallen. See #2845

File:

• trunk/tests/phpunit/testcases/replies/template/reply.php (modified) (4 diffs)

trunk/tests/phpunit/testcases/replies/template/reply.php

@@ -143 +143 @@
         $this->assertSame( $reply_url, bbp_get_reply_url( $r[6] ) );
 
+        // Remove the filter for WordPress < 4.0 compatibility.
+        remove_filter( 'bbp_show_lead_topic', '__return_true' );
+
         // Reply menu position is bumped by 1 when bbp_show_lead_topic() false.
         add_filter( 'bbp_show_lead_topic', '__return_false' );
@@ -173 +176 @@
         $reply_url = bbp_get_topic_permalink( $t ) . '&paged=3#post-' . bbp_get_reply_id( $r[6] );
         $this->assertSame( $reply_url, bbp_get_reply_url( $r[6] ) );
+
+        // Remove the filter for WordPress < 4.0 compatibility.
+        remove_filter( 'bbp_show_lead_topic', '__return_false' );
 
         // Restore the original reply pagination option value.
@@ -537 +543 @@
         $this->assertSame( 4, $position );
 
+        // Remove the filter for WordPress < 4.0 compatibility.
+        remove_filter( 'bbp_show_lead_topic', '__return_true' );
+
         // Reply menu position is bumped by 1 when bbp_show_lead_topic() false.
         add_filter( 'bbp_show_lead_topic', '__return_false' );
@@ -563 +572 @@
         $position = bbp_get_reply_position( $r[3] );
         $this->assertSame( 5, $position );
+
+        // Remove the filter for WordPress < 4.0 compatibility.
+        remove_filter( 'bbp_show_lead_topic', '__return_false' );
     }