Re: [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell
From: SZEDER Gábor <hidden>
Date: 2018-02-28 00:44:16
Possibly related (same subject, not in this thread)
- 2018-02-27 · Re: [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell · SZEDER Gábor <hidden>
- 2018-02-27 · Re: [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell · Junio C Hamano <hidden>
- 2018-02-27 · Re: [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell · Junio C Hamano <hidden>
- 2018-02-24 · Re: [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell · Eric Sunshine <hidden>
- 2018-02-23 · [PATCH 04/11] t3030-merge-recursive: don't check the stderr of a subshell · SZEDER Gábor <hidden>
On Tue, Feb 27, 2018 at 10:17 PM, Junio C Hamano [off-list ref] wrote:
SZEDER Gábor [off-list ref] writes:quoted
+ git read-tree -i -m $c3 2>actual-err && + test_must_be_empty expected-err && + git update-index --ignore-missing --refresh 2>>actual-err && + test_must_be_empty expected-err && + git merge-recursive $c0 -- $c3 $c7 2>>actual-err && + test_must_be_empty expected-err && + git ls-files -s >actual-files 2>>actual-err && + test_must_be_empty expected-errAlso, with the error output of individual steps tested like this (assuming that test-must-be-empty checks are to be done on the actual-err file, not ecpected-err that nobody creates), I do not see a point in appending to the file. So perhaps squash this in?
Agreed again.
quoted hunk ↗ jump to hunk
t/t3030-merge-recursive.sh | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-)diff --git a/t/t3030-merge-recursive.sh b/t/t3030-merge-recursive.sh index cbeea1cf94..3563e77b37 100755 --- a/t/t3030-merge-recursive.sh +++ b/t/t3030-merge-recursive.sh@@ -526,13 +526,13 @@ test_expect_success 'merge-recursive w/ empty work tree - ours has rename' ' export GIT_INDEX_FILE && mkdir "$GIT_WORK_TREE" && git read-tree -i -m $c7 2>actual-err && - test_must_be_empty expected-err && + test_must_be_empty actual-err && git update-index --ignore-missing --refresh 2>actual-err && - test_must_be_empty expected-err && + test_must_be_empty actual-err && git merge-recursive $c0 -- $c7 $c3 2>actual-err && - test_must_be_empty expected-err && + test_must_be_empty actual-err && git ls-files -s >actual-files 2>actual-err && - test_must_be_empty expected-err + test_must_be_empty actual-err ) && cat >expected-files <<-EOF && 100644 $o3 0 b/c@@ -551,13 +551,13 @@ test_expect_success 'merge-recursive w/ empty work tree - theirs has rename' ' export GIT_INDEX_FILE && mkdir "$GIT_WORK_TREE" && git read-tree -i -m $c3 2>actual-err && - test_must_be_empty expected-err && - git update-index --ignore-missing --refresh 2>>actual-err && - test_must_be_empty expected-err && - git merge-recursive $c0 -- $c3 $c7 2>>actual-err && - test_must_be_empty expected-err && - git ls-files -s >actual-files 2>>actual-err && - test_must_be_empty expected-err + test_must_be_empty actual-err && + git update-index --ignore-missing --refresh 2>actual-err && + test_must_be_empty actual-err && + git merge-recursive $c0 -- $c3 $c7 2>actual-err && + test_must_be_empty actual-err && + git ls-files -s >actual-files 2>actual-err && + test_must_be_empty actual-err ) && cat >expected-files <<-EOF && 100644 $o3 0 b/c