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 commentedThe simpletest module is obsolete in core. Moving to SimpleTest project.