Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > Change a couple of uses of "test_expect_failure" to use a > "test_expect_success" to positively assert the current behavior, and > replace the intent of "test_expect_failure" with a "TODO" comment int > the description. > > As noted in [1] the "test_expect_failure" feature is overly eager to And noted in [2], it is not a good idea to abuse "test_expect_success" for this purpose, either, though. [2] https://lore.kernel.org/git/xmqq4k9kj15p.fsf@gitster.g/