Skip to content

Commit 097c28d

Browse files
committed
Merge branch 'rj/test-with-leak-check'
More tests that are supposed to pass leak sanitizer are marked as such. * rj/test-with-leak-check: t0080: mark as leak-free test-lib: check for TEST_PASSES_SANITIZE_LEAK t6113: mark as leak-free t5332: mark as leak-free
2 parents c5887af + 03f72a4 commit 097c28d

File tree

4 files changed

+9
-1
lines changed

4 files changed

+9
-1
lines changed

t/t0080-unit-test-output.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
test_description='Test the output of the unit test framework'
44

5+
TEST_PASSES_SANITIZE_LEAK=true
56
. ./test-lib.sh
67

78
test_expect_success 'TAP output from unit tests' '

t/t5332-multi-pack-reuse.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
test_description='pack-objects multi-pack reuse'
44

5+
TEST_PASSES_SANITIZE_LEAK=true
56
. ./test-lib.sh
67
. "$TEST_DIRECTORY"/lib-bitmap.sh
78

t/t6113-rev-list-bitmap-filters.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
#!/bin/sh
22

33
test_description='rev-list combining bitmaps and filters'
4+
5+
TEST_PASSES_SANITIZE_LEAK=true
46
. ./test-lib.sh
57
. "$TEST_DIRECTORY"/lib-bitmap.sh
68

7-
TEST_PASSES_SANITIZE_LEAK=true
89

910
test_expect_success 'set up bitmapped repo' '
1011
# one commit will have bitmaps, the other will not

t/test-lib.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1297,6 +1297,11 @@ test_done () {
12971297
EOF
12981298
fi
12991299

1300+
if test -z "$passes_sanitize_leak" && test_bool_env TEST_PASSES_SANITIZE_LEAK false
1301+
then
1302+
BAIL_OUT "Please, set TEST_PASSES_SANITIZE_LEAK before sourcing test-lib.sh"
1303+
fi
1304+
13001305
if test "$test_fixed" != 0
13011306
then
13021307
say_color error "# $test_fixed known breakage(s) vanished; please update test(s)"

0 commit comments

Comments
 (0)