[PATCH 07/16] t5304: use test_path_is_* instead of test -f

2014-10-03 Thread Jeff King
This is slightly more robust (checking ! test -f would not
notice a directory of the same name, though that is not
likely to happen here). It also makes debugging easier, as
the test script will output a message on failure.

Signed-off-by: Jeff King p...@peff.net
---
This patch is totally optional. I did it while debugging t5304 (strange
how badly prune works when you accidentally invert the mtime check!)
and figured it might be worth keeping as a cleanup.

 t/t5304-prune.sh | 46 +++---
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/t/t5304-prune.sh b/t/t5304-prune.sh
index 01c6a3f..b0ffb05 100755
--- a/t/t5304-prune.sh
+++ b/t/t5304-prune.sh
@@ -14,7 +14,7 @@ add_blob() {
BLOB=$(echo aleph_0 | git hash-object -w --stdin) 
BLOB_FILE=.git/objects/$(echo $BLOB | sed s/^../\//) 
test $((1 + $before)) = $(git count-objects | sed s/ .*//) 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
test-chmtime =+0 $BLOB_FILE
 }
 
@@ -35,9 +35,9 @@ test_expect_success 'prune stale packs' '
:  .git/objects/tmp_2.pack 
test-chmtime =-86501 .git/objects/tmp_1.pack 
git prune --expire 1.day 
-   test -f $orig_pack 
-   test -f .git/objects/tmp_2.pack 
-   ! test -f .git/objects/tmp_1.pack
+   test_path_is_file $orig_pack 
+   test_path_is_file .git/objects/tmp_2.pack 
+   test_path_is_missing .git/objects/tmp_1.pack
 
 '
 
@@ -46,11 +46,11 @@ test_expect_success 'prune --expire' '
add_blob 
git prune --expire=1.hour.ago 
test $((1 + $before)) = $(git count-objects | sed s/ .*//) 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
test-chmtime =-86500 $BLOB_FILE 
git prune --expire 1.day 
test $before = $(git count-objects | sed s/ .*//) 
-   ! test -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -60,11 +60,11 @@ test_expect_success 'gc: implicit prune --expire' '
test-chmtime =-$((2*$week-30)) $BLOB_FILE 
git gc 
test $((1 + $before)) = $(git count-objects | sed s/ .*//) 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
test-chmtime =-$((2*$week+1)) $BLOB_FILE 
git gc 
test $before = $(git count-objects | sed s/ .*//) 
-   ! test -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -110,7 +110,7 @@ test_expect_success 'prune: do not prune detached HEAD with 
no reflog' '
git commit --allow-empty -m detached commit 
# verify that there is no reflogs
# (should be removed and disabled by previous test)
-   test ! -e .git/logs 
+   test_path_is_missing .git/logs 
git prune -n prune_actual 
: prune_expected 
test_cmp prune_actual prune_expected
@@ -145,7 +145,7 @@ test_expect_success 'gc --no-prune' '
git config gc.pruneExpire 2.days.ago 
git gc --no-prune 
test 1 = $(git count-objects | sed s/ .*//) 
-   test -f $BLOB_FILE
+   test_path_is_file $BLOB_FILE
 
 '
 
@@ -153,10 +153,10 @@ test_expect_success 'gc respects gc.pruneExpire' '
 
git config gc.pruneExpire 5002.days.ago 
git gc 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git config gc.pruneExpire 5000.days.ago 
git gc 
-   test ! -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -165,9 +165,9 @@ test_expect_success 'gc --prune=date' '
add_blob 
test-chmtime =-$((5001*$day)) $BLOB_FILE 
git gc --prune=5002.days.ago 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git gc --prune=5000.days.ago 
-   test ! -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -175,9 +175,9 @@ test_expect_success 'gc --prune=never' '
 
add_blob 
git gc --prune=never 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git gc --prune=now 
-   test ! -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -186,10 +186,10 @@ test_expect_success 'gc respects gc.pruneExpire=never' '
git config gc.pruneExpire never 
add_blob 
git gc 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git config gc.pruneExpire now 
git gc 
-   test ! -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -197,9 +197,9 @@ test_expect_success 'prune --expire=never' '
 
add_blob 
git prune --expire=never 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git prune 
-   test ! -f $BLOB_FILE
+   test_path_is_missing $BLOB_FILE
 
 '
 
@@ -210,10 +210,10 @@ test_expect_success 'gc: prune old objects after local 
clone' '
(
cd aclone 
test 1 = $(git count-objects | sed s/ .*//) 
-   test -f $BLOB_FILE 
+   test_path_is_file $BLOB_FILE 
git gc --prune 
  

Re: [PATCH 07/16] t5304: use test_path_is_* instead of test -f

2014-10-03 Thread Junio C Hamano
Jeff King p...@peff.net writes:

 This is slightly more robust (checking ! test -f would not
 notice a directory of the same name, though that is not
 likely to happen here). It also makes debugging easier, as
 the test script will output a message on failure.

 Signed-off-by: Jeff King p...@peff.net
 ---
 This patch is totally optional. I did it while debugging t5304 (strange
 how badly prune works when you accidentally invert the mtime check!)
 and figured it might be worth keeping as a cleanup.

Looks sensible; thanks.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html