If you try return simpletest as module to be enabled as part of install profile, then there is an error reported on installation while simpletest.install is checking requirements. simpletest_requirements function assumed drupal_get_path can be called but database is not initialized yet.

Attached patch avoids calling drupal_get_path

I realize drupal trunk, this is no longer relevant.

CommentFileSizeAuthor
#1 simpletest-install-1.patch863 bytespwolanin
simpletest-check-requirements-on-install-profile.patch715 bytesdlhubler

Comments

pwolanin’s picture

pwolanin commented
Priority: Normal » Critical
StatusFileSize
new simpletest-install-1.patch863 bytes

I independently found essentially the same solution. I'm not sure if it matters, but this patch only uses the PHP dirname() if the call to druapl_get_path() does not return a result.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
cwgordon7’s picture

cwgordon7 commented
Status: Needs review » Reviewed & tested by the community

Looks good. Works on a fresh install with an altered profile.

cwgordon7’s picture

cwgordon7 commented

(RTBC @ #1)

pwolanin’s picture

pwolanin commented

Since either patch will work, I think the choice is just a matter of taste.

boombatower’s picture

boombatower commented
Status: Reviewed & tested by the community » Fixed

Committed.

Thanks everyone.

Anonymous’s picture

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

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