On Tue, Jan 02, 2018 at 05:49:45PM -0500, Eric Sunshine wrote:
> > diff --git a/t/t5600-clone-fail-cleanup.sh b/t/t5600-clone-fail-cleanup.sh
> > @@ -42,13 +53,48 @@ test_expect_success 'successful clone must leave the
> > directory' '
> > +test_expect_success 'failed clone into empty leaves directory (separate,
> > git)' '
> > + mkdir -p empty-git &&
> > + corrupt_repo &&
> > + test_must_fail git clone --separate-git-dir empty-git foo no-wt &&
> > + test_dir_is_empty empty-git &&
> > + test_path_is_missing no-wt
> > +'
> > +
> > +test_expect_success 'failed clone into empty leaves directory (separate,
> > git)' '
> > + mkdir -p empty-wt &&
> > + corrupt_repo &&
> > + test_must_fail git clone --separate-git-dir no-git foo empty-wt &&
> > + test_path_is_missing no-git &&
> > + test_dir_is_empty empty-wt
> > +'
>
> The final two tests seem to have the same title...
Oops. The second one should be "wt" (since the idea was to flip the
logic from the previous). Like so:
diff --git a/t/t5600-clone-fail-cleanup.sh b/t/t5600-clone-fail-cleanup.sh
index 5cd94d5558..4a1a912e03 100755
--- a/t/t5600-clone-fail-cleanup.sh
+++ b/t/t5600-clone-fail-cleanup.sh
@@ -89,7 +89,7 @@ test_expect_success 'failed clone into empty leaves directory
(separate, git)' '
test_path_is_missing no-wt
'
-test_expect_success 'failed clone into empty leaves directory (separate, git)'
'
+test_expect_success 'failed clone into empty leaves directory (separate, wt)' '
mkdir -p empty-wt &&
corrupt_repo &&
test_must_fail git clone --separate-git-dir no-git foo empty-wt &&
