Merge branch 'jk/use-write-script-more'
authorJunio C Hamano <gitster@pobox.com>
Fri, 22 Apr 2016 22:45:09 +0000 (15:45 -0700)
committerJunio C Hamano <gitster@pobox.com>
Fri, 22 Apr 2016 22:45:09 +0000 (15:45 -0700)
Code clean-up.

* jk/use-write-script-more:
t3404: use write_script
t1020: do not overuse printf and use write_script
t5532: use write_script

1  2 
t/t3404-rebase-interactive.sh
index b79f442acf901db1a9049e46e93f09d44a64e975,d6d65a3a9435a456f8efad15b8c7c63db00a8e5f..d96d0e4c9b517ecb712d643cdd13e08bff066036
@@@ -555,10 -555,9 +555,9 @@@ test_expect_success 'rebase a detached 
  test_expect_success 'rebase a commit violating pre-commit' '
  
        mkdir -p .git/hooks &&
-       PRE_COMMIT=.git/hooks/pre-commit &&
-       echo "#!/bin/sh" > $PRE_COMMIT &&
-       echo "test -z \"\$(git diff --cached --check)\"" >> $PRE_COMMIT &&
-       chmod a+x $PRE_COMMIT &&
+       write_script .git/hooks/pre-commit <<-\EOF &&
+       test -z "$(git diff --cached --check)"
+       EOF
        echo "monde! " >> file1 &&
        test_tick &&
        test_must_fail git commit -m doesnt-verify file1 &&
@@@ -771,6 -770,7 +770,6 @@@ test_expect_success 'rebase-i history w
        test_cmp expect actual
  '
  
 -
  test_expect_success 'prepare for rebase -i --exec' '
        git checkout master &&
        git checkout -b execute &&
        test_commit three_exec main.txt three_exec
  '
  
 -
  test_expect_success 'running "git rebase -i --exec git show HEAD"' '
        set_fake_editor &&
        git rebase -i --exec "git show HEAD" HEAD~2 >actual &&
        test_cmp expected actual
  '
  
 -
  test_expect_success 'running "git rebase --exec git show HEAD -i"' '
        git reset --hard execute &&
        set_fake_editor &&
        test_cmp expected actual
  '
  
 -
  test_expect_success 'running "git rebase -ix git show HEAD"' '
        git reset --hard execute &&
        set_fake_editor &&
@@@ -831,6 -834,7 +830,6 @@@ test_expect_success 'rebase -ix with se
        test_cmp expected actual
  '
  
 -
  test_expect_success 'rebase -ix with several instances of --exec' '
        git reset --hard execute &&
        set_fake_editor &&
        test_cmp expected actual
  '
  
 -
  test_expect_success 'rebase -ix with --autosquash' '
        git reset --hard execute &&
        git checkout -b autosquash &&
        test_cmp expected actual
  '
  
 -
 -test_expect_success 'rebase --exec without -i shows error message' '
 +test_expect_success 'rebase --exec works without -i ' '
        git reset --hard execute &&
 -      set_fake_editor &&
 -      test_must_fail git rebase --exec "git show HEAD" HEAD~2 2>actual &&
 -      echo "The --exec option must be used with the --interactive option" >expected &&
 -      test_i18ncmp expected actual
 +      rm -rf exec_output &&
 +      EDITOR="echo >invoked_editor" git rebase --exec "echo a line >>exec_output"  HEAD~2 2>actual &&
 +      test_i18ngrep  "Successfully rebased and updated" actual &&
 +      test_line_count = 2 exec_output &&
 +      test_path_is_missing invoked_editor
  '
  
 -
  test_expect_success 'rebase -i --exec without <CMD>' '
        git reset --hard execute &&
        set_fake_editor &&