Skip to content

t3420 remove progress from output

Phillip Wood via GitGitGadget git 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest1 patch 0 repliesNew
  2. v11 patch 4 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1t3420: remove progress lines before comparing outputt/t3420-rebase-autostash.sh+11 -80 replies

Reviews trailers per patch

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

Read the v2 thread

Cover letter [PATCH v2 0/1] t3420 remove progress from output

Remove ugliness in the tests that check the output of git rebase

I've updated this to avoid repeated calls to printf as suggested by Junio

Based-On: sg/rebase-progress

Phillip Wood (1):
t3420: remove progress lines before comparing output

 t/t3420-rebase-autostash.sh | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

base-commit: 5b12e3123b7b70e3875404a4ffe571ca079364fe
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-276%2Fphillipwood%2Fwip%2Ft3420-remove-progress-from-output-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-276/phillipwood/wip/t3420-remove-progress-from-output-v2
Pull-Request: https://github.com/gitgitgadget/git/pull/276

Range-diff vs v1:

1: 52501623f6 ! 1: e1b4023bc0 t3420: remove progress lines before comparing output

     @@ -20,6 +20,16 @@
       diff --git a/t/t3420-rebase-autostash.sh b/t/t3420-rebase-autostash.sh
       --- a/t/t3420-rebase-autostash.sh
       +++ b/t/t3420-rebase-autostash.sh
     +@@
     + 	echo conflicting-change >file2 &&
     + 	git add . &&
     + 	test_tick &&
     +-	git commit -m "related commit"
     ++	git commit -m "related commit" &&
     ++	remove_progress_re="$(printf "s/.*\\r//")"
     + '
     + 
     + create_expected_success_am () {
      @@
       	q_to_cr >expected <<-EOF
       	$(grep "^Created autostash: [0-9a-f][0-9a-f]*\$" actual)
     @@ -53,7 +63,7 @@
       		fi &&
       		create_expected_success_$suffix &&
      -		test_i18ncmp expected actual
     -+		sed "$(printf "s/.*\\r//")" <actual >actual2 &&
     ++		sed "$remove_progress_re" <actual >actual2 &&
      +		test_i18ncmp expected actual2
       	'
       
     @@ -63,7 +73,7 @@
       		fi &&
       		create_expected_failure_$suffix &&
      -		test_i18ncmp expected actual
     -+		sed "$(printf "s/.*\\r//")" <actual >actual2 &&
     ++		sed "$remove_progress_re" <actual >actual2 &&
      +		test_i18ncmp expected actual2
       	'
       }
Signature · 1 line
gitgitgadget