]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t/README: document how to loop around test cases
authorPatrick Steinhardt <ps@pks.im>
Fri, 22 Mar 2024 02:23:46 +0000 (03:23 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 22 Mar 2024 14:36:35 +0000 (07:36 -0700)
In some cases it makes sense to loop around test cases so that we can
execute the same test with slightly different arguments. There are some
gotchas around quoting here though that are easy to miss and that may
lead to easy-to-miss errors and portability issues.

Document the proper way to do this in "t/README".

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/README

index 36463d074259e3805e6de6d41d30bc3632b369f5..4422ab9b980d7b1cc9fc21f69c08f843356b4910 100644 (file)
--- a/t/README
+++ b/t/README
@@ -721,6 +721,26 @@ The "do's:"
    Note that we still &&-chain the loop to propagate failures from
    earlier commands.
 
+ - Repeat tests with slightly different arguments in a loop.
+
+   In some cases it may make sense to re-run the same set of tests with
+   different options or commands to ensure that the command behaves
+   despite the different parameters. This can be achieved by looping
+   around a specific parameter:
+
+       for arg in '' "--foo"
+       do
+               test_expect_success "test command ${arg:-without arguments}" '
+                       command $arg
+               '
+       done
+
+   Note that while the test title uses double quotes ("), the test body
+   should continue to use single quotes (') to avoid breakage in case the
+   values contain e.g. quoting characters. The loop variable will be
+   accessible regardless of the single quotes as the test body is passed
+   to `eval`.
+
 
 And here are the "don'ts:"