Problem/Motivation

As per #2183983: Find hidden configuration schema issues most of the remaining fails are either in migrate or config itself. This issue is for the config fails. See #2358269: Migration bugs in block visibility, field overrides, cron, maintenance settings and form modes found by configuration schema checking for migrate.

Proposed resolution

1. Add strict checking to affected config system tests.
2. Fix the fails.

Remaining tasks

Fix fails. Review. Commit.

User interface changes

None.

API changes

Config schemas fixed as appropriate.

Comments

gábor hojtsy’s picture

StatusFileSize
new6.75 KB

Status: Needs review » Needs work

The last submitted patch, 1: 2391021-config-schema.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new5.36 KB
new11.63 KB

1. Making some exemptions in schema checking for things that are not or only partially schema covered *by design*.
2. Add a whole bunch of missing things for tested data structures in config_test.schema.yml.
3. Add missing schemas for config_collection_install_test and config_events_test.
4. Add config_test module as a test dependency to ConfigFileContentTest so we have the schema available.

Only one fail seems to be left locally in ConfigDependencyTest. That does not look trivial to me :/ Maybe I am overlooking something.

Status: Needs review » Needs work

The last submitted patch, 3: 2391021-config-schema-3.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new12.66 KB
new1.96 KB

Ok CRUD test has some more stuff.

Status: Needs review » Needs work

The last submitted patch, 5: 2391021-config-schema-5.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new10.83 KB
new3.77 KB

There are too many intricate details of how the tests test schema-less elements, so some config tests will *need* to be exempt from config schema checking to be able to stay sane.

Status: Needs review » Needs work

The last submitted patch, 7: 2391021-config-schema-7.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new13.72 KB
new2.89 KB

Use config_test.schemaless in the tests where we do not want to exempt config_test.noschema from testing, so we can keep it exempt at other config tests while still testing the rest.

Status: Needs review » Needs work

The last submitted patch, 9: 2391021-config-schema-9.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new14.18 KB
new475 bytes

Finally the ConfigQueryTest was initializing the array as null, not an empty array. To fix that for schema, initialize as array.

Status: Needs review » Needs work

The last submitted patch, 11: 2391021-config-schema-10.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Needs review
StatusFileSize
new12.7 KB
gábor hojtsy’s picture

StatusFileSize
new13.87 KB

Duh, rolled that wrong. Here is the right one.

The last submitted patch, 13: 2391021-config-schema-12.patch, failed testing.

gábor hojtsy’s picture

StatusFileSize
new13.76 KB
new584 bytes

We also don't need the config_test module in ConfigFileContentTest which was only added so we can provide schema, but the things tested there are crazy to provide schema for.

gábor hojtsy’s picture

Issue tags: +Ghent DA sprint
alexpott’s picture

Status: Needs review » Reviewed & tested by the community

This looks good to go. @Gábor Hojtsy discussed if it was worth moving the default config_schema_test.noschema.yml to another module but I think not.

gábor hojtsy’s picture

StatusFileSize
new13.76 KB

Reuploading for bot.

The last submitted patch, 16: 2391021-config-schema-15.patch, failed testing.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed ae049e9 on 8.0.x
    Issue #2391021 by Gábor Hojtsy: Config schema issues in config tests...
gábor hojtsy’s picture

Thanks!

The last submitted patch, 16: 2391021-config-schema-15.patch, failed testing.

The last submitted patch, 16: 2391021-config-schema-15.patch, failed testing.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.