Changeset 5880 for trunk/tests/phpunit/testcases/topics/template/links.php

Timestamp:

07/24/2015 11:21:28 AM (10 years ago)

Author:

netweb

Message:

Replies: In bbp_update_reply_walker() always update the topics last active time with bbp_update_topic_last_active_time() regardless of reply's published status.

This changeset including unit tests ensures that when calling reply actions spam/unspam, trash/untrash, approve/unapprove the reply's parent topic's last active time is updated to the time of the last published reply.

Props netweb. See #2838

File:

• trunk/tests/phpunit/testcases/topics/template/links.php (modified) (3 diffs)

trunk/tests/phpunit/testcases/topics/template/links.php

@@ -59 +59 @@
 
         $link = bbp_get_topic_freshness_link( $t );
-        $this->assertSame( '<a href="http://example.org/?topic=topic-1" title="">4 days, 4 hours ago</a>', $link );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1" title="">4 days, 4 hours ago</a>', $link );
 
         $r1 = $this->factory->reply->create( array(
@@ -71 +71 @@
 
         $link = bbp_get_topic_freshness_link( $t );
-        $this->assertSame( '<a href="http://example.org/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">3 days, 8 hours ago</a>', $link );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">3 days, 8 hours ago</a>', $link );
 
         $r2 = $this->factory->reply->create( array(
@@ -83 +83 @@
 
         $link = bbp_get_topic_freshness_link( $t );
-        $this->assertSame( '<a href="http://example.org/?topic=topic-1/#post-' . bbp_get_reply_id( $r2 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">2 days, 12 hours ago</a>', $link );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r2 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">2 days, 12 hours ago</a>', $link );
+
+        // Retore the user
+        $this->set_current_user( $this->old_current_user );
+    }
+
+    /**
+     * @covers ::bbp_get_topic_freshness_link
+     */
+    public function test_bbp_get_topic_freshness_link_with_unpublished_replies() {
+
+        if ( is_multisite() ) {
+            $this->markTestSkipped( 'Skipping URL tests in multiste for now.' );
+        }
+
+        $now = time();
+        $post_date    = date( 'Y-m-d H:i:s', $now - 60 * 60 * 20 ); // 2o hours ago
+        $post_date_r1 = date( 'Y-m-d H:i:s', $now - 60 * 60 * 18 ); // 18 hours ago
+        $post_date_r2 = date( 'Y-m-d H:i:s', $now - 60 * 60 * 16 ); // 16 hours ago
+        $post_date_r3 = date( 'Y-m-d H:i:s', $now - 60 * 60 * 14 ); // 14 hours ago
+        $post_date_r4 = date( 'Y-m-d H:i:s', $now - 60 * 60 * 12 ); // 12 hours ago
+        $post_date_r5 = date( 'Y-m-d H:i:s', $now - 60 * 60 * 10 ); // 1o hours ago
+
+        $f = $this->factory->forum->create();
+        $t = $this->factory->topic->create( array(
+            'post_parent' => $f,
+            'post_date' => $post_date,
+            'topic_meta' => array(
+                'forum_id' => $f,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1" title="">20 hours ago</a>', $link );
+
+        $r1 = $this->factory->reply->create( array(
+            'post_parent' => $t,
+            'post_date' => $post_date_r1,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">18 hours ago</a>', $link );
+
+        $r2 = $this->factory->reply->create( array(
+            'post_parent' => $t,
+            'post_date' => $post_date_r2,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r2 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">16 hours ago</a>', $link );
+
+        bbp_spam_reply( $r2 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">18 hours ago</a>', $link );
+
+        $r3 = $this->factory->reply->create( array(
+            'post_parent' => $t,
+            'post_date' => $post_date_r3,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://example.org/?topic=topic-1/#post-' . bbp_get_reply_id( $r3 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">14 hours ago</a>', $link );
+
+        // Todo: Use bbp_trash_reply() and not wp_trash_post()
+        wp_trash_post( $r3 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">18 hours ago</a>', $link );
+
+        $r4 = $this->factory->reply->create( array(
+            'post_parent' => $t,
+            'post_date' => $post_date_r4,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r4 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">12 hours ago</a>', $link );
+
+        bbp_unapprove_reply( $r4 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r1 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">18 hours ago</a>', $link );
+
+        bbp_unspam_reply( $r2 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r2 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">16 hours ago</a>', $link );
+
+        // Todo: Use bbp_untrash_reply() and not wp_untrash_post()
+        wp_untrash_post( $r3 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r3 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">14 hours ago</a>', $link );
+
+        bbp_approve_reply( $r4 );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r4 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">12 hours ago</a>', $link );
+
+        $r5 = $this->factory->reply->create( array(
+            'post_parent' => $t,
+            'post_date' => $post_date_r5,
+            'reply_meta' => array(
+                'forum_id' => $f,
+                'topic_id' => $t,
+            ),
+        ) );
+
+        $link = bbp_get_topic_freshness_link( $t );
+        $this->assertSame( '<a href="http://' . WP_TESTS_DOMAIN . '/?topic=topic-1/#post-' . bbp_get_reply_id( $r5 ) . '" title="Reply To: ' . bbp_get_topic_title( $t ) . '">10 hours ago</a>', $link );
     }