>> +cat >expect <<-EOF >> +labelA >> +labelB >> +EOF >> + >> +test_expect_success 'submodule add records multiple labels' ' > > The existing tests in this file may be littered with this bad > construct, but please do not add more example of running things > outside of test_expect_{success,failure} block unless there is a > good reason to do so. I thought that was the standard for tests as I have seen them quite a few times. Looking for those "cat >expect ..." constructs more closely they are often found inside tests as well. Makes sense if the expectation is used for only one test. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html