Skip to content

Commit

Permalink
t0000: drop "known breakage" test
Browse files Browse the repository at this point in the history
Having a simulated "known breakage" test means that the test
suite will always tell us there is a bug to be fixed, even
though it is only simulated.

The right way to test this is in a sub-test, that can also
check that we provide the correct exit status and output.
Fortunately, we already have such a test (added much later
by 5ebf89e).

We could arguably get rid of the simulated success test
immediately above, as well, as it is also redundant with the
tests added in 5ebf89e. However, it does not have the
annoying behavior of the "known breakage" test. It may also
be easier to debug if the test suite is truly broken, since
it is not a test-within-a-test, as the later tests are.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Jeff King authored and Junio C Hamano committed Jan 2, 2014
1 parent a63c12c commit 738a8be
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions t/t0000-basic.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@ test_expect_success '.git/objects should have 3 subdirectories' '
test_expect_success 'success is reported like this' '
:
'
test_expect_failure 'pretend we have a known breakage' '
false
'

run_sub_test_lib_test () {
name="$1" descr="$2" # stdin is the body of the test code
Expand Down

0 comments on commit 738a8be

Please sign in to comment.