From: Junio C Hamano Date: Sat, 11 Jun 2005 01:34:34 +0000 (-0700) Subject: [PATCH] Clean up read-tree two-way tests. X-Git-Tag: v0.99~284 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=76bc82ca0ed4a0af85b7d741c04c8191738e8395;p=git.git [PATCH] Clean up read-tree two-way tests. This is in preparation for "2-way fast-forward emulated with 3-way mechanism" series. It does not change what the tests for pure 2-way do. It only changes how it tests things, to make reviewing of differences of the two tests easier in later steps. Signed-off-by: Junio C Hamano Signed-off-by: Linus Torvalds --- diff --git a/t/t1001-read-tree-m-2way.sh b/t/t1001-read-tree-m-2way.sh index 33041f3cc..309baa5ae 100755 --- a/t/t1001-read-tree-m-2way.sh +++ b/t/t1001-read-tree-m-2way.sh @@ -22,12 +22,18 @@ In the test, these paths are used: ' . ./test-lib.sh +read_tree_twoway () { + git-read-tree -m "$1" "$2" && git-ls-files --stage +} + _x40='[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f]' _x40="$_x40$_x40$_x40$_x40$_x40$_x40$_x40$_x40" compare_change () { - sed >current \ + cat current + sed -n >current \ -e '/^--- /d; /^+++ /d; /^@@ /d;' \ - -e 's/^\(.[0-7][0-7][0-7][0-7][0-7][0-7]\) '"$_x40"' /\1 X /' "$1" + -e 's/^\([-+][0-7][0-7][0-7][0-7][0-7][0-7]\) '"$_x40"' /\1 X /p' \ + "$1" diff -u expected current } @@ -68,9 +74,9 @@ test_expect_success \ test_expect_success \ '1, 2, 3 - no carry forward' \ 'rm -f .git/index && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >1-3.out && - cmp M.out 1-3.out && + diff -u M.out 1-3.out && check_cache_at bozbar dirty && check_cache_at frotz dirty && check_cache_at nitfol dirty' @@ -81,9 +87,9 @@ test_expect_success \ '4 - carry forward local addition.' \ 'rm -f .git/index && git-update-cache --add yomin && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >4.out || exit - diff --unified=0 M.out 4.out >4diff.out + diff -u M.out 4.out >4diff.out compare_change 4diff.out expected && check_cache_at yomin clean' @@ -93,9 +99,9 @@ test_expect_success \ echo yomin >yomin && git-update-cache --add yomin && echo yomin yomin >yomin && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >5.out || exit - diff --unified=0 M.out 5.out >5diff.out + diff -u M.out 5.out >5diff.out compare_change 5diff.out expected && check_cache_at yomin dirty' @@ -103,9 +109,9 @@ test_expect_success \ '6 - local addition already has the same.' \ 'rm -f .git/index && git-update-cache --add frotz && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >6.out && - diff --unified=0 M.out 6.out && + diff -u M.out 6.out && check_cache_at frotz clean' test_expect_success \ @@ -114,9 +120,9 @@ test_expect_success \ echo frotz >frotz && git-update-cache --add frotz && echo frotz frotz >frotz && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >7.out && - diff --unified=0 M.out 7.out && + diff -u M.out 7.out && check_cache_at frotz dirty' test_expect_success \ @@ -124,7 +130,7 @@ test_expect_success \ 'rm -f .git/index && echo frotz frotz >frotz && git-update-cache --add frotz && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '9 - conflicting addition.' \ @@ -132,16 +138,16 @@ test_expect_success \ echo frotz frotz >frotz && git-update-cache --add frotz && echo frotz >frotz && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '10 - path removed.' \ 'rm -f .git/index && echo rezrov >rezrov && git-update-cache --add rezrov && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >10.out && - cmp M.out 10.out' + diff -u M.out 10.out' test_expect_success \ '11 - dirty path removed.' \ @@ -149,14 +155,14 @@ test_expect_success \ echo rezrov >rezrov && git-update-cache --add rezrov && echo rezrov rezrov >rezrov && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '12 - unmatching local changes being removed.' \ 'rm -f .git/index && echo rezrov rezrov >rezrov && git-update-cache --add rezrov && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '13 - unmatching local changes being removed.' \ @@ -164,7 +170,7 @@ test_expect_success \ echo rezrov rezrov >rezrov && git-update-cache --add rezrov && echo rezrov >rezrov && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' cat >expected <nitfol && git-update-cache --add nitfol && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >14.out || exit - diff --unified=0 M.out 14.out >14diff.out + diff -u M.out 14.out >14diff.out compare_change 14diff.out expected && check_cache_at nitfol clean' @@ -188,9 +194,9 @@ test_expect_success \ echo nitfol nitfol >nitfol && git-update-cache --add nitfol && echo nitfol nitfol nitfol >nitfol && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >15.out || exit - diff --unified=0 M.out 15.out >15diff.out + diff -u M.out 15.out >15diff.out compare_change 15diff.out expected && check_cache_at nitfol dirty' @@ -199,7 +205,7 @@ test_expect_success \ 'rm -f .git/index && echo bozbar bozbar >bozbar && git-update-cache --add bozbar && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '17 - conflicting local change.' \ @@ -207,16 +213,16 @@ test_expect_success \ echo bozbar bozbar >bozbar && git-update-cache --add bozbar && echo bozbar bozbar bozbar >bozbar && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' test_expect_success \ '18 - local change already having a good result.' \ 'rm -f .git/index && echo gnusto >bozbar && git-update-cache --add bozbar && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >18.out && - diff --unified=0 M.out 18.out && + diff -u M.out 18.out && check_cache_at bozbar clean' test_expect_success \ @@ -225,9 +231,9 @@ test_expect_success \ echo gnusto >bozbar && git-update-cache --add bozbar && echo gnusto gnusto >bozbar && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >19.out && - diff --unified=0 M.out 19.out && + diff -u M.out 19.out && check_cache_at bozbar dirty' test_expect_success \ @@ -235,9 +241,9 @@ test_expect_success \ 'rm -f .git/index && echo bozbar >bozbar && git-update-cache --add bozbar && - git-read-tree -m $treeH $treeM && + read_tree_twoway $treeH $treeM && git-ls-files --stage >20.out && - diff --unified=0 M.out 20.out && + diff -u M.out 20.out && check_cache_at bozbar dirty' test_expect_success \ @@ -246,7 +252,7 @@ test_expect_success \ echo bozbar >bozbar && git-update-cache --add bozbar && echo gnusto gnusto >bozbar && - if git-read-tree -m $treeH $treeM; then false; else :; fi' + if read_tree_twoway $treeH $treeM; then false; else :; fi' # Also make sure we did not break DF vs DF/DF case. test_expect_success \ @@ -273,9 +279,10 @@ test_expect_success \ rm -fr DF && echo DF >DF && git-update-cache --add DF && - git-read-tree -m $treeDF $treeDFDF && + read_tree_twoway $treeDF $treeDFDF && git-ls-files --stage >DFDFcheck.out && - diff --unified=0 DFDF.out DFDFcheck.out && - check_cache_at DF/DF dirty' + diff -u DFDF.out DFDFcheck.out && + check_cache_at DF/DF dirty && + :' test_done