ArticleRevisionVisibilitySetHookHandlerTest fails on CI
Closed, ResolvedPublicBUG REPORT
Actions

Description

Steps to Reproduce:

upload a new patch

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/618703
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/578298

shows the same errors.

Actual Results:

11:10:21 There were 6 failures:
11:10:21 
11:10:21 1) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "One job with one revision identifier dispatched" (array(array('[{"arrayFormatVersion":1,"ent...":24}]', 1)), array(12), array(array(1, 0)))
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21 
11:10:21 2) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "One job with two revision identifiers dispatched" (array(array('[{"arrayFormatVersion":1,"ent...":25}]', 1)), array(12, 13), array(array(1, 0), array(1, 474)))
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21 
11:10:21 3) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "Two jobs dispatched (different bit flags)" (array(array('[{"arrayFormatVersion":1,"ent...":24}]', 1), array('[{"arrayFormatVersion":1,"ent...":25}]', 6)), array(12, 13), array(array(1, 0), array(6, 2)))
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21 
11:10:21 4) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "Two jobs dispatched (small batch size)" (array(array('[{"arrayFormatVersion":1,"ent...":25}]', 1), array('[{"arrayFormatVersion":1,"ent...":26}]', 1)), array(12, 13, 14), array(array(1, 0), array(1, 2), array(1, 99)), true, 2)
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21 
11:10:21 5) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "Multiple jobs with multiple revisions dispatched" (array(array('[{"arrayFormatVersion":1,"ent...":26}]', 1), array('[{"arrayFormatVersion":1,"ent...":25}]', 6)), array(12, 13, 14), array(array(1, 0), array(6, 2), array(1, 99)))
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21 
11:10:21 6) Wikibase\Repo\Tests\Hooks\ArticleRevisionVisibilitySetHookHandlerTest::testHandle with data set "One job dispatched, some revisions discarded" (array(array('[{"arrayFormatVersion":1,"ent...":24}]', 1)), array(404, 12, 408), array(array(1, 0), array(1, 0), array(1, 2)))
11:10:21 Failed asserting that 0 is identical to 2.
11:10:21 
11:10:21 /workspace/src/extensions/Wikibase/repo/tests/phpunit/includes/Hooks/ArticleRevisionVisibilitySetHookHandlerTest.php:220
11:10:21 /workspace/src/maintenance/doMaintenance.php:107
11:10:21

Expected Results:

CI should pass

Details

Project	Branch	Lines +/-	Subject
mediawiki/core	master	+2 -0	Clean up test static fake time to fix Wikibase tests
mediawiki/extensions/Wikibase	master	+0 -2	Reenable ArticleRevisionVisibilitySetHookHandlerTest
mediawiki/core	master	+1 -2	Remove stray fake time side effect in data provider
mediawiki/core	master	+5 -1	UserDefTest: prevent (time) sideeffect from dataProvider
mediawiki/extensions/Wikibase	master	+7 -1	Test if this fixes CI
mediawiki/core	master	+2 -0	Clean up test fake time singleton to fix Wikibase tests
mediawiki/extensions/Wikibase	master	+2 -0	temporarily disable ArticleRevisionVisibilitySetHookHandlerTest

Customize query in gerrit

Related Objects

Mentioned Here: T259814: SqlChangeStoreTest::testSaveChange_insert fails due to EntityChange change (call to undefined method setMetadataFromRC())
T259744: Argument 3 passed to CachingFallbackLabelDescriptionLookup::buildCacheKey() must be of the type string, null given
T259745: Uncaught ArgumentCountError: Too few arguments to function OOUI\Tag::appendContent(), 0 passed

Event Timeline

toan created this task.Aug 6 2020, 10:14 AM

Restricted Application added a project: Wikidata. · View Herald TranscriptAug 6 2020, 10:14 AM

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

Maintenance_bot moved this task from incoming to in progress on the Wikidata board.Aug 6 2020, 10:15 AM

toan moved this task from To Do (prioritised from top to bottom) to Doing on the Wikidata-Campsite (Wikidata-Campsite-Iteration-∞ (On Hold)) board.Aug 6 2020, 10:19 AM

This seems to be blocking the merging of the patch 618703 for the current UBN T259744 and maybe T259745, thus triaging this as Unbreak Now! as well.

• Addshore added a subscriber: • Addshore.Aug 6 2020, 10:50 AM

Change 618740 had a related patch set uploaded (by Tobias Andersson; owner: Tobias Andersson):
[mediawiki/extensions/Wikibase@master] Test if this fixes CI

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

gerritbot added a project: Patch-For-Review.Aug 6 2020, 12:47 PM

Adding @Pablo-WMDE as this week's Incident Manager

Had a quick chat with @toan - we are looking into this for a little longer (around 30mins) and will consider alternative steps to unblock CI should we not find a solution.

Change 618772 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/extensions/Wikibase@master] temporarily disable ArticleRevisionVisibilitySetHookHandlerTest

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

https://gerrit.wikimedia.org/r/618772 temporarily disables the tests to unblock unrelated work. We will keep full focus on the topic tomorrow.

Observation:

The data provider specified for Wikibase\Lib\Tests\Store\Sql\SqlChangeStoreTest::testSaveChange_insert is invalid.
Error: Call to undefined method Wikibase\Lib\Changes\ItemChange::setMetadataFromRC()
/workspace/src/extensions/Wikibase/lib/tests/phpunit/Store/Sql/SqlChangeStoreTest.php:58

caught my eye but also happens in green runs.

Change 618772 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] temporarily disable ArticleRevisionVisibilitySetHookHandlerTest

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

Patch with disabled test is merged.

Change 618785 had a related patch set uploaded (by Michael Große; owner: Michael Große):
[mediawiki/core@master] Clean up test fake time singleton to fix Wikibase tests

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

Change 618774 had a related patch set uploaded (by Michael Große; owner: Michael Große):
[mediawiki/extensions/Wikibase@master] Reenable ArticleRevisionVisibilitySetHookHandlerTest

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

Observation:

The data provider specified for Wikibase\Lib\Tests\Store\Sql\SqlChangeStoreTest::testSaveChange_insert is invalid.
Error: Call to undefined method Wikibase\Lib\Changes\ItemChange::setMetadataFromRC()
/workspace/src/extensions/Wikibase/lib/tests/phpunit/Store/Sql/SqlChangeStoreTest.php:58

caught my eye but also happens in green runs.

Tracked as T259814.

Error: Call to undefined method Wikibase\Lib\Changes\ItemChange::setMetadataFromRC()

Tracked as T259814.

Thank you!

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.4; 2020-08-11).Aug 6 2020, 5:00 PM

Change 618742 had a related patch set uploaded (by Michael Große; owner: Michael Große):
[mediawiki/core@master] Clean up test static fake time to fix Wikibase tests

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

Change 618785 abandoned by Michael Große:
[mediawiki/core@master] Clean up test fake time singleton to fix Wikibase tests

Reason:
I5430f4dee02216b5d5f48c3b2810f111362f3c4e is the original patch and the one that should be merged.

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

Change 618740 abandoned by Tobias Andersson:
[mediawiki/extensions/Wikibase@master] Test if this fixes CI

Reason:
Work happening in https://gerrit.wikimedia.org/r/c/618774

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

Change 618925 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[mediawiki/core@master] UserDefTest: prevent (time) sideeffect from dataProvider

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

There are two competing patches to be able to re-enable our test,
https://gerrit.wikimedia.org/r/618925 & https://gerrit.wikimedia.org/r/618949 - the original author, @nnikkhoui, is pinged on the patches to kindly advise.

Once one of them is merged, our test should be re-enabled! (c.f. https://gerrit.wikimedia.org/r/618774)

Change 618949 had a related patch set uploaded (by Michael Große; owner: Michael Große):
[mediawiki/core@master] Remove stray fake time side effect in data provider

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

Michael moved this task from Doing to Peer Review on the Wikidata-Campsite (Wikidata-Campsite-Iteration-∞ (On Hold)) board.Aug 7 2020, 12:07 PM

Change 618925 abandoned by Pablo Grass (WMDE):
[mediawiki/core@master] UserDefTest: prevent (time) sideeffect from dataProvider

Reason:
I0df26d6af56ed18eac836929fd469b4a637d4902 was merged instead, which goes one step further.

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

Change 618949 merged by jenkins-bot:
[mediawiki/core@master] Remove stray fake time side effect in data provider

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

Change 618774 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Reenable ArticleRevisionVisibilitySetHookHandlerTest

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

Change 618742 abandoned by Tobias Andersson:
[mediawiki/core@master] Clean up test static fake time to fix Wikibase tests

Reason:
Solution was to just remove the faketime. already solved.

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

ItamarWMDE moved this task from Peer Review to Test (Verification) on the Wikidata-Campsite (Wikidata-Campsite-Iteration-∞ (On Hold)) board.Aug 10 2020, 9:49 AM

Apologies everyone for being the one to merge that in and block CI. It was a big miss on my end. I had needed the fake timestamp (still should definitely not have been in the data provider to begin with). But around the same time another patch was merged that caused me not to need the faketimestamp (https://gerrit.wikimedia.org/r/c/mediawiki/core/+/616120) so removing it is definitely the better solution of the two. Thank you very much for catching it and working to merge in the fix, I hadn't seen this until Petr +2ed the fix patch.

• Addshore closed this task as Resolved.Aug 12 2020, 10:13 PM

• Addshore claimed this task.

• Addshore moved this task from Test (Verification) to Done on the Wikidata-Campsite (Wikidata-Campsite-Iteration-∞ (On Hold)) board.

Restricted Application added a project: User-Addshore. · View Herald TranscriptAug 12 2020, 10:13 PM

ArticleRevisionVisibilitySetHookHandlerTest fails on CIClosed, ResolvedPublicBUG REPORTActions

Description

Details

Related Objects

Event Timeline

ArticleRevisionVisibilitySetHookHandlerTest fails on CI
Closed, ResolvedPublicBUG REPORT
Actions