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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

Status: Active » Reviewed & tested by the community
FileSize
1.82 KB

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.

moshe weitzman’s picture

yes, make sense

Rok Žlender’s picture

Status: Reviewed & tested by the community » Fixed

Committed to both HEAD and D5 versions. Thanks dopry and webchick.

webchick’s picture

Status: Fixed » Patch (to be ported)
webchick’s picture

Status: Patch (to be ported) » Fixed

Er. Sorry, this one was already ported. :)

Anonymous’s picture

Status: Fixed » Closed (fixed)

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