# [PATCH] t7012: Implement test for git-checkout

3 messages from 2016-03-25 to 2016-03-25. Participants: Chhatoi Pritam Baral, Torsten Bögershausen, Eric Sunshine.
Thread: https://gitlist.dev/t/41826

## Chhatoi Pritam Baral, 2016-03-25 17:36

Subject: [PATCH] t7012: Implement test for git-checkout
Message-ID: <56F57714.4050708@gmail.com>
URL: https://gitlist.dev/e/56F57714.4050708%40gmail.com

```
Previously a TODO, this patch adds a test for git-checkout skipping a
file with the
skip-worktree bit set.

I'm doing this as a microproject for my GSoC '16 application.

Signed-off-by: Chhatoi Pritam Baral <chhatoipritam@gmail.com>
---
 t/t7012-skip-worktree-writing.sh | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/t/t7012-skip-worktree-writing.sh
b/t/t7012-skip-worktree-writing.sh
index 9ceaa40..d5b42d6 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 &&
+	test_must_fail grep -q dirty 2
+'
  test_done
-- 
2.7.4

```

## Torsten Bögershausen, 2016-03-25 22:32

Subject: Re: [PATCH] t7012: Implement test for git-checkout
Message-ID: <56F5BC94.1070903@web.de>
URL: https://gitlist.dev/e/56F5BC94.1070903%40web.de
In-Reply-To: <56F57714.4050708@gmail.com>

```
On 2016-03-25 18.36, Chhatoi Pritam Baral wrote:
Thanks for working on Git.
Some comments inline.
> Previously a TODO, this patch adds a test for git-checkout skipping a
> file with the
> skip-worktree bit set.
Micro-nit: the "this patch ..." may be written shorter:
> Previously a TODO, add a test for git-checkout skipping a
> file with the skip-worktree bit set.

> 
> I'm doing this as a microproject for my GSoC '16 application.
This info is good to have, but doesn't need to stay forever in the
commit message. It may be below (see XXX)
> 
> Signed-off-by: Chhatoi Pritam Baral <chhatoipritam@gmail.com>
> ---
XXX Add additional info here.
>  t/t7012-skip-worktree-writing.sh | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/t/t7012-skip-worktree-writing.sh
> b/t/t7012-skip-worktree-writing.sh
> index 9ceaa40..d5b42d6 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 &&
Style: No space after the '>', like this:
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 &&
> +	test_must_fail grep -q dirty 2
> +'
>   test_done
> 

```

## Eric Sunshine, 2016-03-25 23:29

Subject: Re: [PATCH] t7012: Implement test for git-checkout
Message-ID: <CAPig+cR16EOA-EC=6GXXAkoJRC7dP-79xJUjVhyg+62sukK6qg@mail.gmail.com>
URL: https://gitlist.dev/e/CAPig%2BcR16EOA-EC%3D6GXXAkoJRC7dP-79xJUjVhyg%2B62sukK6qg%40mail.gmail.com
In-Reply-To: <56F57714.4050708@gmail.com>

```
On Fri, Mar 25, 2016 at 1:36 PM, Chhatoi Pritam Baral
<chhatoipritam@gmail.com> wrote:
> Previously a TODO, this patch adds a test for git-checkout skipping a
> file with the skip-worktree bit set.

In addition to Torsten's comments...

> diff --git 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 &&
> +       test_must_fail grep -q dirty 2

test_must_fail is for git commands; for system commands, just use '!'
(t/README explains the rationale).

    ! grep -q dirty 2

> +'
>   test_done

```