better to enable modules in setUp by passing a list back to the parent.

CommentFileSizeAuthor
#4 setup-enable-280002-3.patch1.29 KBpwolanin
#1 setup-enable-280002-1.patch655 bytespwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new setup-enable-280002-1.patch655 bytes

Also, contact and devel should not be enabled prior to the test. In general devel may not be present.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Feel free to commit this - I don't have a working SimpleTest setup currently, so I'm unable to run the tests at all.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

btw: #276751: Allow to alter/customize/add links in administration menu might need a test, too

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed
StatusFileSize
new setup-enable-280002-3.patch1.29 KB

committed the attached to 6.x and a corresponding change to 7.x

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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