Re: [PATCH] t0200: use path checking helpers

[Junio C Hamano <gitster@pobox.com>]

Usman Akinyemi <usmanakinyemi202@gmail.com> writes:

>>
>> Replace direct directory and file checks with the corresponding
>> test helpers in t/t0200-gettext-basic.sh.
> This is a little bit more confusing. The commit message does not
> really show why you did what you did. Along the line of
>
> "Modernize the code testing whether a file exists in "filename"' seems
> better. Also, your patches is doing two things, i.e modernizing the 'test -f'
> and 'test -d'. Ideally a patch should only focus on one thing but,
> since this is
> simple patch. It is okay.
>>
>> Instances were located using:
>> git grep -n -E 'test -[efds]' -- 't/t[0-9]*.sh'
> This part is nto necessary.

Since many candidates are working on similar microprojects (which is
fine) and making the same mistakes (which is expected), perhaps mentors
should tell them to read the review exchanges on other candidates'
patches and learn from them.

>>
>> Signed-off-by: Victor Okafor <vua.okafor@gmail.com>
>> ---
>>  t/t0200-gettext-basic.sh | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/t/t0200-gettext-basic.sh b/t/t0200-gettext-basic.sh
>>  test_expect_success GETTEXT 'sanity: Icelandic locale was compiled' '
>> -    test -f "$TEXTDOMAINDIR/is/LC_MESSAGES/git.mo"
>> +    test_path_is_file "$TEXTDOMAINDIR/is/LC_MESSAGES/git.mo"
>>  '
>>
>>  # TODO: When we have more locales, generalize this to test them
>> --
>> 2.55.0
>>
> Thanks
>>