Re: [PATCH v4 3/3] t4140: test apply with i-t-a paths
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 6, 2020, 21:07 UTC
- Message-ID
- <xmqqeeojjxf0.fsf@gitster.c.googlers.com>
- In-Reply-To
- <20200806060119.74587-4-ray@ameretat.dev>
"Raymond E. Pasco" <ray@ameretat.dev> writes:
Show 11 quoted lines
> apply --cached (as used by add -p) should accept creation and deletion > patches to intent-to-add paths in the index. apply --index, however, > should always fail because an intent-to-add path never matches the > worktree (by definition). > > Based-on-patch-by: Junio C Hamano <gitster@pobox.com> > Signed-off-by: Raymond E. Pasco <ray@ameretat.dev> > --- > t/t4140-apply-ita.sh | 56 ++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 56 insertions(+) > create mode 100644 t/t4140-apply-ita.sh
chmod +x
Show 34 quoted lines
> diff --git a/t/t4140-apply-ita.sh b/t/t4140-apply-ita.sh > new file mode 100644 > index 0000000000..c614eaf04c > --- /dev/null > +++ b/t/t4140-apply-ita.sh > @@ -0,0 +1,56 @@ > +#!/bin/sh > + > +test_description='git apply of i-t-a file' > + > +. ./test-lib.sh > + > +test_expect_success setup ' > + test_write_lines 1 2 3 4 5 >blueprint && > + > + cat blueprint >test-file && > + git add -N test-file && > + git diff >creation-patch && > + grep "new file mode 100644" creation-patch && > + > + rm -f test-file && > + git diff >deletion-patch && > + grep "deleted file mode 100644" deletion-patch > +' > + > +test_expect_success 'apply creation patch to ita path (--cached)' ' > + git rm -f test-file && > + cat blueprint >test-file && > + git add -N test-file && > + > + git apply --cached creation-patch && > + git cat-file blob :test-file >actual && > + test_cmp blueprint actual > +'
OK, this is a good positive test case.
Show 8 quoted lines
> +test_expect_success 'apply creation patch to ita path (--index)' ' > + git rm -f test-file && > + cat blueprint >test-file && > + git add -N test-file && > + rm -f test-file && > + > + test_must_fail git apply --index creation-patch > +'
"--index" (not "--cached") notices that test-file does not match between the index and the working tree, and fails. OK.
Show 8 quoted lines
> +test_expect_success 'apply deletion patch to ita path (--cached)' ' > + git rm -f test-file && > + cat blueprint >test-file && > + git add -N test-file && > + > + git apply --cached deletion-patch && > + test_must_fail git ls-files --stage --error-unmatch test-file > +'
We can delete an I-T-A entry. I wonder if
git apply --cached -R creation-patch
also serves as a way to remove the path? It should succeed, right?
Show 7 quoted lines
> +test_expect_success 'apply deletion patch to ita path (--index)' ' > + cat blueprint >test-file && > + git add -N test-file && > + > + test_must_fail git apply --index deletion-patch && > + git ls-files --stage --error-unmatch test-file > +'
Again "--index" notices that the path has ITA bit on, and refuses to remove it.
OK, looking good so far.
Thanks.