]> git.notmuchmail.org Git - notmuch/commitdiff
test: Remove basic testing of broken, fixed, and skipped tests.
authorCarl Worth <cworth@cworth.org>
Fri, 17 Sep 2010 22:30:17 +0000 (15:30 -0700)
committerCarl Worth <cworth@cworth.org>
Fri, 17 Sep 2010 22:30:17 +0000 (15:30 -0700)
These were interfering with the aggregate statistics reported at the
end of the test-suite run. (Always reporting 1 broken, 1 fixed, and 1
skipped). The correct way to test the test-suite itself would be to
run the test suite externally for these cases, capture the expected
result, and then report that as a PASS test.

But, really, there's almost no value in these tests anyway. It's
almost to the level of testing that 'if false; exit 1; fi' returns
1. That is, there are so many ways that the test suite could be broken
internally, that these minor tests don't really help.

test/t0000-basic.sh

index 2cc4ef75758154c7d34418c7e7108b3a05e371fa..eb109533c9d6b9ec56416283c176982c7c92cb67 100755 (executable)
@@ -21,27 +21,12 @@ fi
 test_expect_success 'success is reported like this' '
     :
 '
-test_expect_failure 'pretend we have a known breakage' '
-    false
-'
-test_expect_failure 'pretend we have fixed a known breakage' '
-    :
-'
 test_set_prereq HAVEIT
 haveit=no
 test_expect_success HAVEIT 'test runs if prerequisite is satisfied' '
     test_have_prereq HAVEIT &&
     haveit=yes
 '
-donthaveit=yes
-test_expect_success DONTHAVEIT 'unmet prerequisite causes test to be skipped' '
-    donthaveit=no
-'
-if test $haveit$donthaveit != yesyes
-then
-       say "bug in test framework: prerequisite tags do not work reliably"
-       exit 1
-fi
 
 clean=no
 test_expect_success 'tests clean up after themselves' '