From: Tian Yuchen <cat@malon.dev>
Date: Mon, 06 Apr 2026 17:36:02 GMT
Subject: Re: [PATCH v4 1/1] t2000: modernize overall structure and path checks
Message-ID: <9639df15-8498-4dd0-8ea1-6ebaf4533476@malon.dev>
In-Reply-To: <CAOLa=ZTuk-33xz4RQJDv-nyK-MqFzLWHM7zdmBBGnXjYiJBsSQ@mail.gmail.com>

On 4/6/26 06:04, Karthik Nayak wrote:
> 
> Shouldn't all the tests above (since 'mkdir path2') be a single test
> block? First we setup the data, validate the data, the previous test
> runs 'git-checkout-index' and finally here we're verifying the endstate
> here. I think this should all fit into one test block.

That makes sense. I didn’t mention this earlier because I hadn’t noticed 
that the patch title had changed from 't2000: modernise path checks with 
test_path_is_* helpers' to 't2000: modernise overall structure'. If this 
is a overall refactor, then it would be worth revising this outdated 
test structure as well ;)

> 
> Thanks,
> Karthik
> 
>>   test_done
>> --
>> 2.43.0

Regards, Yuchen