tests: use test_write_lines() to generate line-oriented output
Take advantage of test_write_lines() to generate line-oriented output rather than using for-loops or a series of `echo` commands. Not only is test_write_lines() a natural fit for such a task, but there is less opportunity for a broken &&-chain. Signed-off-by: Eric Sunshine <sunshine@sunshineco.com> Reviewed-by: Elijah Newren <newren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
020b813f40
commit
0849541268
@ -51,7 +51,7 @@ test_expect_success setup '
|
||||
'
|
||||
|
||||
test_expect_success 'rev-list D..M' '
|
||||
for c in E F G H I J K L M; do echo $c; done >expect &&
|
||||
test_write_lines E F G H I J K L M >expect &&
|
||||
git rev-list --format=%s D..M |
|
||||
sed -e "/^commit /d" |
|
||||
sort >actual &&
|
||||
@ -59,7 +59,7 @@ test_expect_success 'rev-list D..M' '
|
||||
'
|
||||
|
||||
test_expect_success 'rev-list --ancestry-path D..M' '
|
||||
for c in E F H I J L M; do echo $c; done >expect &&
|
||||
test_write_lines E F H I J L M >expect &&
|
||||
git rev-list --ancestry-path --format=%s D..M |
|
||||
sed -e "/^commit /d" |
|
||||
sort >actual &&
|
||||
@ -81,7 +81,7 @@ test_expect_success 'rev-list --ancestry-path D..M -- M.t' '
|
||||
'
|
||||
|
||||
test_expect_success 'rev-list F...I' '
|
||||
for c in F G H I; do echo $c; done >expect &&
|
||||
test_write_lines F G H I >expect &&
|
||||
git rev-list --format=%s F...I |
|
||||
sed -e "/^commit /d" |
|
||||
sort >actual &&
|
||||
@ -89,7 +89,7 @@ test_expect_success 'rev-list F...I' '
|
||||
'
|
||||
|
||||
test_expect_success 'rev-list --ancestry-path F...I' '
|
||||
for c in F H I; do echo $c; done >expect &&
|
||||
test_write_lines F H I >expect &&
|
||||
git rev-list --ancestry-path --format=%s F...I |
|
||||
sed -e "/^commit /d" |
|
||||
sort >actual &&
|
||||
@ -111,7 +111,7 @@ test_expect_success 'rev-list --ancestry-path G..M -- G.t' '
|
||||
'
|
||||
|
||||
test_expect_success 'rev-list --ancestry-path --simplify-merges G^..M -- G.t' '
|
||||
for c in G L; do echo $c; done >expect &&
|
||||
test_write_lines G L >expect &&
|
||||
git rev-list --ancestry-path --simplify-merges --format=%s G^..M -- G.t |
|
||||
sed -e "/^commit /d" |
|
||||
sort >actual &&
|
||||
|
Reference in New Issue
Block a user