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
Comment #1
dave reidAs 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.
Comment #2
dave reidFixed in http://drupal.org/cvs?commit=367680 and http://drupal.org/cvs?commit=367684. Thanks Kiam!