Page MenuHomePhabricator

GrowthExperiments\Tests\Integration\SuggestedEditsSchemaTest::testDefaultTaskTypesDataWithEmptyConfig: Failed asserting that two arrays are equal
Closed, ResolvedPublic

Description

Seen in https://integration.wikimedia.org/ci/job/integration-quibble-fullrun-extensions-phpunit/539/console#console-section-20 and I can reproduce it locally as well.

09:47:11 There was 1 failure:
09:47:11 
09:47:11 1) GrowthExperiments\Tests\Integration\SuggestedEditsSchemaTest::testDefaultTaskTypesDataWithEmptyConfig
09:47:11 Failed asserting that two arrays are equal.
09:47:11 --- Expected
09:47:11 +++ Actual
09:47:11 @@ @@
09:47:11  Array (
09:47:11 -    0 => Array (...)
09:47:11 -    1 => Array (...)
09:47:11 -    2 => Array (...)
09:47:11 -    3 => Array (...)
09:47:11 -    4 => Array (...)
09:47:11 -    5 => Array (...)
09:47:11 -    6 => Array (...)
09:47:11 -    7 => Array (...)
09:47:11  )
09:47:11 
09:47:11 /workspace/src/extensions/GrowthExperiments/tests/phpunit/integration/Config/Schemas/SuggestedEditsSchemaTest.php:188
09:47:11 phpvfscomposer:///workspace/src/vendor/phpunit/phpunit/phpunit:106

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
kostajh triaged this task as Unbreak Now! priority.Jun 19 2024, 9:38 AM
kostajh updated the task description. (Show Details)

Marking as UBN as it prevents merges to Quibble.

Strange. Is CommunityConfiguration not part of quibble? We should change that as it is (going to be) a hard dependency for GrowthExperiments. But first I'll upload a change to unbreak this.

Change #1047466 had a related patch set uploaded (by Michael Große; author: Michael Große):

[mediawiki/extensions/GrowthExperiments@master] tests: skip test that depends CommunityConfiguration if it is not loaded

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

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

[integration/quibble@master] ci-fullrun-extensions: Load CommunityConfiguration extension

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

Strange. Is CommunityConfiguration not part of quibble? We should change that as it is (going to be) a hard dependency for GrowthExperiments. But first I'll upload a change two unbreak this.

Thanks @Michael! I made ci-fullrun-extensions: Load CommunityConfiguration extension for that.

kostajh lowered the priority of this task from Unbreak Now! to Needs Triage.Jun 19 2024, 10:15 AM

Change #1047466 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] tests: skip test that depends CommunityConfiguration if it is not loaded

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

Change #1047469 merged by jenkins-bot:

[integration/quibble@master] ci-fullrun-extensions: Load CommunityConfiguration extension

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

Michael claimed this task.

This has been fixed a while ago.