Currently there are no test.
We need to test all of the system in order to maintain the module in the long run.
Especially because we extend an internal class from drupal core that could break at any moment. (We need to also fix that)
But also just in general it is easier to review patches knowing that they didn't break the system in unpredicted ways.

Issue fork config_distro-2959209

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

bircher created an issue. See original summary.

trackleft2 made their first commit to this issue’s fork.

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Version: 8.x-1.x-dev » 2.0.x-dev

trackleft2 opened merge request !9

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Active » Needs review
joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Needs review » Active
joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Active » Needs review

Trying to trigger automated testing config (just enabled)

trackleft2’s picture

trackleft2
Primary language English
Location Tucson, AZ 🇺🇸
commented

Add functional test to ensure that config distro can override/extend core config sync form.

liam morland made their first commit to this issue’s fork.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Version: 2.0.x-dev » 2.1.x-dev

I have rebased this on 2.1.x. The merge request needs to have its target changed to 2.1.x. People who can do this are a maintainer or Chris Green, since he opened the merge request.

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented

Merge request target branch updated. Thanks @liam morland!

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

All tests and checks are now passing.

trackleft2 opened merge request !30

trackleft2 changed the visibility of the branch config_distro-2959209 to hidden.

trackleft2 changed the visibility of the branch 2959209-create-rigurous-tests to hidden.

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Needs review » Reviewed & tested by the community

Latest version of MR !30 LGTM. Thanks @trackleft2

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Reviewed & tested by the community » Fixed

Merged. Thanks all!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Issue tags: +Chicago2026, +Drupalcon Chicago 2026
rachel_norfolk’s picture

rachel_norfolk
Primary language English
Location UK
commented
Issue tags: -Drupalcon Chicago 2026

Correcting the issue tag for DrupalCon

Status: Fixed » Closed (fixed)

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