Skip to content

Commit 62a23c9

Browse files
tgummerergitster
authored andcommitted
perf-lib: fix start/stop of perf tests
ae75342 test-lib: rearrange start/end of test_expect_* and test_skip changed the way tests are started/stopped, but did not update the perf tests. They were therefore giving the wrong output, because of the wrong test count. Fix this by starting and stopping the tests correctly. Signed-off-by: Thomas Gummerer <[email protected]> Acked-by: Thomas Rast <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent ad0e623 commit 62a23c9

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

t/perf/perf-lib.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ exit $ret' >&3 2>&4
150150

151151

152152
test_perf () {
153+
test_start_
153154
test "$#" = 3 && { test_prereq=$1; shift; } || test_prereq=
154155
test "$#" = 2 ||
155156
error "bug in the test script: not 2 or 3 parameters to test-expect-success"
@@ -187,7 +188,7 @@ test_perf () {
187188
base="$perf_results_dir"/"$perf_results_prefix$(basename "$0" .sh)"."$test_count"
188189
"$TEST_DIRECTORY"/perf/min_time.perl test_time.* >"$base".times
189190
fi
190-
echo >&3 ""
191+
test_finish_
191192
}
192193

193194
# We extend test_done to print timings at the end (./run disables this

0 commit comments

Comments
 (0)