Skip to content

Modernize some testcases for merge-recursive corner cases

Elijah Newren git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5t6036, t6042: use test_create_repo to keep tests independentt/t6036-recursive-corner-cases.sh t/t6042-merge-rename-corner-cases.sh+926 -8150 replies
2/5t6036, t6042: use test_line_count instead of wc -lt/t6036-recursive-corner-cases.sh t/t6042-merge-rename-corner-cases.sh+134 -675 replies
3/5t6036, t6042: prefer test_path_is_file, test_path_is_missingt/t6036-recursive-corner-cases.sh t/t6042-merge-rename-corner-cases.sh+22 -220 replies
4/5t6036, t6042: prefer test_cmp to sequences of testt/t6036-recursive-corner-cases.sh t/t6042-merge-rename-corner-cases.sh+130 -730 replies
5/5t6036: prefer test_when_finished to manual cleanup in following testt/t6036-recursive-corner-cases.sh+12 -100 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] Modernize some testcases for merge-recursive corner cases

I have about a dozen more testcases I want to add to the testsuite, in order to document additional situations that merge-recursive handles suboptimally. To avoid sending an excessively long series, I am splitting this up. This series just aims to modernize a pair of testfiles in which several of the new tests would most naturally belong. This modernization serves two purposes: (1) it makes the existing tests easier to understand, and (2) it simplifies adding more tests to these test files.

Elijah Newren (5):
t6036, t6042: use test_create_repo to keep tests independent t6036, t6042: use test_line_count instead of wc -l
t6036, t6042: prefer test_path_is_file, test_path_is_missing t6036, t6042: prefer test_cmp to sequences of test
t6036: prefer test_when_finished to manual cleanup in following test

 t/t6036-recursive-corner-cases.sh    | 991 +++++++++++++++------------
 t/t6042-merge-rename-corner-cases.sh | 968 ++++++++++++++------------
 2 files changed, 1098 insertions(+), 861 deletions(-)
Signature · 1 line
2.17.0.1.gda85003413