My concern is in changing the currently assumed behavior of apply_manifest - this is used a lot in currently built acceptance tests.

Could test_manifest just push either catch_failures/expect_failures and do away with the test_return?  I believe that that was the original goal of test_manifest, to be a helper method so that you have cleaner calls in beaker-rspec.  Changing the behavior of apply_manifest was not part of the plan.