abstract class Foo {
  public static $modules = array('foo');
}
abstract class Bar extends Foo {
}

class Baz extends Bar {
}

When running Baz, WebTestBase::setUp() would call the following:
module_enable(array('foo', 'foo', 'foo'), TRUE);
And you would get the message: Enabled modules: foo, foo, foo

Here's a fix, but it likely needs a test.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Active » Needs review

I've enjoyed seeing this lately: Enabled modules: views, views_ui, views, views_ui, views, views_ui"

tim.plunkett’s picture

Issue tags: -Needs tests
FileSize
1.19 KB

Rerolled, now we have DrupalUnitTestBase as well.

tim.plunkett’s picture

#2: simpletest-1760934-2.patch queued for re-testing.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Looks pretty sane!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Seems sane. I asked Tim about test coverage and he wasn't sure of a way to test it.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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