aboutsummaryrefslogtreecommitdiff
path: root/test/T400-hooks.sh
diff options
context:
space:
mode:
authorJani Nikula <jani@nikula.org>2017-02-26 15:43:00 +0200
committerDavid Bremner <david@tethera.net>2017-03-09 09:01:21 -0400
commitd0cd253b37646d9364d046e1d3edb022a58c105a (patch)
treeff9af83a8bd1e3380e81c3e3d73389fc98d190a7 /test/T400-hooks.sh
parentb8f12bd3717aa446ef04197734af7a055e6909fe (diff)
test: require test_begin_subtest before test_expect_success
Unify the subtests by requiring test_begin_subtest before test_expect_success. (Similar change for test_expect_code will follow.) This increases clarity in the test scripts by having a separate line for the start of the subtest with the heading, and makes it possible to simplify the test infrastructure by making all subtests similar.
Diffstat (limited to 'test/T400-hooks.sh')
-rwxr-xr-xtest/T400-hooks.sh4
1 files changed, 2 insertions, 2 deletions
diff --git a/test/T400-hooks.sh b/test/T400-hooks.sh
index ed119131..ae571ba2 100755
--- a/test/T400-hooks.sh
+++ b/test/T400-hooks.sh
@@ -91,11 +91,11 @@ test_expect_equal_file expected output
# depends on the previous subtest leaving broken hook behind
test_expect_code 1 "post-new non-zero exit status (notmuch status)" "notmuch new"
+test_begin_subtest "post-insert hook does not affect insert status"
rm_hooks
generate_message
create_failing_hook "post-insert"
-test_expect_success "post-insert hook does not affect insert status" \
- "notmuch insert < \"$gen_msg_filename\" > /dev/null"
+test_expect_success "notmuch insert < \"$gen_msg_filename\" > /dev/null"
# test_begin_subtest "hook without executable permissions"
rm_hooks