Changeset 6330 for trunk/tests/phpunit/testcases/topics/functions/topic.php

Timestamp:

02/26/2017 11:12:37 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Engagements: Updates existing functions & unit tests:

• Correct tests so that post_author of 0 does not get included in the overall count
• Add user IDs to all topics & replies where voice counts are being tested
• Update voice-count update function to use the new user-relationships API
• Clean-up topic merge code to more efficiently migrate favorites, subscriptions, and engagements

See #3068.

File:

• trunk/tests/phpunit/testcases/topics/functions/topic.php (modified) (4 diffs)

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

@@ -15 +15 @@
      */
     public function test_bbp_insert_topic() {
-
+        $u = $this->factory->user->create_many( 2 );
         $f = $this->factory->forum->create();
 
@@ -22 +22 @@
 
         $t = $this->factory->topic->create( array(
-            'post_title' => 'Topic 1',
+            'post_title'   => 'Topic 1',
             'post_content' => 'Content for Topic 1',
-            'post_parent' => $f,
-            'post_date' => $post_date,
-            'topic_meta' => array(
+            'post_parent'  => $f,
+            'post_date'    => $post_date,
+            'post_author'  => $u[0],
+            'topic_meta'   => array(
                 'forum_id' => $f,
             ),
@@ -33 +34 @@
         $r = $this->factory->reply->create( array(
             'post_parent' => $t,
-            'post_date' => $post_date,
-            'reply_meta' => array(
+            'post_date'   => $post_date,
+            'post_author'  => $u[1],
+            'reply_meta'  => array(
                 'forum_id' => $f,
                 'topic_id' => $t,
@@ -54 +56 @@
         // Topic meta.
         $this->assertSame( $f, bbp_get_topic_forum_id( $t ) );
-        $this->assertSame( 1, bbp_get_topic_reply_count( $t, true ) );
-        $this->assertSame( 0, bbp_get_topic_reply_count_hidden( $t, true ) );
-        $this->assertSame( 1, bbp_get_topic_voice_count( $t, true ) );
+        $this->assertSame( 1,  bbp_get_topic_reply_count( $t, true ) );
+        $this->assertSame( 0,  bbp_get_topic_reply_count_hidden( $t, true ) );
+        $this->assertSame( 2,  bbp_get_topic_voice_count( $t, true ) );
         $this->assertSame( $r, bbp_get_topic_last_reply_id( $t ) );
         $this->assertSame( $r, bbp_get_topic_last_active_id( $t ) );