Page MenuHomePhabricator

Flow: Fix phpunit failure in SpamBlacklistTest::testSpam
Closed, ResolvedPublic

Description

The assumptions of this test may need to be verified, fails with http://paste.debian.net/111811/

  1. Flow\Tests\SpamBlacklistTest::testSpam with data set #1 (Flow\Model\PostRevision, NULL, false)

Failed asserting that true matches expected false.

/srv/vagrant/mediawiki/extensions/Flow/tests/SpamBlacklistTest.php:61
/srv/vagrant/mediawiki/tests/phpunit/MediaWikiTestCase.php:133
/srv/vagrant/mediawiki/tests/phpunit/MediaWikiPHPUnitCommand.php:42
/srv/vagrant/mediawiki/tests/phpunit/phpunit.php:160


Version: unspecified
Severity: normal

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:39 AM
bzimport set Reference to bz68655.
bzimport added a subscriber: Unknown Object (MLST).

Seems to be a bad relationship between Flow and AbuseFilter?

EBernhardson triaged this task as Medium priority.
EBernhardson set Security to None.

Change 179035 had a related patch set uploaded (by EBernhardson):
Fix phpunit failure in SpamBlacklistTest::testSpam

https://gerrit.wikimedia.org/r/179035

Patch-For-Review

Change 179035 merged by jenkins-bot:
Fix phpunit failure in SpamBlacklistTest::testSpam

https://gerrit.wikimedia.org/r/179035