and sadly, not because I'm running the test just because the tests exist on the file system. The problem is simpletest doesn't smokescreen during test discovery and it requires certain things like a @group annotation on every test at that point. Unfortunately, its not too smart about how it does that and at that point in the code it will apply its assertion to abstract classes that are not actually tests.

The attached patch renames the abstract base class so simpletest doesn't think its a test and everything is happy and we can run tests again.

tl;dr - sometimes simpletest is stupid and we have to work around it.

CommentFileSizeAuthor
dmu-fix-simpletest.patch28.29 KBneclimdul
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neclimdul’s picture

Status: Needs review » Needs work

holy crap i didn't look at some of the changes phpstorm did on the refact... i'm not sure if it was drunk or what...

Gábor Hojtsy’s picture

All of those were abstract base classes?

solideogloria’s picture

Title: Simpletest is dieing on tests » Simpletest is dying on tests
Issue summary: View changes
jcnventura’s picture

Status: Needs work » Postponed (maintainer needs more info)

Is this even applicable still?

jcnventura’s picture

Status: Postponed (maintainer needs more info) » Closed (outdated)