Re: [PATCH v4 6/8] worktree: add relative cli/config options to `move` command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri Nov 22, 2024 at 9:55 AM CST, Phillip Wood wrote:
> On 01/11/2024 04:38, Caleb White wrote:
>> [...]
>> +test_expect_success 'move worktree with absolute path to relative path' '
>> +	git config worktree.useRelativePaths false &&
>> +	git worktree add ./absolute &&
>> +	git worktree move --relative-paths absolute relative &&
>> +	cat relative/.git >actual &&
>> +	echo "gitdir: ../.git/worktrees/absolute" >expect &&
>> +	test_cmp expect actual &&
>> +	git config worktree.useRelativePaths true &&
>> +	git worktree move relative relative2 &&
>> +	cat relative2/.git >actual &&
>> +	echo "gitdir: ../.git/worktrees/absolute" >expect &&
>> +	test_cmp expect actual
>> +'
>
> As with the last patch we should use test_config and stop copying files
> just to compare them. It's probably worth checking the gitdir file as
> well as .git here as well.

Updated, thanks for pointing that out.

Best,

Caleb






[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux