Aside from one tiny formatting nit (see below), the test looks good to me. On the other hand, this is kind of an "aspirational test"; I don't know that the tested functionality has ever worked or that anybody has ever claimed that it works. So my feeling is that the addition of the test would feel more natural in the patch series that implements the new feature. But I don't feel strongly about it. Michael On 11/26/2014 01:59 AM, Stefan Beller wrote: > From: Ronnie Sahlberg <sahlberg@xxxxxxxxxx> > > Signed-off-by: Ronnie Sahlberg <sahlberg@xxxxxxxxxx> > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > --- > Changes v1->v2 > * relocated the test from t1402 to t3200 > * reword the commit message title to fit in with similar commits touching > t/t3200-branch.sh > > t/t3200-branch.sh | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/t/t3200-branch.sh b/t/t3200-branch.sh > index 432921b..fa7d7bd 100755 > --- a/t/t3200-branch.sh > +++ b/t/t3200-branch.sh > @@ -163,6 +163,14 @@ test_expect_success 'git branch --list -d t should fail' ' > test_path_is_missing .git/refs/heads/t > ' > > +test_expect_failure 'git branch -d can delete ref with broken sha1' ' > + echo "pointing nowhere" > .git/refs/heads/brokensha1 && Please no space between the ">" and the filename. > + test_when_finished "rm -f .git/refs/heads/brokensha1" && > + git branch -d brokensha1 && > + git branch >output && > + ! grep -e "brokensha1" output > +' > + > test_expect_success 'git branch --column' ' > COLUMNS=81 git branch --column=column >actual && > cat >expected <<\EOF && > -- Michael Haggerty mhagger@xxxxxxxxxxxx http://softwareswirl.blogspot.com/ -- 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