threads / patch / 41834

v3t7012: Implement test for git-checkout

Subject: [PATCH v3] t7012: Implement test for git-checkout

## tl;dr

3 messages between Mar 26, 2016 and Mar 30, 2016. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Chhatoi Pritam Baral· Mar 26, 2016, 16:37 UTC · lore

Previously a TODO; add a test for git-checkout skipping a file with the skip-worktree bit set.

Signed-off-by: Chhatoi Pritam Baral <chhatoipritam@gmail.com>
---

Replaced test_must_fail around grep with '!', as suggested by Eric.

 t/t7012-skip-worktree-writing.sh | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

Show changes to t/t7012-skip-worktree-writing.sh +11 −1

diff --git a/t/t7012-skip-worktree-writing.sh
b/t/t7012-skip-worktree-writing.sh
index 9ceaa40..276f076 100755
--- a/t/t7012-skip-worktree-writing.sh
+++ b/t/t7012-skip-worktree-writing.sh
@@ -141,6 +141,16 @@ test_expect_success 'git-clean, dirty case' '
 #TODO test_expect_failure 'git-apply removes file' false
 #TODO test_expect_failure 'git-mv to skip-worktree' false
 #TODO test_expect_failure 'git-mv from skip-worktree' false
-#TODO test_expect_failure 'git-checkout' false
+
+test_expect_success 'git-checkout ignores skip-worktree file' '
+	echo >1 &&
+	git commit -m "Add files" &&
+	echo dirty >1 &&
+	echo dirty >2 &&
+	git update-index --skip-worktree 1 &&
+	git checkout -- . &&
+	grep -q dirty 1 &&
+	! grep -q dirty 2
+'
  test_done

-- 
2.7.4

Chhatoi Pritam Baral· Mar 30, 2016, 13:29 UTC · re: Chhatoi Pritam Baral · lore

Re: [PATCH v3] t7012: Implement test for git-checkout

Forgot to mention in the previous message, this is a microproject for my GSoC '16 application.

(Is that redundant to mention after v1 and v2? )

On 03/26/2016 10:07 PM, Chhatoi Pritam Baral wrote:

Show 33 quoted lines

> Previously a TODO; add a test for git-checkout skipping a
> file with the skip-worktree bit set.
>
> Signed-off-by: Chhatoi Pritam Baral <chhatoipritam@gmail.com>
> ---
>
> Replaced test_must_fail around grep with '!', as suggested by Eric.
>
>  t/t7012-skip-worktree-writing.sh | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/t/t7012-skip-worktree-writing.sh
> b/t/t7012-skip-worktree-writing.sh
> index 9ceaa40..276f076 100755
> --- a/t/t7012-skip-worktree-writing.sh
> +++ b/t/t7012-skip-worktree-writing.sh
> @@ -141,6 +141,16 @@ test_expect_success 'git-clean, dirty case' '
>  #TODO test_expect_failure 'git-apply removes file' false
>  #TODO test_expect_failure 'git-mv to skip-worktree' false
>  #TODO test_expect_failure 'git-mv from skip-worktree' false
> -#TODO test_expect_failure 'git-checkout' false
> +
> +test_expect_success 'git-checkout ignores skip-worktree file' '
> +	echo >1 &&
> +	git commit -m "Add files" &&
> +	echo dirty >1 &&
> +	echo dirty >2 &&
> +	git update-index --skip-worktree 1 &&
> +	git checkout -- . &&
> +	grep -q dirty 1 &&
> +	! grep -q dirty 2
> +'
>   test_done

Eric Sunshine· Mar 30, 2016, 15:21 UTC · re: Chhatoi Pritam Baral · lore

Re: [PATCH v3] t7012: Implement test for git-checkout

On Wed, Mar 30, 2016 at 9:29 AM, Chhatoi Pritam Baral <chhatoipritam@gmail.com> wrote:

> Forgot to mention in the previous message, this is a microproject
> for my GSoC '16 application.
>
> (Is that redundant to mention after v1 and v2? )

Readers of v1 knew this since you mentioned with that version, at least.