Changeset 5780

Timestamp:

05/22/2015 10:37:26 AM (10 years ago)

Author:

netweb

Message:

Tests: Updated topic component count functions tests
Props thebrandonallen, netweb. See #2801

File:

• trunk/tests/phpunit/testcases/topics/functions/counts.php (modified) (6 diffs)

trunk/tests/phpunit/testcases/topics/functions/counts.php

@@ -61 +61 @@
         ) );
 
+        // Start with zero
         $count = bbp_get_topic_reply_count( $t );
         $this->assertSame( '0', $count );
@@ -73 +74 @@
         ) );
 
-        // @todo Investigate caching issues in bbp_get_public_child_count()
-        wp_cache_flush();
+        $count = bbp_get_topic_reply_count( $t );
+        $this->assertSame( '3', $count );
 
         bbp_update_topic_reply_count( $t );
-
-        // @todo Investigate caching issues in bbp_get_public_child_count()
-        wp_cache_flush();
 
         $count = bbp_get_topic_reply_count( $t );
@@ -95 +93 @@
         // Test update using reply id
         bbp_update_topic_reply_count( $r2 );
-
         $count = bbp_get_topic_reply_count( $t );
         $this->assertSame( '4', $count );
+
+        // Spam a reply
+        bbp_spam_reply( $r2 );
+
+        bbp_update_topic_reply_count( $t );
+        $count = bbp_get_topic_reply_count( $t );
+        $this->assertSame( '3', $count );
+
+        // Set the reply count manually
+        bbp_update_topic_reply_count( $t, 7 );
+        $count = bbp_get_topic_reply_count( $t );
+        $this->assertSame( '7', $count );
     }
 
@@ -104 +113 @@
      */
     public function test_bbp_update_topic_reply_count_hidden() {
-        $t = $this->factory->topic->create();
+        // Create a forum
+        $f = $this->factory->forum->create();
 
+        // Create a topic
+        $t = $this->factory->topic->create( array(
+            'post_parent' => $f,
+            'topic_meta' => array(
+                'forum_id' => $f,
+            )
+        ) );
+
+        // Start with zero
         $count = bbp_get_topic_reply_count_hidden( $t );
         $this->assertSame( '0', $count );
 
-        $r = $this->factory->reply->create_many( 15, array(
+        $r = $this->factory->reply->create_many( 3, array(
             'post_parent' => $t,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            )
         ) );
 
         bbp_update_topic_reply_count_hidden( $t );
-
         $count = bbp_get_topic_reply_count_hidden( $t );
         $this->assertSame( '0', $count );
 
-        bbp_spam_reply( $r[11] );
+        bbp_spam_reply( $r[2] );
 
         bbp_update_topic_reply_count_hidden( $t );
-
         $count = bbp_get_topic_reply_count_hidden( $t );
         $this->assertSame( '1', $count );
 
-        bbp_unapprove_reply( $r[7] );
+        bbp_unapprove_reply( $r[0] );
 
         bbp_update_topic_reply_count_hidden( $t );
-
         $count = bbp_get_topic_reply_count_hidden( $t );
         $this->assertSame( '2', $count );
@@ -149 +169 @@
 
         bbp_update_topic_voice_count( $t );
-
         $count = bbp_get_topic_voice_count( $t );
         $this->assertSame( '2', $count );
+
+        $count = bbp_update_topic_voice_count( $t );
+        $this->assertSame( 2, $count );
 
         $r = $this->factory->reply->create( array(
@@ -159 +181 @@
 
         bbp_update_topic_voice_count( $t );
-
         $count = bbp_get_topic_voice_count( $t );
         $this->assertSame( '3', $count );