Re: [GSoC] microporject test_path_is_*
- Date: Wed, 27 Mar 2019 12:28:49 +0100
- From: Mooga <contact@xxxxxxxxxxx>
- Subject: Re: [GSoC] microporject test_path_is_*
So for example that is git diff
diff --git a/t/t2400-worktree-add.sh b/t/t2400-worktree-add.sh
index 286bba35d8..fc82965a0f 100755
--- a/t/t2400-worktree-add.sh
+++ b/t/t2400-worktree-add.sh
@@ -66,7 +66,7 @@ test_expect_success '"add" worktree' '
test_expect_success '"add" worktree with lock' '
git rev-parse HEAD >expect &&
git worktree add --detach --lock here-with-lock master &&
- test -f .git/worktrees/here-with-lock/locked
+ test_path_is_file .git/worktrees/here-with-lock/locked
On 26.03.19, 23:07, "Elijah Newren" <git-owner@xxxxxxxxxxxxxxx on behalf of newren@xxxxxxxxx> wrote:
Hi,
On Tue, Mar 26, 2019 at 2:10 PM Mooga <contact@xxxxxxxxxxx> wrote:
>
> Hi,
> I am still a bit confused about the task itself
>
> it’s just text replacing for example:
> t1400-update-ref.sh , line 194 -> `test_path_is_missing` has to be ‘test_path_is_file’
>
> Thanks
There are several places in the code that use test with -e or -f or -d
(or -h or...) in order to check for the presence of a
file/directory/symlink/etc. For example,
test -f path1/file1
This could be made more clear and produce nicer error messages if it
were instead
test_path_is_file path1/file1
There are likewise several that use one of
! test -e path/to/filename
or
! test -f path/to/filename
or
test ! -f path/to/filename
which could be replaced by
test_path_is_missing path/to/filename
This GSoC microproject is just about picking one testfile that has
some of these constructs, and fixing the cases found within that
testfile.