Page MenuHomePhabricator

EmailAuthHooksTest tests fail in CheckUser repo
Closed, ResolvedPublic

Description

The WikimediaEvents\Tests\Unit\EmailAuthHooksTest::testShouldRequireVerificationForNon2FAUsersOnUnknownIPsKnownToIpoid test is failing in the CheckUser gate-and-submit CI pipeline:

15:42:36 1) WikimediaEvents\Tests\Unit\EmailAuthHooksTest::testShouldRequireVerificationForNon2FAUsersOnUnknownIPsKnownToIpoid with data set "2FA disabled, email not confirmed, known to ipoid, not bot user, $wgWikimediaEventsEmailAuthEnforce: false, LoginNotify status: LoginNotify::USER_NOT_KNOWN, last activity unknown" (false, false, true, false, false, 'LoginNotify::USER_NOT_KNOWN', null)
15:42:36 PHPUnit\Framework\MockObject\UnknownTypeException: Class or interface "MediaWiki\Extension\IPReputation\IPoidResponse" does not exist
15:42:36 
15:42:36 /workspace/src/extensions/WikimediaEvents/tests/phpunit/EmailAuthHooksTest.php:202

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1228546 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/WikimediaEvents@master] EmailAuthHooksTest: Fix namespace of IPoidResponse

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

Change #1228551 had a related patch set uploaded (by Aarohisharma3; author: Aarohisharma3):

[mediawiki/extensions/WikimediaEvents@master] tests: Fix IPoidResponse namespace import in EmailAuthHooksTest

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

Change #1228546 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] EmailAuthHooksTest: Fix namespace of IPoidResponse

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

Change #1228551 abandoned by Dreamy Jazz:

[mediawiki/extensions/WikimediaEvents@master] tests: Fix IPoidResponse namespace import in EmailAuthHooksTest

Reason:

Hi. This patch already exists at Ib94eb7474ed3cacdfe8acb046ad8a310ef7d9574 which is already being merged, thanks for this though

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