threads / discuss / 23331

Tests with broken && chains in pu

Subject: Tests with broken && chains in pu

## tl;dr

5 messages between Apr 4, 2010 and Apr 4, 2010.

replies: 4people: 4as markdown or json

Johannes Sixt· Apr 4, 2010, 20:08 UTC · lore
Junio,
two commits in pu currently add tests with broken && chains:
abbfd017 in cw/ws-indent-with-tab:
diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh
index 9e57f80..e92eab0 100755
--- a/t/t4015-diff-whitespace.sh
+++ b/t/t4015-diff-whitespace.sh
@@ -354,7 +354,7 @@ test_expect_success 'check tabs and spaces as indentation (indent-with-non-tab:
 
 test_expect_success 'check tabs as indentation (tab-in-indent: off)' '
 
-	git config core.whitespace "-tab-in-indent"
+	git config core.whitespace "-tab-in-indent" &&
 	echo "	foo ();" > x &&
 	git diff --check
 
@@ -378,7 +378,7 @@ test_expect_success 'check tabs and spaces as indentation (tab-in-indent: on)' '
 
 test_expect_success 'check tab-in-indent and indent-with-non-tab conflict' '
 
-	git config core.whitespace "tab-in-indent,indent-with-non-tab"
+	git config core.whitespace "tab-in-indent,indent-with-non-tab" &&
 	echo "foo ();" > x &&
 	test_must_fail git diff --check
 
@@ -386,7 +386,7 @@ test_expect_success 'check tab-in-indent and indent-with-non-tab conflict' '
 
 test_expect_success 'check tab-in-indent excluded from wildcard whitespace attribute' '
 
-	git config --unset core.whitespace
+	git config --unset core.whitespace &&
 	echo "x whitespace" > .gitattributes &&
 	echo "	  foo ();" > x &&
 	git diff --check &&



c8eeb6301 in sr/remote-helper-export

diff --git a/t/t5800-remote-helpers.sh b/t/t5800-remote-helpers.sh
index 33f546b..eb31709 100755
--- a/t/t5800-remote-helpers.sh
+++ b/t/t5800-remote-helpers.sh
@@ -9,7 +9,7 @@ test_description='Test remote-helper import and export commands'
 
 test_expect_success 'setup repository' '
 	git init --bare server/.git &&
-	git clone server public
+	git clone server public &&
 	(cd public &&
 	 echo content >file &&
 	 git add file &&
@@ -18,7 +18,7 @@ test_expect_success 'setup repository' '
 '
 
 test_expect_success 'cloning from local repo' '
-	git clone "testgit::${PWD}/server" localclone
+	git clone "testgit::${PWD}/server" localclone &&
 	test_cmp public/file localclone/file
 '
 
Sverre Rabbelier· Apr 4, 2010, 20:15 UTC · re: Johannes Sixt · lore

Re: Tests with broken && chains in pu

Heya,
On Sun, Apr 4, 2010 at 15:08, Johannes Sixt <j6t@kdbg.org> wrote:
> two commits in pu currently add tests with broken && chains:
Show 21 quoted lines
> diff --git a/t/t5800-remote-helpers.sh b/t/t5800-remote-helpers.sh
> index 33f546b..eb31709 100755
> --- a/t/t5800-remote-helpers.sh
> +++ b/t/t5800-remote-helpers.sh
> @@ -9,7 +9,7 @@ test_description='Test remote-helper import and export commands'
>
>  test_expect_success 'setup repository' '
>        git init --bare server/.git &&
> -       git clone server public
> +       git clone server public &&
>        (cd public &&
>         echo content >file &&
>         git add file &&
> @@ -18,7 +18,7 @@ test_expect_success 'setup repository' '
>  '
>
>  test_expect_success 'cloning from local repo' '
> -       git clone "testgit::${PWD}/server" localclone
> +       git clone "testgit::${PWD}/server" localclone &&
>        test_cmp public/file localclone/file
>  '

Thanks for noticing, may I ask how you noticed? Do you have a script or was it manual review?

-- 
Cheers,

Sverre Rabbelier
Johannes Sixt· Apr 4, 2010, 20:41 UTC · re: Sverre Rabbelier · lore

Re: Tests with broken && chains in pu

On Sonntag, 4. April 2010, Sverre Rabbelier wrote:
> Thanks for noticing, may I ask how you noticed? Do you have a script
> or was it manual review?

It was manual review. I reviewed the patches in pu because I noticed a broken && chain in a recently submitted patch, but I didn't remember which one it was. So I took the time to look at all new test cases in pu after Junio's recent What's in git message.

-- Hannes
Chris Webb· Apr 4, 2010, 20:45 UTC · re: Johannes Sixt · lore

Re: Tests with broken && chains in pu

Johannes Sixt <j6t@kdbg.org> writes:
> two commits in pu currently add tests with broken && chains:
> abbfd017 in cw/ws-indent-with-tab:

Thanks for spotting this mistake Johannes. Should I resubmit a corrected version of the patch, Junio, or do you prefer to squash fixups directly into the pu branch?

Best wishes,
Chris.

← back to recent threads