Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The current installer tests, like InstallerTranslationTest, have to override Setup() and then copy and modify large swaths of code from base classes. This makes it adding new tests time consuming and the process voodoo-like.
Let's refactor some underlying base classes so that the installer tests can override specific methods instead of the entire Setup method.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2067919-1-refactor-installer-tests.patch | 18.94 KB | JohnAlbin |
Comments
Comment #1
JohnAlbinHere's a rough draft patch that probably needs work.
In addition to refactoring the various classes, it also implements a InstallerRequirementsTest that checks for directory/file permission error messages during installation. That use-case drove a lot of the refactoring in this patch.
Comment #2
JohnAlbinHere's another installer issue; the latest patch in #29 requires the refactoring in this issue. You can see the new InstallerMobileMetaTagsTest for that is much simpler then the current InstallerTranslationTest in core. https://drupal.org/files/1887800-29-installer-mobile-meta.patch
Comment #14
quietone CreditAttribution: quietone at PreviousNext commentedThe simpletest module is obsolete in core. Moving to SimpleTest project.