Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 1 | #!/bin/sh |
| 2 | |
| 3 | test_description='GIT_EDITOR, core.editor, and stuff' |
| 4 | |
| 5 | . ./test-lib.sh |
| 6 | |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 7 | unset EDITOR VISUAL GIT_EDITOR |
| 8 | |
| 9 | test_expect_success 'determine default editor' ' |
| 10 | |
| 11 | vi=$(TERM=vt100 git var GIT_EDITOR) && |
| 12 | test -n "$vi" |
| 13 | |
| 14 | ' |
| 15 | |
Junio C Hamano | e0ae1e6 | 2010-06-21 11:18:54 -0700 | [diff] [blame] | 16 | if ! expr "$vi" : '[a-z]*$' >/dev/null |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 17 | then |
| 18 | vi= |
| 19 | fi |
| 20 | |
| 21 | for i in GIT_EDITOR core_editor EDITOR VISUAL $vi |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 22 | do |
| 23 | cat >e-$i.sh <<-EOF |
Johannes Sixt | 80f0e53 | 2009-03-25 12:48:30 +0100 | [diff] [blame] | 24 | #!$SHELL_PATH |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 25 | echo "Edited by $i" >"\$1" |
| 26 | EOF |
| 27 | chmod +x e-$i.sh |
| 28 | done |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 29 | |
| 30 | if ! test -z "$vi" |
| 31 | then |
| 32 | mv e-$vi.sh $vi |
| 33 | fi |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 34 | |
| 35 | test_expect_success setup ' |
| 36 | |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 37 | msg="Hand-edited" && |
| 38 | test_commit "$msg" && |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 39 | echo "$msg" >expect && |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 40 | git show -s --format=%s > actual && |
Stefan Beller | 9c5b2fa | 2017-10-06 12:00:06 -0700 | [diff] [blame] | 41 | test_cmp expect actual |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 42 | |
| 43 | ' |
| 44 | |
| 45 | TERM=dumb |
| 46 | export TERM |
| 47 | test_expect_success 'dumb should error out when falling back on vi' ' |
| 48 | |
| 49 | if git commit --amend |
| 50 | then |
| 51 | echo "Oops?" |
Junio C Hamano | 7263881 | 2007-12-22 19:14:33 -0800 | [diff] [blame] | 52 | false |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 53 | else |
| 54 | : happy |
| 55 | fi |
| 56 | ' |
| 57 | |
Jonathan Nieder | d33738d | 2009-11-11 17:56:07 -0600 | [diff] [blame] | 58 | test_expect_success 'dumb should prefer EDITOR to VISUAL' ' |
| 59 | |
| 60 | EDITOR=./e-EDITOR.sh && |
| 61 | VISUAL=./e-VISUAL.sh && |
| 62 | export EDITOR VISUAL && |
| 63 | git commit --amend && |
| 64 | test "$(git show -s --format=%s)" = "Edited by EDITOR" |
| 65 | |
| 66 | ' |
| 67 | |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 68 | TERM=vt100 |
| 69 | export TERM |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 70 | for i in $vi EDITOR VISUAL core_editor GIT_EDITOR |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 71 | do |
| 72 | echo "Edited by $i" >expect |
| 73 | unset EDITOR VISUAL GIT_EDITOR |
| 74 | git config --unset-all core.editor |
| 75 | case "$i" in |
| 76 | core_editor) |
| 77 | git config core.editor ./e-core_editor.sh |
| 78 | ;; |
| 79 | [A-Z]*) |
| 80 | eval "$i=./e-$i.sh" |
| 81 | export $i |
| 82 | ;; |
| 83 | esac |
| 84 | test_expect_success "Using $i" ' |
Björn Steinbrink | e70f320 | 2007-11-11 18:38:11 +0100 | [diff] [blame] | 85 | git --exec-path=. commit --amend && |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 86 | git show -s --pretty=oneline | |
| 87 | sed -e "s/^[0-9a-f]* //" >actual && |
Stefan Beller | 9c5b2fa | 2017-10-06 12:00:06 -0700 | [diff] [blame] | 88 | test_cmp expect actual |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 89 | ' |
| 90 | done |
| 91 | |
| 92 | unset EDITOR VISUAL GIT_EDITOR |
| 93 | git config --unset-all core.editor |
Jonathan Nieder | 8f4b576 | 2009-10-30 20:44:41 -0500 | [diff] [blame] | 94 | for i in $vi EDITOR VISUAL core_editor GIT_EDITOR |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 95 | do |
| 96 | echo "Edited by $i" >expect |
| 97 | case "$i" in |
| 98 | core_editor) |
| 99 | git config core.editor ./e-core_editor.sh |
| 100 | ;; |
| 101 | [A-Z]*) |
| 102 | eval "$i=./e-$i.sh" |
| 103 | export $i |
| 104 | ;; |
| 105 | esac |
| 106 | test_expect_success "Using $i (override)" ' |
Björn Steinbrink | e70f320 | 2007-11-11 18:38:11 +0100 | [diff] [blame] | 107 | git --exec-path=. commit --amend && |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 108 | git show -s --pretty=oneline | |
| 109 | sed -e "s/^[0-9a-f]* //" >actual && |
Stefan Beller | 9c5b2fa | 2017-10-06 12:00:06 -0700 | [diff] [blame] | 110 | test_cmp expect actual |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 111 | ' |
| 112 | done |
| 113 | |
SZEDER Gábor | 4362da0 | 2018-05-14 12:28:12 +0200 | [diff] [blame] | 114 | test_expect_success 'editor with a space' ' |
| 115 | echo "echo space >\$1" >"e space.sh" && |
Johannes Sixt | 5b46a42 | 2009-03-17 22:45:22 +0100 | [diff] [blame] | 116 | chmod a+x "e space.sh" && |
| 117 | GIT_EDITOR="./e\ space.sh" git commit --amend && |
| 118 | test space = "$(git show -s --pretty=format:%s)" |
Johannes Schindelin | fc99469 | 2008-03-11 10:56:30 +0100 | [diff] [blame] | 119 | |
| 120 | ' |
| 121 | |
| 122 | unset GIT_EDITOR |
SZEDER Gábor | 4362da0 | 2018-05-14 12:28:12 +0200 | [diff] [blame] | 123 | test_expect_success 'core.editor with a space' ' |
Johannes Schindelin | fc99469 | 2008-03-11 10:56:30 +0100 | [diff] [blame] | 124 | |
Johannes Sixt | 5b46a42 | 2009-03-17 22:45:22 +0100 | [diff] [blame] | 125 | git config core.editor \"./e\ space.sh\" && |
| 126 | git commit --amend && |
| 127 | test space = "$(git show -s --pretty=format:%s)" |
Johannes Schindelin | fc99469 | 2008-03-11 10:56:30 +0100 | [diff] [blame] | 128 | |
| 129 | ' |
| 130 | |
Adam Roben | ef0c2ab | 2007-07-19 22:09:35 -0700 | [diff] [blame] | 131 | test_done |