mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-21 18:38:08 +01:00
test: Remove basic testing of broken, fixed, and skipped tests.
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.
This commit is contained in:
parent
82155c3d96
commit
51cd69feb1
1 changed files with 0 additions and 15 deletions
|
@ -21,27 +21,12 @@ fi
|
||||||
test_expect_success 'success is reported like this' '
|
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
|
test_set_prereq HAVEIT
|
||||||
haveit=no
|
haveit=no
|
||||||
test_expect_success HAVEIT 'test runs if prerequisite is satisfied' '
|
test_expect_success HAVEIT 'test runs if prerequisite is satisfied' '
|
||||||
test_have_prereq HAVEIT &&
|
test_have_prereq HAVEIT &&
|
||||||
haveit=yes
|
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
|
clean=no
|
||||||
test_expect_success 'tests clean up after themselves' '
|
test_expect_success 'tests clean up after themselves' '
|
||||||
|
|
Loading…
Reference in a new issue