]>
Commit | Line | Data |
---|---|---|
264474f2 WC |
1 | #!/bin/sh |
2 | ||
3 | test_description='commit-msg hook' | |
4 | ||
5 | . ./test-lib.sh | |
6 | ||
80f86605 | 7 | test_expect_success 'with no hook' ' |
264474f2 | 8 | |
80f86605 WC |
9 | echo "foo" > file && |
10 | git add file && | |
11 | git commit -m "first" | |
12 | ||
13 | ' | |
14 | ||
15 | # set up fake editor for interactive editing | |
16 | cat > fake-editor <<'EOF' | |
17 | #!/bin/sh | |
18 | cp FAKE_MSG "$1" | |
19 | exit 0 | |
20 | EOF | |
21 | chmod +x fake-editor | |
f69e836f BD |
22 | |
23 | ## Not using test_set_editor here so we can easily ensure the editor variable | |
24 | ## is only set for the editor tests | |
80f86605 WC |
25 | FAKE_EDITOR="$(pwd)/fake-editor" |
26 | export FAKE_EDITOR | |
27 | ||
28 | test_expect_success 'with no hook (editor)' ' | |
29 | ||
30 | echo "more foo" >> file && | |
31 | git add file && | |
32 | echo "more foo" > FAKE_MSG && | |
f69e836f | 33 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit |
80f86605 WC |
34 | |
35 | ' | |
36 | ||
37 | test_expect_success '--no-verify with no hook' ' | |
38 | ||
39 | echo "bar" > file && | |
40 | git add file && | |
41 | git commit --no-verify -m "bar" | |
42 | ||
43 | ' | |
44 | ||
45 | test_expect_success '--no-verify with no hook (editor)' ' | |
46 | ||
47 | echo "more bar" > file && | |
48 | git add file && | |
49 | echo "more bar" > FAKE_MSG && | |
f69e836f | 50 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit --no-verify |
80f86605 WC |
51 | |
52 | ' | |
264474f2 WC |
53 | |
54 | # now install hook that always succeeds | |
55 | HOOKDIR="$(git rev-parse --git-dir)/hooks" | |
56 | HOOK="$HOOKDIR/commit-msg" | |
57 | mkdir -p "$HOOKDIR" | |
58 | cat > "$HOOK" <<EOF | |
59 | #!/bin/sh | |
60 | exit 0 | |
61 | EOF | |
62 | chmod +x "$HOOK" | |
63 | ||
80f86605 | 64 | test_expect_success 'with succeeding hook' ' |
264474f2 | 65 | |
80f86605 WC |
66 | echo "more" >> file && |
67 | git add file && | |
68 | git commit -m "more" | |
69 | ||
70 | ' | |
71 | ||
72 | test_expect_success 'with succeeding hook (editor)' ' | |
73 | ||
74 | echo "more more" >> file && | |
75 | git add file && | |
76 | echo "more more" > FAKE_MSG && | |
f69e836f | 77 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit |
80f86605 WC |
78 | |
79 | ' | |
80 | ||
81 | test_expect_success '--no-verify with succeeding hook' ' | |
82 | ||
83 | echo "even more" >> file && | |
84 | git add file && | |
85 | git commit --no-verify -m "even more" | |
86 | ||
87 | ' | |
88 | ||
89 | test_expect_success '--no-verify with succeeding hook (editor)' ' | |
90 | ||
91 | echo "even more more" >> file && | |
92 | git add file && | |
93 | echo "even more more" > FAKE_MSG && | |
f69e836f | 94 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit --no-verify |
80f86605 WC |
95 | |
96 | ' | |
264474f2 WC |
97 | |
98 | # now a hook that fails | |
99 | cat > "$HOOK" <<EOF | |
100 | #!/bin/sh | |
101 | exit 1 | |
102 | EOF | |
103 | ||
f8b86359 SB |
104 | commit_msg_is () { |
105 | test "$(git log --pretty=format:%s%b -1)" = "$1" | |
106 | } | |
107 | ||
41ac414e | 108 | test_expect_success 'with failing hook' ' |
80f86605 WC |
109 | |
110 | echo "another" >> file && | |
111 | git add file && | |
d492b31c | 112 | test_must_fail git commit -m "another" |
80f86605 WC |
113 | |
114 | ' | |
115 | ||
41ac414e | 116 | test_expect_success 'with failing hook (editor)' ' |
80f86605 WC |
117 | |
118 | echo "more another" >> file && | |
119 | git add file && | |
120 | echo "more another" > FAKE_MSG && | |
f69e836f | 121 | ! (GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit) |
264474f2 | 122 | |
80f86605 WC |
123 | ' |
124 | ||
125 | test_expect_success '--no-verify with failing hook' ' | |
126 | ||
127 | echo "stuff" >> file && | |
128 | git add file && | |
129 | git commit --no-verify -m "stuff" | |
130 | ||
131 | ' | |
132 | ||
133 | test_expect_success '--no-verify with failing hook (editor)' ' | |
134 | ||
135 | echo "more stuff" >> file && | |
136 | git add file && | |
137 | echo "more stuff" > FAKE_MSG && | |
f69e836f | 138 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit --no-verify |
80f86605 WC |
139 | |
140 | ' | |
264474f2 | 141 | |
f8b86359 SB |
142 | test_expect_success 'merge fails with failing hook' ' |
143 | ||
144 | test_when_finished "git branch -D newbranch" && | |
145 | test_when_finished "git checkout -f master" && | |
146 | git checkout --orphan newbranch && | |
147 | : >file2 && | |
148 | git add file2 && | |
149 | git commit --no-verify file2 -m in-side-branch && | |
150 | test_must_fail git merge --allow-unrelated-histories master && | |
151 | commit_msg_is "in-side-branch" # HEAD before merge | |
152 | ||
153 | ' | |
154 | ||
155 | test_expect_success 'merge bypasses failing hook with --no-verify' ' | |
156 | ||
157 | test_when_finished "git branch -D newbranch" && | |
158 | test_when_finished "git checkout -f master" && | |
159 | git checkout --orphan newbranch && | |
160 | : >file2 && | |
161 | git add file2 && | |
162 | git commit --no-verify file2 -m in-side-branch && | |
163 | git merge --no-verify --allow-unrelated-histories master && | |
164 | commit_msg_is "Merge branch '\''master'\'' into newbranch" | |
165 | ' | |
166 | ||
167 | ||
264474f2 | 168 | chmod -x "$HOOK" |
ee9fb68c | 169 | test_expect_success POSIXPERM 'with non-executable hook' ' |
80f86605 WC |
170 | |
171 | echo "content" >> file && | |
172 | git add file && | |
173 | git commit -m "content" | |
174 | ||
175 | ' | |
176 | ||
ee9fb68c | 177 | test_expect_success POSIXPERM 'with non-executable hook (editor)' ' |
80f86605 WC |
178 | |
179 | echo "content again" >> file && | |
180 | git add file && | |
181 | echo "content again" > FAKE_MSG && | |
f69e836f | 182 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit -m "content again" |
80f86605 WC |
183 | |
184 | ' | |
185 | ||
ee9fb68c | 186 | test_expect_success POSIXPERM '--no-verify with non-executable hook' ' |
80f86605 WC |
187 | |
188 | echo "more content" >> file && | |
189 | git add file && | |
190 | git commit --no-verify -m "more content" | |
191 | ||
192 | ' | |
264474f2 | 193 | |
ee9fb68c | 194 | test_expect_success POSIXPERM '--no-verify with non-executable hook (editor)' ' |
80f86605 WC |
195 | |
196 | echo "even more content" >> file && | |
197 | git add file && | |
198 | echo "even more content" > FAKE_MSG && | |
f69e836f | 199 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit --no-verify |
80f86605 WC |
200 | |
201 | ' | |
264474f2 WC |
202 | |
203 | # now a hook that edits the commit message | |
204 | cat > "$HOOK" <<'EOF' | |
205 | #!/bin/sh | |
206 | echo "new message" > "$1" | |
207 | exit 0 | |
208 | EOF | |
209 | chmod +x "$HOOK" | |
210 | ||
80f86605 WC |
211 | test_expect_success 'hook edits commit message' ' |
212 | ||
213 | echo "additional" >> file && | |
214 | git add file && | |
215 | git commit -m "additional" && | |
216 | commit_msg_is "new message" | |
217 | ||
218 | ' | |
219 | ||
220 | test_expect_success 'hook edits commit message (editor)' ' | |
221 | ||
222 | echo "additional content" >> file && | |
223 | git add file && | |
224 | echo "additional content" > FAKE_MSG && | |
f69e836f | 225 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit && |
80f86605 WC |
226 | commit_msg_is "new message" |
227 | ||
228 | ' | |
229 | ||
230 | test_expect_success "hook doesn't edit commit message" ' | |
231 | ||
232 | echo "plus" >> file && | |
233 | git add file && | |
234 | git commit --no-verify -m "plus" && | |
235 | commit_msg_is "plus" | |
236 | ||
237 | ' | |
238 | ||
239 | test_expect_success "hook doesn't edit commit message (editor)" ' | |
240 | ||
241 | echo "more plus" >> file && | |
242 | git add file && | |
243 | echo "more plus" > FAKE_MSG && | |
f69e836f | 244 | GIT_EDITOR="\"\$FAKE_EDITOR\"" git commit --no-verify && |
80f86605 | 245 | commit_msg_is "more plus" |
f8b86359 | 246 | ' |
80f86605 | 247 | |
f8b86359 SB |
248 | test_expect_success 'hook called in git-merge picks up commit message' ' |
249 | test_when_finished "git branch -D newbranch" && | |
250 | test_when_finished "git checkout -f master" && | |
251 | git checkout --orphan newbranch && | |
252 | : >file2 && | |
253 | git add file2 && | |
254 | git commit --no-verify file2 -m in-side-branch && | |
255 | git merge --allow-unrelated-histories master && | |
256 | commit_msg_is "new message" | |
257 | ' | |
258 | ||
259 | test_expect_failure 'merge --continue remembers --no-verify' ' | |
260 | test_when_finished "git branch -D newbranch" && | |
261 | test_when_finished "git checkout -f master" && | |
262 | git checkout master && | |
263 | echo a >file2 && | |
264 | git add file2 && | |
265 | git commit --no-verify -m "add file2 to master" && | |
266 | git checkout -b newbranch master^ && | |
267 | echo b >file2 && | |
268 | git add file2 && | |
269 | git commit --no-verify file2 -m in-side-branch && | |
270 | git merge --no-verify -m not-rewritten-by-hook master && | |
271 | # resolve conflict: | |
272 | echo c >file2 && | |
273 | git add file2 && | |
274 | git merge --continue && | |
275 | commit_msg_is not-rewritten-by-hook | |
80f86605 | 276 | ' |
264474f2 | 277 | |
cb7fb9ed JS |
278 | # set up fake editor to replace `pick` by `reword` |
279 | cat > reword-editor <<'EOF' | |
280 | #!/bin/sh | |
281 | mv "$1" "$1".bup && | |
282 | sed 's/^pick/reword/' <"$1".bup >"$1" | |
283 | EOF | |
284 | chmod +x reword-editor | |
285 | REWORD_EDITOR="$(pwd)/reword-editor" | |
286 | export REWORD_EDITOR | |
287 | ||
b92ff6e8 | 288 | test_expect_success 'hook is called for reword during `rebase -i`' ' |
cb7fb9ed JS |
289 | |
290 | GIT_SEQUENCE_EDITOR="\"$REWORD_EDITOR\"" git rebase -i HEAD^ && | |
291 | commit_msg_is "new message" | |
292 | ||
293 | ' | |
294 | ||
f8b86359 | 295 | |
264474f2 | 296 | test_done |