SettingsTest::testCurrentSettingsValidate fails in quibble-vendor-mysql-php72-noselenium-docker for multiple repos
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Michael
	Apr 7 2022, 2:15 PM

Description

The error log says:

15:34:43 There was 1 failure:
15:34:43 
15:34:43 1) MediaWiki\Tests\Structure\SettingsTest::testCurrentSettingsValidate
15:34:43 Status should be OK
15:34:43 <Error, collected 10 error(s) on the way, no value set>
15:34:43 +------+---------------------------+------------------------------------------+
15:34:43 |    1 | config-invalid-key        | NamespaceContentModels | Skipping valida |
15:34:43 |      |                           | tion of object with integer keys         |
15:34:43 |    2 | config-invalid-key        | ObjectCaches | Skipping validation of ob |
15:34:43 |      |                           | ject with integer keys                   |
15:34:43 |    3 | config-invalid-key        | WANObjectCaches | Skipping validation of |
15:34:43 |      |                           |  object with integer keys                |
15:34:43 |    4 | config-invalid-key        | CanonicalNamespaceNames | Skipping valid |
15:34:43 |      |                           | ation of object with integer keys        |
15:34:43 |    5 | config-invalid-key        | ExtraNamespaces | Skipping validation of |
15:34:43 |      |                           |  object with integer keys                |
15:34:43 |    6 | config-invalid-key        | NamespacesWithSubpages | Skipping valida |
15:34:43 |      |                           | tion of object with integer keys         |
15:34:43 |    7 | config-invalid-key        | NamespaceProtection | Skipping validatio |
15:34:43 |      |                           | n of object with integer keys            |
15:34:43 |    8 | config-invalid-key        | NamespacesToBeSearchedDefault | Skipping |
15:34:43 |      |                           |  validation of object with integer keys  |
15:34:43 |    9 | config-invalid-key        | PreviewOnOpenNamespaces | Skipping valid |
15:34:43 |      |                           | ation of object with integer keys        |
15:34:43 |   10 | config-invalid-key        | LogActions | Array value found, but an a |
15:34:43 |      |                           | rray is required | array (
15:34:43   'globalauth |
15:34:43 |      |                           | /lock' => 'centralauth-log-entry-lock',
15:34:43  |
15:34:43 |      |                           |   'globalauth/unlock' => 'centralauth-.. |
15:34:43 |      |                           | .                                        |
15:34:43 +------+---------------------------+------------------------------------------+
15:34:43 
15:34:43 /workspace/src/tests/phpunit/MediaWikiTestCaseTrait.php:304
15:34:43 /workspace/src/tests/phpunit/MediaWikiTestCaseTrait.php:309
15:34:43 /workspace/src/tests/phpunit/structure/SettingsTest.php:71
15:34:43 /workspace/src/tests/phpunit/MediaWikiIntegrationTestCase.php:467
15:34:43 === Logs generated by test case
15:34:43 [objectcache] [debug] MainWANObjectCache using store {class} {"class":"EmptyBagOStuff"}
15:34:43 ===

Wikibase Lexeme, README.md change: https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php72-noselenium-docker/145629/console
Wikibase: https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php72-noselenium-docker/145631/console
Content Translation: https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php72-noselenium-docker/145635/console

Details

Subject	Repo	Branch	Lines +/-
LogActions is a map, not a list.	mediawiki/core	REL1_38	+9 -9
LogActions is a map, not a list.	mediawiki/core	master	+10 -8
Temporarily skip SettingsTest::testCurrentSettingsValidate()	mediawiki/core	master	+2 -0
Fix type of LogActions	mediawiki/core	master	+2 -2

Customize query in gerrit

Related Objects

Mentioned In: T307769: SettingsTest::testCurrentSettingsValidate fails in quibble-vendor-mysql-php72-noselenium-docker for multiple repos
Mentioned Here: T304460: Roll out loading of default settings via SettingsBuilder

Event Timeline

Michael created this task.Apr 7 2022, 2:15 PM

Michael updated the task description. (Show Details)Apr 7 2022, 2:19 PM

Since this affects multiple teams, I'm making the task Unbreak Now! Feel free to change if you disagree.

Michael removed projects: Wikidata, Wikidata Lexicographical data.Apr 7 2022, 2:23 PM

Addshore added a project: Platform Engineering.Apr 7 2022, 2:35 PM

Addshore added a project: MediaWiki-Configuration.

Probably related to @daniel’s work on T304460.

I can reproduce this locally, by the way, with 11 errors in the status (CapitalLinkOverrides seems to be the extra one compared to the task description; I’m not sure why, since I don’t have it in my LocalSettings and grepping for it in extensions/ and skins/ doesn’t find anything either).

If I comment out StatusValue::warning(), the one non-warning error seems to be the LogActions one: “Array value found, but an array is required”

I think the nonsensical error message “Array value found, but an array is required” actually means that it requires a non-associative array.

vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/TypeCheck/LooseTypeCheck.php

public static function isArray($value)
{
    return
        is_array($value) &&
        (count($value) == 0 || !self::isAssociativeArray($value));
}

And LogActions is something like [ 'rights/erevoke' => 'rights-editor-revoke' ] (at least on my system), which is associative, hence not an array according to justinrainbow/json-schema.

editor-revoke seems to come from FlaggedRevs; when I remove that in my LocalSettings (why, for the love of all that is holy, did I have this accursed extension enabled?!), the test passes. In CI, it sounds like CentralAuth is responsible for the associative array instead.

Change 778309 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/core@master] Temporarily skip SettingsTest::testCurrentSettingsValidate()

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

gerritbot added a project: Patch-For-Review.Apr 7 2022, 3:42 PM

Change 778311 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/core@master] Fix type of LogActions

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

In T305635#7838633, @gerritbot wrote:

Change 778311 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/core@master] Fix type of LogActions

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

Testing this one via Depends-On at https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikibaseLexeme/+/778290; if it works, we can abandon the change that skips the test (unless you think that’s the safer course of action, because who knows how many other incorrect types are waiting to be discovered in extensions).

Note that all the "Skipping validation" messages are just warnings. They do nto make the test fail. The failues is for LogAction.

"LogActions | Array value found, but an array is required" is a very confusing message, it comes from the JSON Schema validator. It really means "string key found in what is supposed to be a sequential list".

Looking at LogPage::actionText(), LogAction is not a sequential list, but a map. I'll fix that in the MainConfigSchema.

Change 778313 had a related patch set uploaded (by Daniel Kinzler; author: Daniel Kinzler):

[mediawiki/core@master] LogActions is a map, not a list.

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

Change 778311 abandoned by Lucas Werkmeister (WMDE):

[mediawiki/core@master] Fix type of LogActions

Reason:

superseded by I681b2d7004

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

Change 778309 abandoned by Lucas Werkmeister (WMDE):

[mediawiki/core@master] Temporarily skip SettingsTest::testCurrentSettingsValidate()

Reason: