Posted by dopry on January 9, 2008 at 9:42pm
| Project: | SimpleTest |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Currently if you want to distribute tests with your module you have to write addition code in your module, which is completely superfluous if you are not running tests. This patch removes hook_simpletest completely and looks in the /tests/ folder of each module directory.
I think it is a nice simplification simply because it will remove 5 repeated lines of code in every module that wants to implement simpletests. It will also create a standard location where people know to look for/place tests.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| nohook_simpletest.patch | 1.83 KB | Ignored: Check issue status. | None | None |
Comments
#1
Oh, +1 for this! I've always thought this was rather silly.
Tested the patch and it works. There were a couple minor spacing errors that I fixed. Please don't credit me for the patch.
#2
yes, make sense
#3
Committed to both HEAD and D5 versions. Thanks dopry and webchick.
#4
#5
Er. Sorry, this one was already ported. :)
#6
Automatically closed -- issue fixed for two weeks with no activity.