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.
Comment | File | Size | Author |
---|---|---|---|
dmu-fix-simpletest.patch | 28.29 KB | neclimdul | |
Comments
Comment #1
neclimdulholy 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...
Comment #2
Gábor HojtsyAll of those were abstract base classes?
Comment #3
solideogloria CreditAttribution: solideogloria commentedComment #4
jcnventura CreditAttribution: jcnventura at 1xINTERNET commentedIs this even applicable still?
Comment #5
jcnventura CreditAttribution: jcnventura at 1xINTERNET commented