Patch adds support for failed builds & allows for messages as test conditions.

Currently no tests actually test these features, the particular test I had in mind is for testing drupal.org integration and failing on non-drupal.org sources.

Requires fixes to d.o integration.

CommentFileSizeAuthor
drush_make_test_redux.patch6.55 KByhahn

Comments

dmitrig01’s picture

dmitrig01 commented

Can this work with failing tests as well?

dmitrig01’s picture

dmitrig01 commented
Status: Needs review » Needs work

I committed this for now, but we should add an option like 'should fail' => TRUE

yhahn’s picture

yhahn commented

Hm, this is covered by 'build' => FALSE ... maybe the key needs to be renamed? (I was thinking "build" means "whether this drush make build completes" TRUE/FALSE)

dmitrig01’s picture

dmitrig01 commented
Status: Needs work » Fixed

yeah, nm

Status: Fixed » Closed (fixed)

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