git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] test_must_be_empty: simplify file existence check

From
SZEDER Gábor <szeder.dev@gmail.com>
Date
Mar 26, 2018, 11:58 UTC
Message-ID
<20180326115815.10180-1-szeder.dev@gmail.com>

Commit ea3c87d0b7 (test_must_be_empty: make sure the file exists, not just empty, 2018-02-27) basically duplicated the 'test_path_is_file' helper function in 'test_must_be_empty'.

Just call 'test_path_is_file' to avoid this code duplication.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
---
 t/test-lib-functions.sh | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh
index d2eaf5ab67..36ad8accdd 100644
--- a/t/test-lib-functions.sh
+++ b/t/test-lib-functions.sh
@@ -718,11 +718,8 @@ verbose () {
 # otherwise.
 
 test_must_be_empty () {
-	if ! test -f "$1"
-	then
-		echo "'$1' is missing"
-		return 1
-	elif test -s "$1"
+	test_path_is_file "$1" &&
+	if test -s "$1"
 	then
 		echo "'$1' is not empty, it contains:"
 		cat "$1"
-- 
2.17.0.rc1.145.gfac1bf9500
Next: Jeff King
Message 1 of 6 in “test_must_be_empty: simplify file existence check”
  1. test_must_be_empty: simplify file existence checkSZEDER Gábor, Mar 26, 2018
  2. Jeff KingMar 26, 2018
  3. SZEDER GáborMar 26, 2018
  4. test_must_be_empty: simplify file existence checkSZEDER Gábor, Mar 26, 2018
  5. Junio C HamanoMar 28, 2018
  6. Junio C HamanoMar 28, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.