The code for setUp() contained in the helper class contains the following code:

function setUp() {
    // Call parent::setUp() allowing test cases to pass further modules.
    $modules = func_get_args();
    $modules[] = 'xmlsitemap';
    call_user_func_array(array($this, 'parent::setUp'), $modules);

The same function contained in the classes derived from that contains parent::setUp('xmlsitemap'), with the result that the array <code>$modules will contain the same module name twice.

Also, it should probably better to add xmlsitemap at the top of the array, in the case the subclass is trying to add a module that depends from xmlsitemap.module. As far as I can understand, SimpleTest doesn't enables the modules respecting the dependencies between them (or is it that done from Drupal core code?).

Comments

dave reid’s picture

As far as I can tell it looks like http://api.drupal.org/api/function/drupal_install_modules/6 does dependency and weight checking for D6 (D7 tests use module_enable which does the same). Still, its redundant code.

dave reid’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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