Problem/Motivation

The purpose of the config_export_test.module is to provide a config override for ConfigExportUITest.php.

Proposed resolution

Use WebTestBase::writeSettings() instead.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug because we're using a module where we only need config.
Issue priority Normal because nothing's horribly broken.
Unfrozen changes Unfrozen because it improves/speeds up automated tests.
CommentFileSizeAuthor
#1 2409547-remove-config-export-test.diff1.84 KBznerol

Comments

znerol’s picture

znerol commented
Status: Active » Needs review
StatusFileSize
new 2409547-remove-config-export-test.diff1.84 KB
znerol’s picture

znerol commented
Issue tags: +SprintWeekend2015
mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Issue summary: View changes

Patch still applies.

ConfigExportUITest passes locally.

Restarting test here.

Mile23 queued 1: 2409547-remove-config-export-test.diff for re-testing.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Status: Needs review » Reviewed & tested by the community

RTBC because this improves the test and also removes a reference to $GLOBALS.

+++ /dev/null
@@ -1,9 +0,0 @@
-$GLOBALS['config']['system.maintenance']['message'] = 'Foo';
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Based on @xjm's post release triage document, committed be64bb6 and pushed to 8.0.x and 8.1.x. Thanks!

  • alexpott committed 7c7504c on 8.1.x 
    Issue #2409547 by znerol: Remove config_export_test.module

  • alexpott committed be64bb6 on 
    Issue #2409547 by znerol: Remove config_export_test.module

Status: Fixed » Closed (fixed)

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