Problem/Motivation

Several configuration elements have tests for schema compliance but not all. Using capabilities in latest 8.x and the config_inspector module, people can review compliance of the configuration at any point in time on their site with the schema. While there may be (and likely are) holes in the schema for a well developed site, even a default installation of Drupal 8 has schema holes.

Reviewing the errors found by config_schema, the following issues were found on a default standard install:

entity.form_display.node.article.default
content.path.settings (array) missing schema
entity.form_display.node.page.default
content.path.settings (array) missing schema
field.field.node.field_image
settings.target_type, settings.display_field, settings.display_default and settings.target_bundle missing schema
field.field.node.field_tags
settings.target_type and settings.target_bundle missing schema
field.field.user.user_picture
settings.target_type, settings.display_field, settings.display_default and settings.target_bundle missing schema
field.instance.node.article.field_image
settings.handler missing schema
field.instance.user.user.user_picture
settings.handler missing schema
menu.entity.node.article
parent missing schema
menu.entity.node.node
parent missing schema
menu.entity.node.page
parent missing schema
shortcut.set.default
links key Variable type is NULL but applied schema class is Drupal\Core\Config\Schema\Sequence.

Testing was added which found some more issues with the following:

entity.form_display.***
content.path.settings: Missing schema
field.field.**:settings.translation_sync and field.instance.**:settings.translation_sync
settings.translation_sync: Missing schema
migrate.migration.*
Following top level items are only added dynamically, lack schema: row, idMap, sourceIds, destinationIds, highwaterProperty, systemOfRecord, sourceRowStatus, trackLastImported.

Additional issues in views were identified and moved to #2301045: Standard profile has views which include elements dependent on uninstalled modules, not valid in config.

Proposed resolution

Add missing schemas.
Keep the test to ensure that when all modules are enabled, all schemas are good.
Exempt translation_sync for now.

Remaining tasks

Decide if enabling all modules in the environment (maybe even contrib modules) is find.
Decide if the translation_sync workaround is fine.

User interface changes

None.

API changes

None.

Files: 
CommentFileSizeAuthor
#50 2295737.50.patch8.89 KBalexpott
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,327 pass(es). View
#50 46-50-interdiff.txt471 bytesalexpott
#46 2295737-missing-config-schema-46.patch8.21 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,374 pass(es), 1 fail(s), and 0 exception(s). View
#46 interdiff.txt1.14 KBGábor Hojtsy
#43 interdiff.txt1.91 KBGábor Hojtsy
#43 2295737-missing-config-schema-43.patch8.23 KBGábor Hojtsy
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,779 pass(es). View
#41 interdiff.txt1.92 KBGábor Hojtsy
#41 2295737-missing-config-schema-41.patch7.82 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,432 pass(es), 2 fail(s), and 0 exception(s). View
#37 2295737-missing-config-schema-37.patch7.82 KBGábor Hojtsy
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,790 pass(es). View
#37 interdiff.txt4.45 KBGábor Hojtsy
#36 interdiff.txt2.9 KBGábor Hojtsy
#36 2295737-missing-config-schema-36.patch5.6 KBGábor Hojtsy
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,745 pass(es). View
#33 interdiff.txt759 bytesGábor Hojtsy
#33 2295737-missing-config-schema-32.patch6.24 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,742 pass(es), 5 fail(s), and 0 exception(s). View
#33 jrLEZP4.png26.83 KBGábor Hojtsy
#29 interdiff.txt753 bytesGábor Hojtsy
#29 2295737-missing-config-schema-29.patch6.62 KBGábor Hojtsy
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,590 pass(es). View
#26 interdiff.txt1 KBGábor Hojtsy
#26 2295737-missing-config-schema-26.patch5.88 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,575 pass(es), 17 fail(s), and 0 exception(s). View
#24 interdiff.txt598 bytesGábor Hojtsy
#24 2295737-missing-config-schema-24.patch5.4 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,568 pass(es), 23 fail(s), and 0 exception(s). View
#23 interdiff.txt1.19 KBGábor Hojtsy
#23 2295737-missing-config-schema-23.patch5.98 KBGábor Hojtsy
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,570 pass(es), 49 fail(s), and 0 exception(s). View
#19 2295737-diff-14-19.txt1.07 KBvijaycs85
#19 2295737-missing-config-schema-19.patch5.33 KBvijaycs85
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,770 pass(es), 55 fail(s), and 0 exception(s). View

Comments

Gábor Hojtsy’s picture

Discussed the display.default.display_options.filters.langcode.value missing schema one with @alexpott. We figured the language filter (Drupal\language\Plugin\views\filter\LanguageFilter) is not actually dependent on language module, so can move to views with its schema, so it would be available generally.

Also discussed the other two Views ones with @alexpott and as per him, discussion in Austin resulted in that optional dependencies like that in config entities will not be possible. Content translation will need to provide an alternate view on its own for the front page. So if we consider that translation link feature important enough to warrant a new view, we should make a copy of the front page view and the user admin view in content translation and provide it from there. Then remove these fields from the standard views. BTW #2161845: Node views (front page, admin) do not use the proper language filter proposes to make the front page view even more tailored for language.

These changes will allow to provide a test for all the things. Again @alexpott suggests we put the test in ConfigImportAllTest since it does all the prepwork.

So those should provide the way for the trickier parts of the issue. Hopefully the other ones are not optional dependencies :)

vijaycs85’s picture

Status: Active » Needs review
FileSize
2.75 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 72,777 pass(es). View

Solves issues of below items:

entity.form_display.node.article.default
content.path.settings (array) missing schema
entity.form_display.node.page.default
content.path.settings (array) missing schema
field.field.node.field_image
settings.target_type, settings.display_field, settings.display_default and settings.target_bundle missing schema
field.field.node.field_tags
settings.target_type and settings.target_bundle missing schema
field.field.user.user_picture
settings.target_type, settings.display_field, settings.display_default and settings.target_bundle missing schema
field.instance.node.article.field_image
settings.handler missing schema
field.instance.user.user.user_picture
settings.handler missing schema
menu.entity.node.article
parent missing schema
menu.entity.node.node
parent missing schema
menu.entity.node.page
parent missing schema
Gábor Hojtsy’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests

Looks good. Are you up to continuing with the views issues as per above? :) That would allow to add a test as well so all standard profile config is tested. Needs work to add the remaining things, not because I found anything wrong :)

vijaycs85’s picture

Just for update on remaining 3 items:

shortcut.set.default - links key Variable type is NULL but applied schema class is Drupal\Core\Config\Schema\Sequence.

Not used. created new issue #2296835: Remove 'links' config element from shortcut set config file to remove this.

views.view.content - display.default.display_options.filters.langcode.value missing schema.

Need to move this out of language module as per #1

views.view.content - display.default.display_options.fields.translation_link.text missing schema.
views.view.user_admin_people - display.default.display_options.fields.translation_link.text missing schema.

translation_link will be removed from default view as per #1

So we just need to add test to make sure the patch in #2 works.

Gábor Hojtsy’s picture

Why not fix all 3 here? Then we can add an overall test on the standard profile and ensure it will not fail again at least for shipped standard config. I consider that way better compared to keep doing one-off fixes and hope nobody introduces more bugs, while they in fact do...

Gábor Hojtsy’s picture

Title: Missing schemas for default configuration » Not all configuration in standard profile passes validation
Status: Needs work » Needs review
Issue tags: -Needs tests
FileSize
1.09 KB
3.05 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 72,883 pass(es), 1 fail(s), and 0 exception(s). View

So I think the scope of this issue is to make all standard configuration pass validation, period. I think the attached test would theoretically prove that. I did not test it though :D Retitled to make it clearer. Adding the test is the only way to make this not happen again and again.

Status: Needs review » Needs work

The last submitted patch, 6: 2295737-missing-config-schema-4.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
3.06 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,087 pass(es), 671 fail(s), and 0 exception(s). View
813 bytes

How did I not recognise that PHP error :D

Status: Needs review » Needs work

The last submitted patch, 8: 2295737-missing-config-schema-8.patch, failed testing.

Gábor Hojtsy’s picture

Woah lots more fails.

- entity.form_display.***:content.path.settings: Missing schema.
- field.field.**:settings.translation_sync: Missing schema.
- field.instance.**:settings.translation_sync: Missing schema.
- migrate.migration.*: BAZILLIONS of top level items across lots of files: Missing schema.

Most bizarre that it DID NOT fail on the shortcut or views that we expected. Heh.

Gábor Hojtsy’s picture

Oh, there is a pattern to the migrate fails. These are not in the migration files shipped but I guess they get added when imported. Top level keys: row, idMap, sourceIds, destinationIds, highwaterProperty, systemOfRecord, sourceRowStatus, trackLastImported. All of these are missing schema for all of the migrate files, so it fails 8 times for all migrate files. No wonder the hundreds of fails.

Gábor Hojtsy’s picture

Title: Not all configuration in standard profile passes validation » Not all shipped configuration passes validation even with all modules enabled

So this test actually enabled all the modules so we are testing against a full site. So the optional dependencies will not come up here and we can move that to a different issue :) At least this will ensure if ALL modules are enabled then the schemas are correct for all shipped config, which is very valuable in itself. We should open an issue for the views problems, but the rest should still be fixed there to make the test green :)

vijaycs85’s picture

Status: Needs work » Needs review
FileSize
5.33 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 70,176 pass(es), 1,047 fail(s), and 130 exception(s). View
2.28 KB

Thanks @Gábor Hojtsy for the great tests:

- field.field.**:settings.translation_sync: Missing schema.
- field.instance.**:settings.translation_sync: Missing schema.

Won't be fixed until #2224761: Add a generic way to add third party configuration on configuration entities and implement for field configuration gets in. There are 17 fails associated with this.

P.S: not sure, if it is good idea to scan all the available modules (excluding hidden/disabled module). As this would get a situation that this test never pass in a typical website with bunch of custom/contrib modules. I found it hard to make the test work with config_devel module in my /modules dir (not enabled).

entity.form_display.node.article.default:content.path.settings: Missing schema.
...
...

We have fix for this in #2 and missed somehow. Adding it back here.

Reg: #11 - Yes, Adding them with 'ignore' for now.

vijaycs85’s picture

FileSize
5.34 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 70,166 pass(es), 1,047 fail(s), and 130 exception(s). View
516 bytes

Typo dMap => idMap

Status: Needs review » Needs work

The last submitted patch, 14: 2295737-missing-config-schema-14.patch, failed testing.

The last submitted patch, 14: 2295737-missing-config-schema-14.patch, failed testing.

The last submitted patch, 13: 2295737-missing-config-schema-13.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review
FileSize
5.33 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,770 pass(es), 55 fail(s), and 0 exception(s). View
1.07 KB

Status: Needs review » Needs work

The last submitted patch, 19: 2295737-missing-config-schema-19.patch, failed testing.

The last submitted patch, 19: 2295737-missing-config-schema-19.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
5.98 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,570 pass(es), 49 fail(s), and 0 exception(s). View
1.19 KB

Fixing all the migrate fails. The defaults for file field display settings were not the right type. The schema definition on images was also not the right type, checked with the form code that it is a checkbox on images as well. The rest not fixed yet.

Gábor Hojtsy’s picture

FileSize
5.4 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,568 pass(es), 23 fail(s), and 0 exception(s). View
598 bytes

Most of the rest of the new fails were due to the workaround for translation sync, since it was not picking up the field specific elements anymore. I looked into introducing a field_settings base type but there are no common settings between all the fields whatsoever. translation_sync would be (also on field instances) but that is better dealt with in a test exception I think.

This I think may still fail on entity display issues (additionally to all the translation_sync fails).

Status: Needs review » Needs work

The last submitted patch, 24: 2295737-missing-config-schema-24.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
5.88 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,575 pass(es), 17 fail(s), and 0 exception(s). View
1 KB

The shortcut schema includes the list of shortcuts element which is not there anymore, the shortcuts are now content entities and not stored in the config entity. Fixed that.

The entity display schema for the path field was defined for the entity *view* display not for the form display. There were never fails for the view display, but there are for the forms. The schema looks correct for the form although I'm not 100% positive on how to verify, the form has these elements. So I think vijaycs85 just misnamed the element in the schema.

This should now only fail on translation_sync.

The last submitted patch, 23: 2295737-missing-config-schema-23.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 26: 2295737-missing-config-schema-26.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
6.62 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,590 pass(es). View
753 bytes

All right, this one should fix all remaining fails. I opted to have test exceptions for the translation_sync setting only. This ensures that all other parts of all other files are tested. I think it would be really valuable to get this one in with the workaround and remove later.

Gábor Hojtsy’s picture

@vijaycs85 pointed out that we should not remove the links element from the shortcut set because a null is still written out somehow for that element, so we need to figure out why. The reason ironically is the schema itself. Now the schema is used to persist config entities, and since the links element was there, even though shortcut sets don't have that anymore, the process looks up that property and takes a null value, since it is not there. That results in a null export due to the schema. So we need to remove the schema (which results in no links element exported).

Gábor Hojtsy’s picture

Assigned: Unassigned » alexpott
Issue summary: View changes

Updated the issue summary. Opened #2301045: Standard profile has views which include elements dependent on uninstalled modules, not valid in config for the views issues. Once this is fixed, only that should remain for standard profile. Remaining tasks now accurate, with the two questions I think we need to figure out. Assigning this to alexpott as CMI maintainer for those two questions.

andypost’s picture

+++ b/core/modules/path/config/schema/path.schema.yml
@@ -0,0 +1,22 @@
+entity_form_display.field.path:

how that possible? this is a form structure of widget but path widget does not have settings!

Gábor Hojtsy’s picture

FileSize
26.83 KB
6.24 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,742 pass(es), 5 fail(s), and 0 exception(s). View
759 bytes

@andypost points out path form displays don't have settings (see image by @andpost). Righto.

Status: Needs review » Needs work

The last submitted patch, 33: 2295737-missing-config-schema-32.patch, failed testing.

alexpott’s picture

  1. +++ b/core/lib/Drupal/Core/Config/Schema/SchemaCheckTrait.php
    @@ -80,6 +80,11 @@ protected function checkValue($key, $value) {
    +      // @todo Temporary workaround for https://www.drupal.org/node/2224761.
    +      $key_parts = explode('.', $key);
    +      if (array_pop($key_parts) == 'translation_sync') {
    +        return array();
    +      }
    

    let's check $this->configName as well here - just in case.

  2. +++ b/core/modules/config/src/Tests/ConfigImportAllTest.php
    @@ -123,5 +125,14 @@ public function testInstallUninstall() {
    +    $names = $this->container->get('config.storage')->listAll();
    +    $factory = $this->container->get('config.factory');
    +    /** @var \Drupal\Core\Config\TypedConfigManagerInterface $typed_config */
    +    $typed_config = $this->container->get('config.typed');
    +    foreach ($names as $name) {
    +      $config = $factory->get($name);
    +      $this->assertConfigSchema($typed_config, $name, $config->get());
    +    }
    

    Let's have a comment about why we're doing this - since it is not the point of the test.

  3. +++ b/core/modules/image/config/schema/image.schema.yml
    @@ -82,6 +82,18 @@ field.image.settings:
    +    target_type:
    +      type: string
    +      label: 'Target type'
    +    display_field:
    +      type: boolean
    +      label: 'Display field'
    +    display_default:
    +      type: boolean
    +      label: 'Displayed by default'
    ...
    +      type: string
    +      label: 'Target bundle'
    
    @@ -117,6 +129,9 @@ field.image.instance_settings:
    +    handler:
    +      type: string
    +      label: 'Handler'
    

    These are coming from the FileItem field type so they are missing from there too. In fact couldn't we inherit from there like the code does. And that should inherit from entity_reference? The same thing goes for the instance settings.

  4. +++ b/core/modules/migrate/config/schema/migrate.schema.yml
    @@ -38,3 +38,27 @@ migrate.migration.*:
    +    row:
    +      type: ignore
    +      label: 'Row'
    +    idMap:
    +      type: ignore
    +      label: 'ID Map'
    +    sourceIds:
    +      type: ignore
    +      label: 'Source IDs'
    +    destinationIds:
    +      type: ignore
    +      label: 'Destination IDs'
    +    highwaterProperty:
    +      type: ignore
    +      label: 'Property'
    +    systemOfRecord:
    +      type: ignore
    +      label: 'System of record'
    +    sourceRowStatus:
    +      type: ignore
    +      label: 'Sourc row status'
    +    trackLastImported:
    +      type: ignore
    +      label: 'Tracked last import'
    

    I suspect that this is because Migration config entity was exporting public properties - this is fixed now - should not be necessary.

  5. +++ b/core/modules/taxonomy/config/schema/taxonomy.schema.yml
    @@ -54,6 +54,12 @@ field.taxonomy_term_reference.settings:
    +    target_type:
    +      type: string
    +      label: 'Target type'
    +    target_bundle:
    +      type: string
    +      label: 'Target bundle'
    

    This should inherit from the entity reference settings so the schema is doing the same

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
5.6 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,745 pass(es). View
2.9 KB

- Fixed the path schema so it again contains a settings key (but no element definition since no elements are allowed).
- Added more specifics with config names to the workaround in the test on translation_sync
- Add comment on test to ConfigImportAllTest
- Remove the unnecessary elements from the migrate schema, we'll see :D

Gábor Hojtsy’s picture

FileSize
4.45 KB
7.82 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,790 pass(es). View

So looked into the common properties to ER, taxonomy, files and images.

Issues found with entity reference:
- target_bundle was missing from schema, but in settings
- the handler_settings in instance settings is only on this type, altered in, not on the other types, so we cannot inherit from the instance settings; we could add a base type but for the handler key only that sounds a bit superfluous

Issues found with taxonomy:
- the instance settings should be a mapping not a sequence, has a handler key

Issues found with file:
- has most instance settings common with images but the display_field is only here (unset in image), so a new type was needed

Issues found with image:
- instance settings should depend on base type from file

I think this covers the data properly now :) Looking forward to test run.

andypost’s picture

once entity reference module enabled ER fields could use array for target_bundles

+++ b/core/modules/entity_reference/config/schema/entity_reference.schema.yml
@@ -2,11 +2,14 @@
 field.entity_reference.settings:
...
+    target_bundle:
+      type: string
+      label: 'Budle of item to reference'

Actually there's 2 different settings: target_bundle and target_bundles

Gábor Hojtsy’s picture

@andypost: well, currently it is a string. #2064191: Fix the 'target_bundle' field setting - 'target_bundles' are not validated when entity_reference is enabled says it should be an array, but there is no such array on the entity reference settings top level now is there? There *is* a target_bundles on the handler settings, that is already covered by the schema as well that I see. So I think you are pointing out possible future changes and not issues with the patch?

Gábor Hojtsy’s picture

All right, in IRC, @anypost pointed to this:

class ConfigurableEntityReferenceItem extends EntityReferenceItem implements AllowedValuesInterface {

  public static function defaultSettings() {
    $settings = parent::defaultSettings();
    // The target bundle is handled by the 'target_bundles' property in the
    // 'handler_settings' instance setting.
    unset($settings['target_bundle']);
    return $settings;
  }

That's not schema compatible at all. Schema cannot be written for something and then depend on the class implementing it. Unless the data would somehow indicate which one it is, but here it tries to disguise as the other one. Lol.

function entity_reference_field_info_alter(&$info) {
  // ...  
  $info['entity_reference']['class'] = '\Drupal\entity_reference\ConfigurableEntityReferenceItem';
  // ...
}

I don't think we have any other way right now but to keep that property on the schema because we cannot know in any way which implementation is running or which implementation a config file belongs to for that matter.

Also that does not result in test fails here in any way and I would avoid turning this into a "fix everything related" issue :) The added tests will be very valuable and should keep people in check at least for more obvious mistakes.

Gábor Hojtsy’s picture

FileSize
7.82 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,432 pass(es), 2 fail(s), and 0 exception(s). View
1.92 KB

So all the things I found in this exploration:
- entity_reference field type is in fact defined by code, but no schema is provided by core
- the configurable entity reference alters in implementation for entity_reference AND defines the schema
- since they use the same type name, there is impossible to provide distinct schemas for them
=> so we need to keep not providing a schema for the base type

- the configurable entity reference removes the target_bundle key
=> so it should not be on its schema

- file and taxonomy module are inheriting from the core entity_reference implementation, they do not depend on the entity reference *module*
=> they will not have access to inherit from their schema, so we need to start their schemas fresh (we can keep the file -> image inheritance)

In short very flexible magic things that the schema is not fully ready to follow.

Status: Needs review » Needs work

The last submitted patch, 41: 2295737-missing-config-schema-41.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
8.23 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,779 pass(es). View
1.91 KB

So taxonomy should have had target_type as well. Oh well. We can introduce a base type for the combo of target bundle and type then BUT not use it with the actual configurable entity reference schema because that does not have the target bundle. Yeah.

Gábor Hojtsy’s picture

Postponed #2301045: Standard profile has views which include elements dependent on uninstalled modules, not valid in config on this one, since its test would result in overlapping problems with this one. We can write a test for that once these issues are fixed. Let's get it in? :)

alexpott’s picture

Status: Needs review » Needs work
+++ b/core/modules/system/config/schema/system.schema.yml
@@ -391,3 +391,13 @@ condition.plugin.request_path:
+base_entity_reference_field_settings:
+  type: mapping
+  mapping:
+    target_type:
+      type: string
+      label: 'Type of item to reference'
+    target_bundle:
+      type: string
+      label: 'Bundle of item to reference'

This belongs in the core schema file. Since the EntityReferenceItem is a core class.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
1.14 KB
8.21 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,374 pass(es), 1 fail(s), and 0 exception(s). View

Works for me. Moved there.

alexpott’s picture

Status: Needs review » Reviewed & tested by the community

Thanks.

Move coverage of config using schema to test FTW

alexpott’s picture

Assigned: alexpott » Unassigned

I rtbc'd so I can't commit

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 46: 2295737-missing-config-schema-46.patch, failed testing.

alexpott’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
471 bytes
8.89 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 74,327 pass(es). View

Fixed up migration file - new key added by #2202511: Implement migration groups

Gábor Hojtsy’s picture

Yay for tests! Yay!

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • catch committed e25a1ec on 8.x
    Issue #2295737 by Gábor Hojtsy, vijaycs85, alexpott: Fixed Not all...
Gábor Hojtsy’s picture

Yay, thanks!

Status: Fixed » Closed (fixed)

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