t0000: drop "known breakage" test
authorJeff King <peff@peff.net>
Sat, 28 Dec 2013 09:33:40 +0000 (04:33 -0500)
committerJunio C Hamano <gitster@pobox.com>
Thu, 2 Jan 2014 22:43:12 +0000 (14:43 -0800)
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>
t/t0000-basic.sh
index e6c5b6383914c5010c36c3dc058e9216925777b1..a2bb63ce8e5e5586691affb51f9cff1f4d1e327a 100755 (executable)
@@ -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