#40584 closed defect (bug) (fixed)
Missing `comment_whitelist` reset in `Tests_Comment_CheckComment::test_should_return_true_when_content_does_not_match_moderation_keys`
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | has-patch |
Focuses: | Cc: |
Description
I came to @boone with a question about this test because I could not wrap my head around why it would be asserting False when the name indicates it should assert true.
He investigated a bit, and found that the test is missing a update_option( 'comment_whitelist', 0 );
, as done in the previous test.
Creating a ticket to implement his findings.
Original Slack discussion at https://wordpress.slack.com/archives/C02RQBWTW/p1493306172794672
Attachments (1)
Change History (4)
Note: See
TracTickets for help on using
tickets.
Adding required
update_option( 'comment_whitelist', 0 );
, and switching to rightful$this->assertTrue( $results );