PageTriage ArticleMetadataTest::testValidatePageIds failing
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Reedy
	Apr 20 2020, 2:26 AM

Description

https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php72-docker/62473/console

03:21:22 Failed asserting that two arrays are equal.
03:21:22 --- Expected
03:21:22 +++ Actual
03:21:22 @@ @@
03:21:22  Array (
03:21:22 -    0 => 2
03:21:22 -    1 => 3
03:21:22 -    2 => 4
03:21:22 -    3 => 5
03:21:22 -    4 => 6
03:21:22 -    5 => 7
03:21:22  )
03:21:22 
03:21:22 /workspace/src/tests/phpunit/MediaWikiTestCaseTrait.php:122
03:21:22 /workspace/src/extensions/PageTriage/tests/phpunit/ArticleMetadataTest.php:91
03:21:22 /workspace/src/tests/phpunit/MediaWikiIntegrationTestCase.php:424
03:21:22 /workspace/src/maintenance/doMaintenance.php:105

The immediate cause is that the test assumes that the starting $this->pageIds are all valid, which is not the case

Details

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/PageTriage	master	+2 -0	ArticleMetadataTest: Clear static cache during setup

Customize query in gerrit

Event Timeline

Reedy created this task.Apr 20 2020, 2:26 AM

Restricted Application added a project: Growth-Team. · View Herald TranscriptApr 20 2020, 2:26 AM

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

Reedy added a project: ci-test-error (WMF-deployed Build Failure).Apr 20 2020, 2:29 AM

See https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageTriage/+/590816/ for a demonstration of the immediate cause

This is preventing merges or deploys for a production extension. (results)

Screenshot 2020-04-30 at 22.31.45.png (782×1 px, 269 KB)

This is happening because validatePageIds caches its results, but it's also called from the LinksUpdateComplete hook (and the ORESCheckModels hook, too) during page creation. The test suite's setup process creates a bunch of pages, then adds them to the PageTriage queue. But because validatePageIds is called during page creation, when the page isn't yet in the queue, the fact that it's not in the queue is cached, and validatePageIds rejects basically all of its input (except for page ID 1, for some reason).

Change 593644 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/PageTriage@master] ArticleMetadataTest: Clear static cache during setup

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

gerritbot added a project: Patch-For-Review.May 1 2020, 12:37 AM

Change 593644 merged by Jforrester:
[mediawiki/extensions/PageTriage@master] ArticleMetadataTest: Clear static cache during setup

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

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.31; 2020-05-05).May 1 2020, 1:00 AM

Thank you!

Maintenance_bot removed a project: Patch-For-Review.May 1 2020, 1:10 AM

	F31789069: Screenshot 2020-04-30 at 22.31.45.png
	Apr 30 2020, 9:32 PM

PageTriage ArticleMetadataTest::testValidatePageIds failingClosed, ResolvedPublicActions

Description

Details

Event Timeline

PageTriage ArticleMetadataTest::testValidatePageIds failing
Closed, ResolvedPublic
Actions