Changeset 34250 for trunk/tests/phpunit/tests/comment.php

Timestamp:

09/16/2015 09:59:16 PM (9 years ago)

Author:

boonebgorges

Message:

Don't notify post authors about spam comments.

[34106] moved post author notification to a hook, and in the process, missed
the 'spam' check. This changeset restores that check.

To make unit testing easier, the notification callbacks have been refactored
to return values: false when various conditions aren't met (eg, approved
comments should not trigger moderation emails), and the return value of the
wp_notify_*() function otherwise.

Props cfinke, kraftbj.
See #33587.

File:

• trunk/tests/phpunit/tests/comment.php (modified) (1 diff)

trunk/tests/phpunit/tests/comment.php

@@ -260 +260 @@
         $this->assertTrue( wp_notify_moderator( $c ) );
     }
+
+    /**
+     * @ticket 33587
+     */
+    public function test_wp_new_comment_notify_postauthor_should_not_send_email_when_comment_has_been_marked_as_spam() {
+        $p = $this->factory->post->create();
+        $c = $this->factory->comment->create( array(
+            'comment_post_ID' => $p,
+            'comment_approved' => 'spam',
+        ) );
+
+        $sent = wp_new_comment_notify_postauthor( $c );
+        $this->assertFalse( $sent );
+    }
 }