On Fri, Mar 11 2022, Jeff Hostetler via GitGitGadget wrote:
> From: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx>
> Fix code style of added "if then".
*nod*
> Add body of builtin test to a test_expect_success.
>
> Signed-off-by: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx>
> ---
> t/perf/p7519-fsmonitor.sh | 21 +++++++++++----------
> 1 file changed, 11 insertions(+), 10 deletions(-)
>
> diff --git a/t/perf/p7519-fsmonitor.sh b/t/perf/p7519-fsmonitor.sh
> index 5f97edf6a11..7a7981b0e61 100755
> --- a/t/perf/p7519-fsmonitor.sh
> +++ b/t/perf/p7519-fsmonitor.sh
> @@ -141,7 +141,7 @@ test_expect_success "one time repo setup" '
> fi
> '
>
> -setup_for_fsmonitor () {
> +setup_for_fsmonitor_hook () {
> # set INTEGRATION_SCRIPT depending on the environment
> if test -n "$INTEGRATION_PATH"
> then
> @@ -182,8 +182,7 @@ test_perf_w_drop_caches () {
> }
>
> test_fsmonitor_suite () {
> - if test -n "$USE_FSMONITOR_DAEMON"
> - then
> + if test -n "$USE_FSMONITOR_DAEMON"; then
> DESC="builtin fsmonitor--daemon"
This "; then" etc. still uses the non-standard style (and this was added
in aa072da617e (t/perf/p7519: add fsmonitor--daemon test cases,
2022-03-01).
