Problem/Motivation

As this module is important and a basis for other projects (like drowl_layouts), while it is quite simple and might be useful to many Drupal users as core conceptual issue workaround, it should have tests for install / uninstall and functionality.

@thomas.frobieter can give examples how and what to test. I guess we'll need a testing submodule here to create the required twig theme overrides for testing.

Installation tests could live in the main module.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

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:

Comments

Anybody created an issue. See original summary.

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

grevil’s picture

I created basic install / uninstall tests and added a template module for the twig theme overrides. Let's wait for @thomas.frobieter for providing examples on how and what to test. Maybe he can also provide help for creating the required test module.

anybody’s picture

Assigned: grevil » thomas.frobieter
Status: Active » Needs review

@Grevil: Thanks a lot! As @thomas.frobieter is very busy currently, I think this will need some time.

Perhaps already merge the install / uninstall tests and afterwards set the MR to the main branch state for further additions. I think it's better to have these tests in the module now, than to wait for 1Y.

@thomas.frobieter will have a look together with you as soon as he has the time, currently the priority here isn't high enough.

grevil’s picture

Status: Needs review » Active

Will do so!

  • Grevil committed ef042a0 on 1.0.x
    Partial fix for Issue #3307617 by Grevil: Add installation tests and...