test_must_be_empty: make sure the file exists, not just empty
authorJunio C Hamano <gitster@pobox.com>
Tue, 27 Feb 2018 21:27:29 +0000 (13:27 -0800)
committerJunio C Hamano <gitster@pobox.com>
Tue, 27 Feb 2018 21:58:43 +0000 (13:58 -0800)
commit11395a3b4b16d9fc637ca2e41a6892ea2e6289ce
tree81e3d1e93b0025f8b4eafc5dde30f405426140be
parent38e79b1fdab9244e1727d0698afcf3bb8956c0a4
test_must_be_empty: make sure the file exists, not just empty

The helper function test_must_be_empty is meant to make sure the
given file is empty, but its implementation is:

if test -s "$1"
then
... not empty, we detected a failure ...
fi

Surely, the file having non-zero size is a sign that the condition
"the file must be empty" is violated, but it misses the case where
the file does not even exist.  It is an accident waiting to happen
with a buggy test like this:

git frotz 2>error-message &&
test_must_be_empty errro-message

that won't get caught until you deliberately break 'git frotz' and
notice why the test does not fail.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/test-lib-functions.sh