Johannes Schindelin <johannes.schindelin@xxxxxx> writes:
>> +test_expect_success 'fail if wildcard spec does not match any refs' '
>> + git checkout -b test copy^ &&
>> + test_when_finished "git checkout -f copy && git branch -D test" &&
>
> When I read this line, I immediately asked myself whether the
> branch would be deleted even if the test case failed. I then
> tested this theory by editing the first test case ("setup") like
> this:
> ...
> and indeed, the file "file" was gone, even if the test case
> failed. I therefore believe that this "test_when_finished" cleanup
> might make debugging substantially harder. Maybe we can drop these
> lines from this patch?
The test framework is aware of the fact that it needs to help the
people who are debugging the scripts. The support is limited to the
case in which you run it under the -i option, i.e.
$ cd t
$ sh ./t5520-pull.sh -i -v
will refrain from running test_when_finished scripts when the test
piece fails. Even though this is only limited to -i, I found it
often sufficient for debugging.
--
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