Test 48 from t6030-bisect-porcelain.sh claims it tests whether bisection fails if tree is broken on start commit. However that actually does not happen because the bisection really only fails because the first trial point we choose happens to be broken. Furthermore there is another equivalent trial point which is not broken so the test is not reliable. Remove it as test 49 tests the same behavior is a more reliable setting. Signed-off-by: Jan Kara <jack@xxxxxxx> --- t/t6030-bisect-porcelain.sh | 6 ------ 1 file changed, 6 deletions(-) diff --git a/t/t6030-bisect-porcelain.sh b/t/t6030-bisect-porcelain.sh index 0f2a91996393..4ec7b5b5a72e 100755 --- a/t/t6030-bisect-porcelain.sh +++ b/t/t6030-bisect-porcelain.sh @@ -675,12 +675,6 @@ cat > expected.missing-tree.default <<EOF fatal: unable to read tree $deleted EOF -test_expect_success 'bisect fails if tree is broken on start commit' ' - git bisect reset && - test_must_fail git bisect start BROKEN_HASH7 BROKEN_HASH4 2>error.txt && - test_cmp expected.missing-tree.default error.txt -' - test_expect_success 'bisect fails if tree is broken on trial commit' ' git bisect reset && test_must_fail git bisect start BROKEN_HASH9 BROKEN_HASH4 2>error.txt && -- 2.26.2
