* [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing
@ 2021-10-30 22:24 Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 01/15] leak tests: mark a read-tree test as passing SANITIZE=leak Ævar Arnfjörð Bjarmason
` (15 more replies)
0 siblings, 16 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
A follow-up to the now-landed series[1] to mark various tests as
passing under SANITIZE=leak.
Existing passing tests were found with:
rm .prove; GIT_SKIP_TESTS=t0027 prove -j8 --state=save t[0-9]*.sh :: --immediate
GIT_TEST_PASSING_SANITIZE_LEAK=true prove -j8 --state=passed
These are all the tests that aren't "git-svn", "git-p4",
Windows-specific, or icase, need to write to / etc.
As before I've merged this to "seen" and all tests pass with whatever
we've got in-flight at the moment, so this should cause minimal
disruption. Passing CI run on top of "master" at [2].
1. https://lore.kernel.org/git/cover-v2-0.9-00000000000-20211012T135343Z-avarab@gmail.com/
2. https://github.com/avar/git/actions/runs/1403093377
Ævar Arnfjörð Bjarmason (15):
leak tests: mark a read-tree test as passing SANITIZE=leak
leak tests: mark "sort" test as passing SANITIZE=leak
leak tests: mark most gettext tests as passing with SANITIZE=leak
leak tests: mark some misc tests as passing with SANITIZE=leak
leak tests: mark some rev-list tests as passing with SANITIZE=leak
leak tests: mark some rev-parse tests as passing with SANITIZE=leak
leak tests: mark some update-index tests as passing with SANITIZE=leak
leak tests: mark some notes tests as passing with SANITIZE=leak
leak tests: mark some apply tests as passing with SANITIZE=leak
leak tests: mark some diff tests as passing with SANITIZE=leak
leak tests: mark some add tests as passing with SANITIZE=leak
leak tests: mark some clone tests as passing with SANITIZE=leak
leak tests: mark some status tests as passing with SANITIZE=leak
leak tests: mark some config tests as passing with SANITIZE=leak
leak tests: mark some fast-import tests as passing with SANITIZE=leak
t/t0071-sort.sh | 1 +
t/t0200-gettext-basic.sh | 1 +
t/t0201-gettext-fallbacks.sh | 1 +
t/t0202-gettext-perl.sh | 1 +
t/t0204-gettext-reencode-sanity.sh | 1 +
t/t1002-read-tree-m-u-2way.sh | 2 ++
t/t1300-config.sh | 1 +
t/t1303-wacky-config.sh | 2 ++
t/t1307-config-blob.sh | 2 ++
t/t1308-config-set.sh | 1 +
t/t1309-early-config.sh | 1 +
t/t1310-config-default.sh | 1 +
t/t1420-lost-found.sh | 2 ++
t/t1503-rev-parse-verify.sh | 1 +
t/t1506-rev-parse-diagnosis.sh | 1 +
t/t1513-rev-parse-prefix.sh | 1 +
t/t1515-rev-parse-outside-repo.sh | 2 ++
t/t1600-index.sh | 1 +
t/t2000-conflict-when-checking-files-out.sh | 1 +
t/t2100-update-cache-badpath.sh | 1 +
t/t2101-update-index-reupdate.sh | 1 +
t/t2102-update-index-symlinks.sh | 1 +
t/t2103-update-index-ignore-missing.sh | 1 +
t/t2104-update-index-skip-worktree.sh | 1 +
t/t2105-update-index-gitfile.sh | 1 +
t/t2200-add-update.sh | 1 +
t/t2201-add-update-typechange.sh | 1 +
t/t2202-add-addremove.sh | 1 +
t/t2204-add-ignored.sh | 1 +
t/t2404-worktree-config.sh | 1 +
t/t3302-notes-index-expensive.sh | 1 +
t/t3303-notes-subtrees.sh | 1 +
t/t3320-notes-merge-worktrees.sh | 1 +
t/t3702-add-edit.sh | 2 ++
t/t3703-add-magic-pathspec.sh | 1 +
t/t3704-add-pathspec-file.sh | 1 +
t/t3908-stash-in-worktree.sh | 1 +
t/t4000-diff-format.sh | 2 ++
t/t4003-diff-rename-1.sh | 2 ++
t/t4004-diff-rename-symlink.sh | 2 ++
t/t4005-diff-rename-2.sh | 2 ++
t/t4006-diff-mode.sh | 2 ++
t/t4007-rename-3.sh | 2 ++
t/t4009-diff-rename-4.sh | 2 ++
t/t4010-diff-pathspec.sh | 2 ++
t/t4011-diff-symlink.sh | 2 ++
t/t4012-diff-binary.sh | 1 +
t/t4020-diff-external.sh | 1 +
t/t4024-diff-optimize-common.sh | 1 +
t/t4027-diff-submodule.sh | 1 +
t/t4029-diff-trailing-space.sh | 1 +
t/t4032-diff-inter-hunk-context.sh | 1 +
t/t4033-diff-patience.sh | 1 +
t/t4034-diff-words.sh | 1 +
t/t4035-diff-quiet.sh | 1 +
t/t4037-diff-r-t-dirs.sh | 1 +
t/t4040-whitespace-status.sh | 2 ++
t/t4046-diff-unmerged.sh | 2 ++
t/t4049-diff-stat-count.sh | 2 ++
t/t4050-diff-histogram.sh | 1 +
t/t4054-diff-bogus-tree.sh | 2 ++
t/t4062-diff-pickaxe.sh | 1 +
t/t4063-diff-blobs.sh | 2 ++
t/t4100-apply-stat.sh | 2 ++
t/t4101-apply-nonl.sh | 2 ++
t/t4102-apply-rename.sh | 2 ++
t/t4105-apply-fuzz.sh | 2 ++
t/t4106-apply-stdin.sh | 2 ++
t/t4109-apply-multifrag.sh | 2 ++
t/t4110-apply-scan.sh | 2 ++
t/t4112-apply-renames.sh | 2 ++
t/t4116-apply-reverse.sh | 2 ++
t/t4118-apply-empty-context.sh | 2 ++
t/t4119-apply-config.sh | 2 ++
t/t4123-apply-shrink.sh | 2 ++
t/t4126-apply-empty.sh | 2 ++
t/t4127-apply-same-fn.sh | 2 ++
t/t4128-apply-root.sh | 2 ++
t/t4129-apply-samemode.sh | 2 ++
t/t4130-apply-criss-cross-rename.sh | 2 ++
t/t4132-apply-removal.sh | 2 ++
t/t4133-apply-filenames.sh | 2 ++
t/t4134-apply-submodule.sh | 2 ++
t/t4136-apply-check.sh | 2 ++
t/t4139-apply-escape.sh | 2 ++
t/t5002-archive-attr-pattern.sh | 1 +
t/t5200-update-server-info.sh | 1 +
t/t5307-pack-missing-commit.sh | 1 +
t/t5555-http-smart-common.sh | 1 +
t/t5602-clone-remote-exec.sh | 1 +
t/t5603-clone-dirname.sh | 2 ++
t/t5701-git-serve.sh | 1 +
t/t5704-protocol-violations.sh | 2 ++
t/t6005-rev-list-count.sh | 1 +
t/t6102-rev-list-unexpected-objects.sh | 1 +
t/t6136-pathspec-in-bare.sh | 1 +
t/t7101-reset-empty-subdirs.sh | 2 ++
t/t7103-reset-bare.sh | 2 ++
t/t7511-status-index.sh | 1 +
t/t7515-status-symlinks.sh | 1 +
t/t7525-status-rename.sh | 1 +
t/t7526-commit-pathspec-file.sh | 1 +
t/t9302-fast-import-unpack-limit.sh | 2 ++
t/t9303-fast-import-compression.sh | 2 ++
104 files changed, 152 insertions(+)
--
2.33.1.1570.g069344fdd45
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 01/15] leak tests: mark a read-tree test as passing SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 02/15] leak tests: mark "sort" " Ævar Arnfjörð Bjarmason
` (14 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
The "t1002-read-tree-m-u-2way.sh" test has passed under SANITIZE=leak
since 04988c8d182 (unpack-trees: introduce preserve_ignored to
unpack_trees_options, 2021-09-27) was combined with
e5a917fcf42 (unpack-trees: don't leak memory in
verify_clean_subdirectory(), 2021-10-07), but as both were in-flight
at the time neither could mark it as passing.
It will now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t1002-read-tree-m-u-2way.sh | 2 ++
1 file changed, 2 insertions(+)
diff --git a/t/t1002-read-tree-m-u-2way.sh b/t/t1002-read-tree-m-u-2way.sh
index 9c05f5e1f51..ca5c5510c73 100755
--- a/t/t1002-read-tree-m-u-2way.sh
+++ b/t/t1002-read-tree-m-u-2way.sh
@@ -8,6 +8,8 @@ test_description='Two way merge with read-tree -m -u $H $M
This is identical to t1001, but uses -u to update the work tree as well.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-read-tree.sh
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 02/15] leak tests: mark "sort" test as passing SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 01/15] leak tests: mark a read-tree test as passing SANITIZE=leak Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 03/15] leak tests: mark most gettext tests as passing with SANITIZE=leak Ævar Arnfjörð Bjarmason
` (13 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark a test that was recently added in e031e9719d2 (test-mergesort:
add test subcommand, 2021-10-01) as passing with SANITIZE=leak. It
will now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t0071-sort.sh | 1 +
1 file changed, 1 insertion(+)
diff --git a/t/t0071-sort.sh b/t/t0071-sort.sh
index a8ab1748796..6f9a501c72b 100755
--- a/t/t0071-sort.sh
+++ b/t/t0071-sort.sh
@@ -2,6 +2,7 @@
test_description='verify sort functions'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'llist_mergesort()' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 03/15] leak tests: mark most gettext tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 01/15] leak tests: mark a read-tree test as passing SANITIZE=leak Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 02/15] leak tests: mark "sort" " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 04/15] leak tests: mark some misc " Ævar Arnfjörð Bjarmason
` (12 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark all but one tests that match "*gettext*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
In the case of "t0202-gettext-perl.sh" this isn't very meaningful as
most of the work is on the Perl side, but let's mark it anyway.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t0200-gettext-basic.sh | 1 +
t/t0201-gettext-fallbacks.sh | 1 +
t/t0202-gettext-perl.sh | 1 +
t/t0204-gettext-reencode-sanity.sh | 1 +
4 files changed, 4 insertions(+)
diff --git a/t/t0200-gettext-basic.sh b/t/t0200-gettext-basic.sh
index 8853d8afb92..522fb2ae696 100755
--- a/t/t0200-gettext-basic.sh
+++ b/t/t0200-gettext-basic.sh
@@ -5,6 +5,7 @@
test_description='Gettext support for Git'
+TEST_PASSES_SANITIZE_LEAK=true
. ./lib-gettext.sh
test_expect_success "sanity: \$GIT_INTERNAL_GETTEXT_SH_SCHEME is set (to $GIT_INTERNAL_GETTEXT_SH_SCHEME)" '
diff --git a/t/t0201-gettext-fallbacks.sh b/t/t0201-gettext-fallbacks.sh
index 6c74df0dc67..8724ce1052d 100755
--- a/t/t0201-gettext-fallbacks.sh
+++ b/t/t0201-gettext-fallbacks.sh
@@ -8,6 +8,7 @@ test_description='Gettext Shell fallbacks'
GIT_INTERNAL_GETTEXT_TEST_FALLBACKS=YesPlease
export GIT_INTERNAL_GETTEXT_TEST_FALLBACKS
+TEST_PASSES_SANITIZE_LEAK=true
. ./lib-gettext.sh
test_expect_success "sanity: \$GIT_INTERNAL_GETTEXT_SH_SCHEME is set (to $GIT_INTERNAL_GETTEXT_SH_SCHEME)" '
diff --git a/t/t0202-gettext-perl.sh b/t/t0202-gettext-perl.sh
index a29d166e007..df2ea34932b 100755
--- a/t/t0202-gettext-perl.sh
+++ b/t/t0202-gettext-perl.sh
@@ -5,6 +5,7 @@
test_description='Perl gettext interface (Git::I18N)'
+TEST_PASSES_SANITIZE_LEAK=true
. ./lib-gettext.sh
if ! test_have_prereq PERL; then
diff --git a/t/t0204-gettext-reencode-sanity.sh b/t/t0204-gettext-reencode-sanity.sh
index 8437e51eb54..4f2e0dcb02b 100755
--- a/t/t0204-gettext-reencode-sanity.sh
+++ b/t/t0204-gettext-reencode-sanity.sh
@@ -5,6 +5,7 @@
test_description="Gettext reencoding of our *.po/*.mo files works"
+TEST_PASSES_SANITIZE_LEAK=true
. ./lib-gettext.sh
# The constants used in a tricky observation for undefined behaviour
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 04/15] leak tests: mark some misc tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (2 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 03/15] leak tests: mark most gettext tests as passing with SANITIZE=leak Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 05/15] leak tests: mark some rev-list " Ævar Arnfjörð Bjarmason
` (11 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
As in 7ff24785cb7 (leak tests: mark some misc tests as passing with
SANITIZE=leak, 2021-10-12) continue marking various miscellaneous
tests as passing when git is compiled with SANITIZE=leak. They'll now
be listed as running under the "GIT_TEST_PASSING_SANITIZE_LEAK=true"
test mode (the "linux-leaks" CI target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t1420-lost-found.sh | 2 ++
t/t1600-index.sh | 1 +
t/t2000-conflict-when-checking-files-out.sh | 1 +
t/t2100-update-cache-badpath.sh | 1 +
t/t3908-stash-in-worktree.sh | 1 +
t/t4007-rename-3.sh | 2 ++
t/t5002-archive-attr-pattern.sh | 1 +
t/t5200-update-server-info.sh | 1 +
t/t5307-pack-missing-commit.sh | 1 +
t/t5555-http-smart-common.sh | 1 +
t/t5701-git-serve.sh | 1 +
t/t5704-protocol-violations.sh | 2 ++
t/t6136-pathspec-in-bare.sh | 1 +
t/t7101-reset-empty-subdirs.sh | 2 ++
t/t7103-reset-bare.sh | 2 ++
t/t7525-status-rename.sh | 1 +
t/t7526-commit-pathspec-file.sh | 1 +
17 files changed, 22 insertions(+)
diff --git a/t/t1420-lost-found.sh b/t/t1420-lost-found.sh
index dc9e402c555..dbe15a0be10 100755
--- a/t/t1420-lost-found.sh
+++ b/t/t1420-lost-found.sh
@@ -4,6 +4,8 @@
#
test_description='Test fsck --lost-found'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t1600-index.sh b/t/t1600-index.sh
index 46329c488b1..010989f90e6 100755
--- a/t/t1600-index.sh
+++ b/t/t1600-index.sh
@@ -2,6 +2,7 @@
test_description='index file specific tests'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
sane_unset GIT_TEST_SPLIT_INDEX
diff --git a/t/t2000-conflict-when-checking-files-out.sh b/t/t2000-conflict-when-checking-files-out.sh
index f18616ad2be..79fc97f1d7e 100755
--- a/t/t2000-conflict-when-checking-files-out.sh
+++ b/t/t2000-conflict-when-checking-files-out.sh
@@ -21,6 +21,7 @@ test_description='git conflicts when checking files out test.'
# path1 is occupied by a non-directory. With "-f" flag, it should remove
# the conflicting paths and succeed.
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
show_files() {
diff --git a/t/t2100-update-cache-badpath.sh b/t/t2100-update-cache-badpath.sh
index 2df3fdde8bf..7915e7b8211 100755
--- a/t/t2100-update-cache-badpath.sh
+++ b/t/t2100-update-cache-badpath.sh
@@ -22,6 +22,7 @@ and tries to git update-index --add the following:
All of the attempts should fail.
'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
mkdir path2 path3
diff --git a/t/t3908-stash-in-worktree.sh b/t/t3908-stash-in-worktree.sh
index 2b2b366ef94..347a89b030b 100755
--- a/t/t3908-stash-in-worktree.sh
+++ b/t/t3908-stash-in-worktree.sh
@@ -5,6 +5,7 @@
test_description='Test git stash in a worktree'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t4007-rename-3.sh b/t/t4007-rename-3.sh
index c634653b5be..b86165cbac5 100755
--- a/t/t4007-rename-3.sh
+++ b/t/t4007-rename-3.sh
@@ -6,6 +6,8 @@
test_description='Rename interaction with pathspec.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh ;# test-lib chdir's into trash
diff --git a/t/t5002-archive-attr-pattern.sh b/t/t5002-archive-attr-pattern.sh
index bda6d7d7e9e..a66b5ba27e8 100755
--- a/t/t5002-archive-attr-pattern.sh
+++ b/t/t5002-archive-attr-pattern.sh
@@ -2,6 +2,7 @@
test_description='git archive attribute pattern tests'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_exists() {
diff --git a/t/t5200-update-server-info.sh b/t/t5200-update-server-info.sh
index 21a58eecb9b..ed9dfd624c7 100755
--- a/t/t5200-update-server-info.sh
+++ b/t/t5200-update-server-info.sh
@@ -2,6 +2,7 @@
test_description='Test git update-server-info'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' 'test_commit file'
diff --git a/t/t5307-pack-missing-commit.sh b/t/t5307-pack-missing-commit.sh
index f4338abb78a..d3482ab279c 100755
--- a/t/t5307-pack-missing-commit.sh
+++ b/t/t5307-pack-missing-commit.sh
@@ -2,6 +2,7 @@
test_description='pack should notice missing commit objects'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t5555-http-smart-common.sh b/t/t5555-http-smart-common.sh
index 49faf5e283b..b1cfe8b7dba 100755
--- a/t/t5555-http-smart-common.sh
+++ b/t/t5555-http-smart-common.sh
@@ -2,6 +2,7 @@
test_description='test functionality common to smart fetch & push'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t5701-git-serve.sh b/t/t5701-git-serve.sh
index aa1827d841d..1896f671cb3 100755
--- a/t/t5701-git-serve.sh
+++ b/t/t5701-git-serve.sh
@@ -5,6 +5,7 @@ test_description='test protocol v2 server commands'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'test capability advertisement' '
diff --git a/t/t5704-protocol-violations.sh b/t/t5704-protocol-violations.sh
index bc393d7c319..ae1a00afb09 100755
--- a/t/t5704-protocol-violations.sh
+++ b/t/t5704-protocol-violations.sh
@@ -4,6 +4,8 @@ test_description='Test responses to violations of the network protocol. In most
of these cases it will generally be acceptable for one side to break off
communications if the other side says something unexpected. We are mostly
making sure that we do not segfault or otherwise behave badly.'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'extra delim packet in v2 ls-refs args' '
diff --git a/t/t6136-pathspec-in-bare.sh b/t/t6136-pathspec-in-bare.sh
index b117251366d..ae8b5379e24 100755
--- a/t/t6136-pathspec-in-bare.sh
+++ b/t/t6136-pathspec-in-bare.sh
@@ -2,6 +2,7 @@
test_description='diagnosing out-of-scope pathspec'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup a bare and non-bare repository' '
diff --git a/t/t7101-reset-empty-subdirs.sh b/t/t7101-reset-empty-subdirs.sh
index 5530651eea4..638bb04e217 100755
--- a/t/t7101-reset-empty-subdirs.sh
+++ b/t/t7101-reset-empty-subdirs.sh
@@ -4,6 +4,8 @@
#
test_description='git reset should cull empty subdirs'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff-data.sh
diff --git a/t/t7103-reset-bare.sh b/t/t7103-reset-bare.sh
index afe36a533c4..0de83e36199 100755
--- a/t/t7103-reset-bare.sh
+++ b/t/t7103-reset-bare.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='git reset in a bare repository'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup non-bare' '
diff --git a/t/t7525-status-rename.sh b/t/t7525-status-rename.sh
index a62736dce09..22bf5c7e5dc 100755
--- a/t/t7525-status-rename.sh
+++ b/t/t7525-status-rename.sh
@@ -2,6 +2,7 @@
test_description='git status rename detection options'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t7526-commit-pathspec-file.sh b/t/t7526-commit-pathspec-file.sh
index 5fbe47ebcd0..dca62fc48e5 100755
--- a/t/t7526-commit-pathspec-file.sh
+++ b/t/t7526-commit-pathspec-file.sh
@@ -2,6 +2,7 @@
test_description='commit --pathspec-from-file'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_tick
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 05/15] leak tests: mark some rev-list tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (3 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 04/15] leak tests: mark some misc " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 06/15] leak tests: mark some rev-parse " Ævar Arnfjörð Bjarmason
` (10 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*rev-list*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t6005-rev-list-count.sh | 1 +
t/t6102-rev-list-unexpected-objects.sh | 1 +
2 files changed, 2 insertions(+)
diff --git a/t/t6005-rev-list-count.sh b/t/t6005-rev-list-count.sh
index 0b64822bf62..d763de0041c 100755
--- a/t/t6005-rev-list-count.sh
+++ b/t/t6005-rev-list-count.sh
@@ -2,6 +2,7 @@
test_description='git rev-list --max-count and --skip test'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t6102-rev-list-unexpected-objects.sh b/t/t6102-rev-list-unexpected-objects.sh
index 52cde097dd5..6f0902b8638 100755
--- a/t/t6102-rev-list-unexpected-objects.sh
+++ b/t/t6102-rev-list-unexpected-objects.sh
@@ -2,6 +2,7 @@
test_description='git rev-list should handle unexpected object types'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup well-formed objects' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 06/15] leak tests: mark some rev-parse tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (4 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 05/15] leak tests: mark some rev-list " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 07/15] leak tests: mark some update-index " Ævar Arnfjörð Bjarmason
` (9 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*rev-parse*" as passing when git is
compiled with SANITIZE=leak. They'll now be listed as running under
the "GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks"
CI target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t1503-rev-parse-verify.sh | 1 +
t/t1506-rev-parse-diagnosis.sh | 1 +
t/t1513-rev-parse-prefix.sh | 1 +
t/t1515-rev-parse-outside-repo.sh | 2 ++
4 files changed, 5 insertions(+)
diff --git a/t/t1503-rev-parse-verify.sh b/t/t1503-rev-parse-verify.sh
index 40958615ebb..94fe413ee37 100755
--- a/t/t1503-rev-parse-verify.sh
+++ b/t/t1503-rev-parse-verify.sh
@@ -9,6 +9,7 @@ exec </dev/null
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
add_line_into_file()
diff --git a/t/t1506-rev-parse-diagnosis.sh b/t/t1506-rev-parse-diagnosis.sh
index 65a154a8a20..18688cae17d 100755
--- a/t/t1506-rev-parse-diagnosis.sh
+++ b/t/t1506-rev-parse-diagnosis.sh
@@ -7,6 +7,7 @@ exec </dev/null
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_did_you_mean ()
diff --git a/t/t1513-rev-parse-prefix.sh b/t/t1513-rev-parse-prefix.sh
index 5f437be8c9e..ba43387bf16 100755
--- a/t/t1513-rev-parse-prefix.sh
+++ b/t/t1513-rev-parse-prefix.sh
@@ -5,6 +5,7 @@ test_description='Tests for rev-parse --prefix'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t1515-rev-parse-outside-repo.sh b/t/t1515-rev-parse-outside-repo.sh
index 3ec2971ee5b..cdb26a30d72 100755
--- a/t/t1515-rev-parse-outside-repo.sh
+++ b/t/t1515-rev-parse-outside-repo.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='check that certain rev-parse options work outside repo'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'set up non-repo directory' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 07/15] leak tests: mark some update-index tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (5 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 06/15] leak tests: mark some rev-parse " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 08/15] leak tests: mark some notes " Ævar Arnfjörð Bjarmason
` (8 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*update-index*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t2101-update-index-reupdate.sh | 1 +
t/t2102-update-index-symlinks.sh | 1 +
t/t2103-update-index-ignore-missing.sh | 1 +
t/t2104-update-index-skip-worktree.sh | 1 +
t/t2105-update-index-gitfile.sh | 1 +
5 files changed, 5 insertions(+)
diff --git a/t/t2101-update-index-reupdate.sh b/t/t2101-update-index-reupdate.sh
index 6c32d42c8c6..e3c7acdbf91 100755
--- a/t/t2101-update-index-reupdate.sh
+++ b/t/t2101-update-index-reupdate.sh
@@ -6,6 +6,7 @@
test_description='git update-index --again test.
'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'update-index --add' '
diff --git a/t/t2102-update-index-symlinks.sh b/t/t2102-update-index-symlinks.sh
index 22f2c730ae8..d7a3485582d 100755
--- a/t/t2102-update-index-symlinks.sh
+++ b/t/t2102-update-index-symlinks.sh
@@ -8,6 +8,7 @@ test_description='git update-index on filesystem w/o symlinks test.
This tests that git update-index keeps the symbolic link property
even if a plain file is in the working tree if core.symlinks is false.'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success \
diff --git a/t/t2103-update-index-ignore-missing.sh b/t/t2103-update-index-ignore-missing.sh
index 0114f052280..b304714fdb1 100755
--- a/t/t2103-update-index-ignore-missing.sh
+++ b/t/t2103-update-index-ignore-missing.sh
@@ -2,6 +2,7 @@
test_description='update-index with options'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success basics '
diff --git a/t/t2104-update-index-skip-worktree.sh b/t/t2104-update-index-skip-worktree.sh
index 30666fc70d2..b8686aabd38 100755
--- a/t/t2104-update-index-skip-worktree.sh
+++ b/t/t2104-update-index-skip-worktree.sh
@@ -5,6 +5,7 @@
test_description='skip-worktree bit test'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
sane_unset GIT_TEST_SPLIT_INDEX
diff --git a/t/t2105-update-index-gitfile.sh b/t/t2105-update-index-gitfile.sh
index a7f3d47aec2..963ebe77eb6 100755
--- a/t/t2105-update-index-gitfile.sh
+++ b/t/t2105-update-index-gitfile.sh
@@ -6,6 +6,7 @@
test_description='git update-index for gitlink to .git file.
'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'submodule with absolute .git file' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 08/15] leak tests: mark some notes tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (6 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 07/15] leak tests: mark some update-index " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 09/15] leak tests: mark some apply " Ævar Arnfjörð Bjarmason
` (7 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*notes*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t3302-notes-index-expensive.sh | 1 +
t/t3303-notes-subtrees.sh | 1 +
t/t3320-notes-merge-worktrees.sh | 1 +
3 files changed, 3 insertions(+)
diff --git a/t/t3302-notes-index-expensive.sh b/t/t3302-notes-index-expensive.sh
index ef8b63952e3..bc9d8ee1e6a 100755
--- a/t/t3302-notes-index-expensive.sh
+++ b/t/t3302-notes-index-expensive.sh
@@ -8,6 +8,7 @@ test_description='Test commit notes index (expensive!)'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
create_repo () {
diff --git a/t/t3303-notes-subtrees.sh b/t/t3303-notes-subtrees.sh
index d47ce00f694..f5fd80d4d3f 100755
--- a/t/t3303-notes-subtrees.sh
+++ b/t/t3303-notes-subtrees.sh
@@ -5,6 +5,7 @@ test_description='Test commit notes organized in subtrees'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
number_of_commits=100
diff --git a/t/t3320-notes-merge-worktrees.sh b/t/t3320-notes-merge-worktrees.sh
index 6b2d507f3e7..bff0aea550f 100755
--- a/t/t3320-notes-merge-worktrees.sh
+++ b/t/t3320-notes-merge-worktrees.sh
@@ -8,6 +8,7 @@ test_description='Test merging of notes trees in multiple worktrees'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup commit' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 09/15] leak tests: mark some apply tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (7 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 08/15] leak tests: mark some notes " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 10/15] leak tests: mark some diff " Ævar Arnfjörð Bjarmason
` (6 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*apply*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t4100-apply-stat.sh | 2 ++
t/t4101-apply-nonl.sh | 2 ++
t/t4102-apply-rename.sh | 2 ++
t/t4105-apply-fuzz.sh | 2 ++
t/t4106-apply-stdin.sh | 2 ++
t/t4109-apply-multifrag.sh | 2 ++
t/t4110-apply-scan.sh | 2 ++
t/t4112-apply-renames.sh | 2 ++
t/t4116-apply-reverse.sh | 2 ++
t/t4118-apply-empty-context.sh | 2 ++
t/t4119-apply-config.sh | 2 ++
t/t4123-apply-shrink.sh | 2 ++
t/t4126-apply-empty.sh | 2 ++
t/t4127-apply-same-fn.sh | 2 ++
t/t4128-apply-root.sh | 2 ++
t/t4129-apply-samemode.sh | 2 ++
t/t4130-apply-criss-cross-rename.sh | 2 ++
t/t4132-apply-removal.sh | 2 ++
t/t4133-apply-filenames.sh | 2 ++
t/t4134-apply-submodule.sh | 2 ++
t/t4136-apply-check.sh | 2 ++
t/t4139-apply-escape.sh | 2 ++
22 files changed, 44 insertions(+)
diff --git a/t/t4100-apply-stat.sh b/t/t4100-apply-stat.sh
index 9b433de8363..d503547732c 100755
--- a/t/t4100-apply-stat.sh
+++ b/t/t4100-apply-stat.sh
@@ -6,6 +6,8 @@
test_description='git apply --stat --summary test, with --recount
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
UNC='s/^\(@@ -[1-9][0-9]*\),[0-9]* \(+[1-9][0-9]*\),[0-9]* @@/\1,999 \2,999 @@/'
diff --git a/t/t4101-apply-nonl.sh b/t/t4101-apply-nonl.sh
index e3443d004d0..b1169193ef5 100755
--- a/t/t4101-apply-nonl.sh
+++ b/t/t4101-apply-nonl.sh
@@ -6,6 +6,8 @@
test_description='git apply should handle files with incomplete lines.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# setup
diff --git a/t/t4102-apply-rename.sh b/t/t4102-apply-rename.sh
index fae305979a8..d1e06fc1ac4 100755
--- a/t/t4102-apply-rename.sh
+++ b/t/t4102-apply-rename.sh
@@ -6,6 +6,8 @@
test_description='git apply handling copy/rename patch.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# setup
diff --git a/t/t4105-apply-fuzz.sh b/t/t4105-apply-fuzz.sh
index 3266e394003..76e2c01b0f5 100755
--- a/t/t4105-apply-fuzz.sh
+++ b/t/t4105-apply-fuzz.sh
@@ -2,6 +2,8 @@
test_description='apply with fuzz and offset'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
dotest () {
diff --git a/t/t4106-apply-stdin.sh b/t/t4106-apply-stdin.sh
index 72467a1e8ee..a57a318699e 100755
--- a/t/t4106-apply-stdin.sh
+++ b/t/t4106-apply-stdin.sh
@@ -2,6 +2,8 @@
test_description='git apply --numstat - <patch'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4109-apply-multifrag.sh b/t/t4109-apply-multifrag.sh
index ac58083fe22..4dc6d8e7d3c 100755
--- a/t/t4109-apply-multifrag.sh
+++ b/t/t4109-apply-multifrag.sh
@@ -6,6 +6,8 @@
test_description='git apply test patches with multiple fragments.'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
cp "$TEST_DIRECTORY/t4109/patch1.patch" .
diff --git a/t/t4110-apply-scan.sh b/t/t4110-apply-scan.sh
index 09f58112e02..266302a1829 100755
--- a/t/t4110-apply-scan.sh
+++ b/t/t4110-apply-scan.sh
@@ -7,6 +7,8 @@
test_description='git apply test for patches which require scanning forwards and backwards.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'git apply scan' '
diff --git a/t/t4112-apply-renames.sh b/t/t4112-apply-renames.sh
index f9ad183758c..d53aa4222ea 100755
--- a/t/t4112-apply-renames.sh
+++ b/t/t4112-apply-renames.sh
@@ -7,6 +7,8 @@ test_description='git apply should not get confused with rename/copy.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# setup
diff --git a/t/t4116-apply-reverse.sh b/t/t4116-apply-reverse.sh
index b99e65c0863..f3b635475a4 100755
--- a/t/t4116-apply-reverse.sh
+++ b/t/t4116-apply-reverse.sh
@@ -7,6 +7,8 @@ test_description='git apply in reverse
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4118-apply-empty-context.sh b/t/t4118-apply-empty-context.sh
index 65f2e4c3efb..5fc6d3e4e7c 100755
--- a/t/t4118-apply-empty-context.sh
+++ b/t/t4118-apply-empty-context.sh
@@ -7,6 +7,8 @@ test_description='git apply with new style GNU diff with empty context
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4119-apply-config.sh b/t/t4119-apply-config.sh
index a9a05838119..208c961d376 100755
--- a/t/t4119-apply-config.sh
+++ b/t/t4119-apply-config.sh
@@ -7,6 +7,8 @@ test_description='git apply --whitespace=strip and configuration file.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4123-apply-shrink.sh b/t/t4123-apply-shrink.sh
index 984157f03b9..ef57cd3aebb 100755
--- a/t/t4123-apply-shrink.sh
+++ b/t/t4123-apply-shrink.sh
@@ -2,6 +2,8 @@
test_description='apply a patch that is larger than the preimage'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
cat >F <<\EOF
diff --git a/t/t4126-apply-empty.sh b/t/t4126-apply-empty.sh
index ceb6a79fe0c..a361e79a815 100755
--- a/t/t4126-apply-empty.sh
+++ b/t/t4126-apply-empty.sh
@@ -2,6 +2,8 @@
test_description='apply empty'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4127-apply-same-fn.sh b/t/t4127-apply-same-fn.sh
index 305b7e649eb..c27e9aec570 100755
--- a/t/t4127-apply-same-fn.sh
+++ b/t/t4127-apply-same-fn.sh
@@ -2,6 +2,8 @@
test_description='apply same filename'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
modify () {
diff --git a/t/t4128-apply-root.sh b/t/t4128-apply-root.sh
index 6cc741a634b..cb3181e8b71 100755
--- a/t/t4128-apply-root.sh
+++ b/t/t4128-apply-root.sh
@@ -2,6 +2,8 @@
test_description='apply same filename'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t4129-apply-samemode.sh b/t/t4129-apply-samemode.sh
index 576632f8681..a1c7686519e 100755
--- a/t/t4129-apply-samemode.sh
+++ b/t/t4129-apply-samemode.sh
@@ -2,6 +2,8 @@
test_description='applying patch with mode bits'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4130-apply-criss-cross-rename.sh b/t/t4130-apply-criss-cross-rename.sh
index f8a313bcb98..f3ea6327425 100755
--- a/t/t4130-apply-criss-cross-rename.sh
+++ b/t/t4130-apply-criss-cross-rename.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='git apply handling criss-cross rename patch.'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
create_file() {
diff --git a/t/t4132-apply-removal.sh b/t/t4132-apply-removal.sh
index fec1d6fa51f..c1e3049c041 100755
--- a/t/t4132-apply-removal.sh
+++ b/t/t4132-apply-removal.sh
@@ -4,6 +4,8 @@
test_description='git-apply notices removal patches generated by GNU diff'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4133-apply-filenames.sh b/t/t4133-apply-filenames.sh
index c5ed3b17c4a..35f1060bc8b 100755
--- a/t/t4133-apply-filenames.sh
+++ b/t/t4133-apply-filenames.sh
@@ -5,6 +5,8 @@
test_description='git apply filename consistency check'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4134-apply-submodule.sh b/t/t4134-apply-submodule.sh
index d1c16ba33c1..aceb4c42b0f 100755
--- a/t/t4134-apply-submodule.sh
+++ b/t/t4134-apply-submodule.sh
@@ -5,6 +5,8 @@
test_description='git apply submodule tests'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4136-apply-check.sh b/t/t4136-apply-check.sh
index 4c3f264a633..dfec1c5f0f6 100755
--- a/t/t4136-apply-check.sh
+++ b/t/t4136-apply-check.sh
@@ -2,6 +2,8 @@
test_description='git apply should exit non-zero with unrecognized input.'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t4139-apply-escape.sh b/t/t4139-apply-escape.sh
index 45b5660a47d..e5c7439df13 100755
--- a/t/t4139-apply-escape.sh
+++ b/t/t4139-apply-escape.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='paths written by git-apply cannot escape the working tree'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# tests will try to write to ../foo, and we do not
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 10/15] leak tests: mark some diff tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (8 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 09/15] leak tests: mark some apply " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 11/15] leak tests: mark some add " Ævar Arnfjörð Bjarmason
` (5 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*diff*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t4000-diff-format.sh | 2 ++
t/t4003-diff-rename-1.sh | 2 ++
t/t4004-diff-rename-symlink.sh | 2 ++
t/t4005-diff-rename-2.sh | 2 ++
t/t4006-diff-mode.sh | 2 ++
t/t4009-diff-rename-4.sh | 2 ++
t/t4010-diff-pathspec.sh | 2 ++
t/t4011-diff-symlink.sh | 2 ++
t/t4012-diff-binary.sh | 1 +
t/t4020-diff-external.sh | 1 +
t/t4024-diff-optimize-common.sh | 1 +
t/t4027-diff-submodule.sh | 1 +
t/t4029-diff-trailing-space.sh | 1 +
t/t4032-diff-inter-hunk-context.sh | 1 +
t/t4033-diff-patience.sh | 1 +
t/t4034-diff-words.sh | 1 +
t/t4035-diff-quiet.sh | 1 +
t/t4037-diff-r-t-dirs.sh | 1 +
t/t4040-whitespace-status.sh | 2 ++
t/t4046-diff-unmerged.sh | 2 ++
t/t4049-diff-stat-count.sh | 2 ++
t/t4050-diff-histogram.sh | 1 +
t/t4054-diff-bogus-tree.sh | 2 ++
t/t4062-diff-pickaxe.sh | 1 +
t/t4063-diff-blobs.sh | 2 ++
25 files changed, 38 insertions(+)
diff --git a/t/t4000-diff-format.sh b/t/t4000-diff-format.sh
index cce334981e1..bfcaae390f3 100755
--- a/t/t4000-diff-format.sh
+++ b/t/t4000-diff-format.sh
@@ -6,6 +6,8 @@
test_description='Test built-in diff output engine.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh
diff --git a/t/t4003-diff-rename-1.sh b/t/t4003-diff-rename-1.sh
index f4485a87c63..181e9683a79 100755
--- a/t/t4003-diff-rename-1.sh
+++ b/t/t4003-diff-rename-1.sh
@@ -6,6 +6,8 @@
test_description='More rename detection
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh ;# test-lib chdir's into trash
diff --git a/t/t4004-diff-rename-symlink.sh b/t/t4004-diff-rename-symlink.sh
index 3d495e37bb1..8def4d4aee9 100755
--- a/t/t4004-diff-rename-symlink.sh
+++ b/t/t4004-diff-rename-symlink.sh
@@ -9,6 +9,8 @@ The rename detection logic should be able to detect pure rename or
copy of symbolic links, but should not produce rename/copy followed
by an edit for them.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh
diff --git a/t/t4005-diff-rename-2.sh b/t/t4005-diff-rename-2.sh
index 6f1b323f979..5c756dc2435 100755
--- a/t/t4005-diff-rename-2.sh
+++ b/t/t4005-diff-rename-2.sh
@@ -5,6 +5,8 @@
test_description='Same rename detection as t4003 but testing diff-raw.'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh ;# test-lib chdir's into trash
diff --git a/t/t4006-diff-mode.sh b/t/t4006-diff-mode.sh
index 6cdee2a2164..dbd4c0da213 100755
--- a/t/t4006-diff-mode.sh
+++ b/t/t4006-diff-mode.sh
@@ -6,6 +6,8 @@
test_description='Test mode change diffs.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
sed_script='s/\(:100644 100755\) \('"$OID_REGEX"'\) \2 /\1 X X /'
diff --git a/t/t4009-diff-rename-4.sh b/t/t4009-diff-rename-4.sh
index 59b7f44f058..3480781dabf 100755
--- a/t/t4009-diff-rename-4.sh
+++ b/t/t4009-diff-rename-4.sh
@@ -6,6 +6,8 @@
test_description='Same rename detection as t4003 but testing diff-raw -z.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh ;# test-lib chdir's into trash
diff --git a/t/t4010-diff-pathspec.sh b/t/t4010-diff-pathspec.sh
index 1bbced79ece..9d9650eba7e 100755
--- a/t/t4010-diff-pathspec.sh
+++ b/t/t4010-diff-pathspec.sh
@@ -9,6 +9,8 @@ Prepare:
file0
path1/file1
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh ;# test-lib chdir's into trash
diff --git a/t/t4011-diff-symlink.sh b/t/t4011-diff-symlink.sh
index 5a25c259fe3..d7a5f7ae780 100755
--- a/t/t4011-diff-symlink.sh
+++ b/t/t4011-diff-symlink.sh
@@ -6,6 +6,8 @@
test_description='Test diff of symlinks.
'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh
diff --git a/t/t4012-diff-binary.sh b/t/t4012-diff-binary.sh
index 33ff588ebca..00eeafb2ace 100755
--- a/t/t4012-diff-binary.sh
+++ b/t/t4012-diff-binary.sh
@@ -6,6 +6,7 @@
test_description='Binary diff and apply
'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
cat >expect.binary-numstat <<\EOF
diff --git a/t/t4020-diff-external.sh b/t/t4020-diff-external.sh
index e009826fcbe..54bb8ef27e7 100755
--- a/t/t4020-diff-external.sh
+++ b/t/t4020-diff-external.sh
@@ -2,6 +2,7 @@
test_description='external diff interface test'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4024-diff-optimize-common.sh b/t/t4024-diff-optimize-common.sh
index 6b44ce14933..540f93715e4 100755
--- a/t/t4024-diff-optimize-common.sh
+++ b/t/t4024-diff-optimize-common.sh
@@ -2,6 +2,7 @@
test_description='common tail optimization'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
z=zzzzzzzz ;# 8
diff --git a/t/t4027-diff-submodule.sh b/t/t4027-diff-submodule.sh
index 94ef77e1dfe..6cef0da982f 100755
--- a/t/t4027-diff-submodule.sh
+++ b/t/t4027-diff-submodule.sh
@@ -2,6 +2,7 @@
test_description='difference in submodules'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh
diff --git a/t/t4029-diff-trailing-space.sh b/t/t4029-diff-trailing-space.sh
index 32b6e9a4e76..5f8ffef74b6 100755
--- a/t/t4029-diff-trailing-space.sh
+++ b/t/t4029-diff-trailing-space.sh
@@ -4,6 +4,7 @@
#
test_description='diff honors config option, diff.suppressBlankEmpty'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
cat <<\EOF >expected ||
diff --git a/t/t4032-diff-inter-hunk-context.sh b/t/t4032-diff-inter-hunk-context.sh
index bada0cbd32f..7db92d0d9f4 100755
--- a/t/t4032-diff-inter-hunk-context.sh
+++ b/t/t4032-diff-inter-hunk-context.sh
@@ -2,6 +2,7 @@
test_description='diff hunk fusing'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
f() {
diff --git a/t/t4033-diff-patience.sh b/t/t4033-diff-patience.sh
index 113304dc596..f7be7f5ef01 100755
--- a/t/t4033-diff-patience.sh
+++ b/t/t4033-diff-patience.sh
@@ -2,6 +2,7 @@
test_description='patience diff algorithm'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff-alternative.sh
diff --git a/t/t4034-diff-words.sh b/t/t4034-diff-words.sh
index 561c582d161..d5abcf4b4c6 100755
--- a/t/t4034-diff-words.sh
+++ b/t/t4034-diff-words.sh
@@ -2,6 +2,7 @@
test_description='word diff colors'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff.sh
diff --git a/t/t4035-diff-quiet.sh b/t/t4035-diff-quiet.sh
index 0352bf81a90..76f8034c60f 100755
--- a/t/t4035-diff-quiet.sh
+++ b/t/t4035-diff-quiet.sh
@@ -2,6 +2,7 @@
test_description='Return value of diffs'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t4037-diff-r-t-dirs.sh b/t/t4037-diff-r-t-dirs.sh
index f5ce3b29a2a..b5f96fe23bd 100755
--- a/t/t4037-diff-r-t-dirs.sh
+++ b/t/t4037-diff-r-t-dirs.sh
@@ -2,6 +2,7 @@
test_description='diff -r -t shows directory additions and deletions'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4040-whitespace-status.sh b/t/t4040-whitespace-status.sh
index 3c728a3ebf9..e70e020ae93 100755
--- a/t/t4040-whitespace-status.sh
+++ b/t/t4040-whitespace-status.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='diff --exit-code with whitespace'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4046-diff-unmerged.sh b/t/t4046-diff-unmerged.sh
index ff7cfd884a4..fd3f3a7260b 100755
--- a/t/t4046-diff-unmerged.sh
+++ b/t/t4046-diff-unmerged.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='diff with unmerged index entries'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4049-diff-stat-count.sh b/t/t4049-diff-stat-count.sh
index 53061b104ec..f5b35e7860e 100755
--- a/t/t4049-diff-stat-count.sh
+++ b/t/t4049-diff-stat-count.sh
@@ -2,6 +2,8 @@
# Copyright (c) 2011, Google Inc.
test_description='diff --stat-count'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t4050-diff-histogram.sh b/t/t4050-diff-histogram.sh
index fd3e86a74f3..c61b30f96da 100755
--- a/t/t4050-diff-histogram.sh
+++ b/t/t4050-diff-histogram.sh
@@ -2,6 +2,7 @@
test_description='histogram diff algorithm'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
. "$TEST_DIRECTORY"/lib-diff-alternative.sh
diff --git a/t/t4054-diff-bogus-tree.sh b/t/t4054-diff-bogus-tree.sh
index 8c95f152b23..294fb553137 100755
--- a/t/t4054-diff-bogus-tree.sh
+++ b/t/t4054-diff-bogus-tree.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='test diff with a bogus tree containing the null sha1'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'create bogus tree' '
diff --git a/t/t4062-diff-pickaxe.sh b/t/t4062-diff-pickaxe.sh
index 1130c8019b4..9aaa068ed9b 100755
--- a/t/t4062-diff-pickaxe.sh
+++ b/t/t4062-diff-pickaxe.sh
@@ -5,6 +5,7 @@
test_description='Pickaxe options'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t4063-diff-blobs.sh b/t/t4063-diff-blobs.sh
index bc69e26c524..7e6c9d63843 100755
--- a/t/t4063-diff-blobs.sh
+++ b/t/t4063-diff-blobs.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='test direct comparison of blobs via git-diff'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
run_diff () {
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 11/15] leak tests: mark some add tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (9 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 10/15] leak tests: mark some diff " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 12/15] leak tests: mark some clone " Ævar Arnfjörð Bjarmason
` (4 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*add*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t2200-add-update.sh | 1 +
t/t2201-add-update-typechange.sh | 1 +
t/t2202-add-addremove.sh | 1 +
t/t2204-add-ignored.sh | 1 +
t/t3702-add-edit.sh | 2 ++
t/t3703-add-magic-pathspec.sh | 1 +
t/t3704-add-pathspec-file.sh | 1 +
7 files changed, 8 insertions(+)
diff --git a/t/t2200-add-update.sh b/t/t2200-add-update.sh
index 94c4cb06721..a8297c29434 100755
--- a/t/t2200-add-update.sh
+++ b/t/t2200-add-update.sh
@@ -14,6 +14,7 @@ only the updates to dir/sub.
Also tested are "git add -u" without limiting, and "git add -u"
without contents changes, and other conditions'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t2201-add-update-typechange.sh b/t/t2201-add-update-typechange.sh
index a4eec0a3465..c6876b120f8 100755
--- a/t/t2201-add-update-typechange.sh
+++ b/t/t2201-add-update-typechange.sh
@@ -2,6 +2,7 @@
test_description='more git add -u'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t2202-add-addremove.sh b/t/t2202-add-addremove.sh
index 9ee659098c4..24c60bfd790 100755
--- a/t/t2202-add-addremove.sh
+++ b/t/t2202-add-addremove.sh
@@ -2,6 +2,7 @@
test_description='git add --all'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t2204-add-ignored.sh b/t/t2204-add-ignored.sh
index 2e07365bbb0..89424abccd1 100755
--- a/t/t2204-add-ignored.sh
+++ b/t/t2204-add-ignored.sh
@@ -2,6 +2,7 @@
test_description='giving ignored paths to git add'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t3702-add-edit.sh b/t/t3702-add-edit.sh
index 6c676645d83..a1801a8cbd4 100755
--- a/t/t3702-add-edit.sh
+++ b/t/t3702-add-edit.sh
@@ -4,6 +4,8 @@
#
test_description='add -e basic tests'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
diff --git a/t/t3703-add-magic-pathspec.sh b/t/t3703-add-magic-pathspec.sh
index 3ef525a559d..d84071038e0 100755
--- a/t/t3703-add-magic-pathspec.sh
+++ b/t/t3703-add-magic-pathspec.sh
@@ -2,6 +2,7 @@
test_description='magic pathspec tests using git-add'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
diff --git a/t/t3704-add-pathspec-file.sh b/t/t3704-add-pathspec-file.sh
index 9e35c1fbca6..5d5164d1fc6 100755
--- a/t/t3704-add-pathspec-file.sh
+++ b/t/t3704-add-pathspec-file.sh
@@ -2,6 +2,7 @@
test_description='add --pathspec-from-file'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_tick
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 12/15] leak tests: mark some clone tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (10 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 11/15] leak tests: mark some add " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 13/15] leak tests: mark some status " Ævar Arnfjörð Bjarmason
` (3 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*clone*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t5602-clone-remote-exec.sh | 1 +
t/t5603-clone-dirname.sh | 2 ++
2 files changed, 3 insertions(+)
diff --git a/t/t5602-clone-remote-exec.sh b/t/t5602-clone-remote-exec.sh
index cbcceab9d56..56329aa160e 100755
--- a/t/t5602-clone-remote-exec.sh
+++ b/t/t5602-clone-remote-exec.sh
@@ -2,6 +2,7 @@
test_description=clone
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success setup '
diff --git a/t/t5603-clone-dirname.sh b/t/t5603-clone-dirname.sh
index 13b5e5eb9b9..8ca1f094237 100755
--- a/t/t5603-clone-dirname.sh
+++ b/t/t5603-clone-dirname.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='check output directory names used by git-clone'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# we use a fake ssh wrapper that ignores the arguments
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 13/15] leak tests: mark some status tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (11 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 12/15] leak tests: mark some clone " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 14/15] leak tests: mark some config " Ævar Arnfjörð Bjarmason
` (2 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*status*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t7511-status-index.sh | 1 +
t/t7515-status-symlinks.sh | 1 +
2 files changed, 2 insertions(+)
diff --git a/t/t7511-status-index.sh b/t/t7511-status-index.sh
index b5fdc048a54..4ffa45a7bf3 100755
--- a/t/t7511-status-index.sh
+++ b/t/t7511-status-index.sh
@@ -2,6 +2,7 @@
test_description='git status with certain file name lengths'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
files="0 1 2 3 4 5 6 7 8 9 a b c d e f g h i j k l m n o p q r s t u v w x y z"
diff --git a/t/t7515-status-symlinks.sh b/t/t7515-status-symlinks.sh
index 9f989be01b9..e3d6bb67bf9 100755
--- a/t/t7515-status-symlinks.sh
+++ b/t/t7515-status-symlinks.sh
@@ -2,6 +2,7 @@
test_description='git status and symlinks'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 14/15] leak tests: mark some config tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (12 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 13/15] leak tests: mark some status " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 15/15] leak tests: mark some fast-import " Ævar Arnfjörð Bjarmason
2021-11-15 22:14 ` [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Elijah Newren
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*config*" as passing when git is compiled
with SANITIZE=leak. They'll now be listed as running under the
"GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks" CI
target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t1300-config.sh | 1 +
t/t1303-wacky-config.sh | 2 ++
t/t1307-config-blob.sh | 2 ++
t/t1308-config-set.sh | 1 +
t/t1309-early-config.sh | 1 +
t/t1310-config-default.sh | 1 +
t/t2404-worktree-config.sh | 1 +
7 files changed, 9 insertions(+)
diff --git a/t/t1300-config.sh b/t/t1300-config.sh
index 9ff46f3b047..f8031afaaf9 100755
--- a/t/t1300-config.sh
+++ b/t/t1300-config.sh
@@ -8,6 +8,7 @@ test_description='Test git config in different settings'
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'clear default config' '
diff --git a/t/t1303-wacky-config.sh b/t/t1303-wacky-config.sh
index 0000e664e7b..0506f3d6bba 100755
--- a/t/t1303-wacky-config.sh
+++ b/t/t1303-wacky-config.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='Test wacky input to git config'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# Leaving off the newline is intentional!
diff --git a/t/t1307-config-blob.sh b/t/t1307-config-blob.sh
index 930dce06f0f..0a7099d6f52 100755
--- a/t/t1307-config-blob.sh
+++ b/t/t1307-config-blob.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='support for reading config from a blob'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'create config blob' '
diff --git a/t/t1308-config-set.sh b/t/t1308-config-set.sh
index 88b119a0a35..b38e158d3b2 100755
--- a/t/t1308-config-set.sh
+++ b/t/t1308-config-set.sh
@@ -2,6 +2,7 @@
test_description='Test git config-set API in different settings'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
# 'check_config get_* section.key value' verifies that the entry for
diff --git a/t/t1309-early-config.sh b/t/t1309-early-config.sh
index b4a9158307f..537435b90ae 100755
--- a/t/t1309-early-config.sh
+++ b/t/t1309-early-config.sh
@@ -2,6 +2,7 @@
test_description='Test read_early_config()'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'read early config' '
diff --git a/t/t1310-config-default.sh b/t/t1310-config-default.sh
index 6049d917081..09b10c144ba 100755
--- a/t/t1310-config-default.sh
+++ b/t/t1310-config-default.sh
@@ -2,6 +2,7 @@
test_description='Test git config in different settings (with --default)'
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'uses --default when entry missing' '
diff --git a/t/t2404-worktree-config.sh b/t/t2404-worktree-config.sh
index 9536d109195..842937bfb9a 100755
--- a/t/t2404-worktree-config.sh
+++ b/t/t2404-worktree-config.sh
@@ -2,6 +2,7 @@
test_description="config file in multi worktree"
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 15/15] leak tests: mark some fast-import tests as passing with SANITIZE=leak
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (13 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 14/15] leak tests: mark some config " Ævar Arnfjörð Bjarmason
@ 2021-10-30 22:24 ` Ævar Arnfjörð Bjarmason
2021-11-15 22:14 ` [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Elijah Newren
15 siblings, 0 replies; 17+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2021-10-30 22:24 UTC (permalink / raw)
To: git
Cc: Junio C Hamano, Elijah Newren, Martin Ågren, Andrzej Hunt,
Jeff King, Ævar Arnfjörð Bjarmason
Mark some tests that match "*fast-import*" as passing when git is
compiled with SANITIZE=leak. They'll now be listed as running under
the "GIT_TEST_PASSING_SANITIZE_LEAK=true" test mode (the "linux-leaks"
CI target).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
t/t9302-fast-import-unpack-limit.sh | 2 ++
t/t9303-fast-import-compression.sh | 2 ++
2 files changed, 4 insertions(+)
diff --git a/t/t9302-fast-import-unpack-limit.sh b/t/t9302-fast-import-unpack-limit.sh
index f519e4f1bfe..d8b1f9442e8 100755
--- a/t/t9302-fast-import-unpack-limit.sh
+++ b/t/t9302-fast-import-unpack-limit.sh
@@ -1,5 +1,7 @@
#!/bin/sh
test_description='test git fast-import unpack limit'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'create loose objects on import' '
diff --git a/t/t9303-fast-import-compression.sh b/t/t9303-fast-import-compression.sh
index 57d916524ec..4f5bf40587c 100755
--- a/t/t9303-fast-import-compression.sh
+++ b/t/t9303-fast-import-compression.sh
@@ -1,6 +1,8 @@
#!/bin/sh
test_description='compression setting of fast-import utility'
+
+TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
import_large () {
--
2.33.1.1570.g069344fdd45
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
` (14 preceding siblings ...)
2021-10-30 22:24 ` [PATCH 15/15] leak tests: mark some fast-import " Ævar Arnfjörð Bjarmason
@ 2021-11-15 22:14 ` Elijah Newren
15 siblings, 0 replies; 17+ messages in thread
From: Elijah Newren @ 2021-11-15 22:14 UTC (permalink / raw)
To: Ævar Arnfjörð Bjarmason
Cc: Git Mailing List, Junio C Hamano, Martin Ågren, Andrzej Hunt,
Jeff King
On Sat, Oct 30, 2021 at 3:24 PM Ævar Arnfjörð Bjarmason
<avarab@gmail.com> wrote:
>
> A follow-up to the now-landed series[1] to mark various tests as
> passing under SANITIZE=leak.
>
> Existing passing tests were found with:
>
> rm .prove; GIT_SKIP_TESTS=t0027 prove -j8 --state=save t[0-9]*.sh :: --immediate
> GIT_TEST_PASSING_SANITIZE_LEAK=true prove -j8 --state=passed
>
> These are all the tests that aren't "git-svn", "git-p4",
> Windows-specific, or icase, need to write to / etc.
>
> As before I've merged this to "seen" and all tests pass with whatever
> we've got in-flight at the moment, so this should cause minimal
> disruption. Passing CI run on top of "master" at [2].
>
> 1. https://lore.kernel.org/git/cover-v2-0.9-00000000000-20211012T135343Z-avarab@gmail.com/
> 2. https://github.com/avar/git/actions/runs/1403093377
>
> Ævar Arnfjörð Bjarmason (15):
> leak tests: mark a read-tree test as passing SANITIZE=leak
> leak tests: mark "sort" test as passing SANITIZE=leak
> leak tests: mark most gettext tests as passing with SANITIZE=leak
> leak tests: mark some misc tests as passing with SANITIZE=leak
> leak tests: mark some rev-list tests as passing with SANITIZE=leak
> leak tests: mark some rev-parse tests as passing with SANITIZE=leak
> leak tests: mark some update-index tests as passing with SANITIZE=leak
> leak tests: mark some notes tests as passing with SANITIZE=leak
> leak tests: mark some apply tests as passing with SANITIZE=leak
> leak tests: mark some diff tests as passing with SANITIZE=leak
> leak tests: mark some add tests as passing with SANITIZE=leak
> leak tests: mark some clone tests as passing with SANITIZE=leak
> leak tests: mark some status tests as passing with SANITIZE=leak
> leak tests: mark some config tests as passing with SANITIZE=leak
> leak tests: mark some fast-import tests as passing with SANITIZE=leak
>
> t/t0071-sort.sh | 1 +
> t/t0200-gettext-basic.sh | 1 +
> t/t0201-gettext-fallbacks.sh | 1 +
> t/t0202-gettext-perl.sh | 1 +
> t/t0204-gettext-reencode-sanity.sh | 1 +
> t/t1002-read-tree-m-u-2way.sh | 2 ++
> t/t1300-config.sh | 1 +
> t/t1303-wacky-config.sh | 2 ++
> t/t1307-config-blob.sh | 2 ++
> t/t1308-config-set.sh | 1 +
> t/t1309-early-config.sh | 1 +
> t/t1310-config-default.sh | 1 +
> t/t1420-lost-found.sh | 2 ++
> t/t1503-rev-parse-verify.sh | 1 +
> t/t1506-rev-parse-diagnosis.sh | 1 +
> t/t1513-rev-parse-prefix.sh | 1 +
> t/t1515-rev-parse-outside-repo.sh | 2 ++
> t/t1600-index.sh | 1 +
> t/t2000-conflict-when-checking-files-out.sh | 1 +
> t/t2100-update-cache-badpath.sh | 1 +
> t/t2101-update-index-reupdate.sh | 1 +
> t/t2102-update-index-symlinks.sh | 1 +
> t/t2103-update-index-ignore-missing.sh | 1 +
> t/t2104-update-index-skip-worktree.sh | 1 +
> t/t2105-update-index-gitfile.sh | 1 +
> t/t2200-add-update.sh | 1 +
> t/t2201-add-update-typechange.sh | 1 +
> t/t2202-add-addremove.sh | 1 +
> t/t2204-add-ignored.sh | 1 +
> t/t2404-worktree-config.sh | 1 +
> t/t3302-notes-index-expensive.sh | 1 +
> t/t3303-notes-subtrees.sh | 1 +
> t/t3320-notes-merge-worktrees.sh | 1 +
> t/t3702-add-edit.sh | 2 ++
> t/t3703-add-magic-pathspec.sh | 1 +
> t/t3704-add-pathspec-file.sh | 1 +
> t/t3908-stash-in-worktree.sh | 1 +
> t/t4000-diff-format.sh | 2 ++
> t/t4003-diff-rename-1.sh | 2 ++
> t/t4004-diff-rename-symlink.sh | 2 ++
> t/t4005-diff-rename-2.sh | 2 ++
> t/t4006-diff-mode.sh | 2 ++
> t/t4007-rename-3.sh | 2 ++
> t/t4009-diff-rename-4.sh | 2 ++
> t/t4010-diff-pathspec.sh | 2 ++
> t/t4011-diff-symlink.sh | 2 ++
> t/t4012-diff-binary.sh | 1 +
> t/t4020-diff-external.sh | 1 +
> t/t4024-diff-optimize-common.sh | 1 +
> t/t4027-diff-submodule.sh | 1 +
> t/t4029-diff-trailing-space.sh | 1 +
> t/t4032-diff-inter-hunk-context.sh | 1 +
> t/t4033-diff-patience.sh | 1 +
> t/t4034-diff-words.sh | 1 +
> t/t4035-diff-quiet.sh | 1 +
> t/t4037-diff-r-t-dirs.sh | 1 +
> t/t4040-whitespace-status.sh | 2 ++
> t/t4046-diff-unmerged.sh | 2 ++
> t/t4049-diff-stat-count.sh | 2 ++
> t/t4050-diff-histogram.sh | 1 +
> t/t4054-diff-bogus-tree.sh | 2 ++
> t/t4062-diff-pickaxe.sh | 1 +
> t/t4063-diff-blobs.sh | 2 ++
> t/t4100-apply-stat.sh | 2 ++
> t/t4101-apply-nonl.sh | 2 ++
> t/t4102-apply-rename.sh | 2 ++
> t/t4105-apply-fuzz.sh | 2 ++
> t/t4106-apply-stdin.sh | 2 ++
> t/t4109-apply-multifrag.sh | 2 ++
> t/t4110-apply-scan.sh | 2 ++
> t/t4112-apply-renames.sh | 2 ++
> t/t4116-apply-reverse.sh | 2 ++
> t/t4118-apply-empty-context.sh | 2 ++
> t/t4119-apply-config.sh | 2 ++
> t/t4123-apply-shrink.sh | 2 ++
> t/t4126-apply-empty.sh | 2 ++
> t/t4127-apply-same-fn.sh | 2 ++
> t/t4128-apply-root.sh | 2 ++
> t/t4129-apply-samemode.sh | 2 ++
> t/t4130-apply-criss-cross-rename.sh | 2 ++
> t/t4132-apply-removal.sh | 2 ++
> t/t4133-apply-filenames.sh | 2 ++
> t/t4134-apply-submodule.sh | 2 ++
> t/t4136-apply-check.sh | 2 ++
> t/t4139-apply-escape.sh | 2 ++
> t/t5002-archive-attr-pattern.sh | 1 +
> t/t5200-update-server-info.sh | 1 +
> t/t5307-pack-missing-commit.sh | 1 +
> t/t5555-http-smart-common.sh | 1 +
> t/t5602-clone-remote-exec.sh | 1 +
> t/t5603-clone-dirname.sh | 2 ++
> t/t5701-git-serve.sh | 1 +
> t/t5704-protocol-violations.sh | 2 ++
> t/t6005-rev-list-count.sh | 1 +
> t/t6102-rev-list-unexpected-objects.sh | 1 +
> t/t6136-pathspec-in-bare.sh | 1 +
> t/t7101-reset-empty-subdirs.sh | 2 ++
> t/t7103-reset-bare.sh | 2 ++
> t/t7511-status-index.sh | 1 +
> t/t7515-status-symlinks.sh | 1 +
> t/t7525-status-rename.sh | 1 +
> t/t7526-commit-pathspec-file.sh | 1 +
> t/t9302-fast-import-unpack-limit.sh | 2 ++
> t/t9303-fast-import-compression.sh | 2 ++
> 104 files changed, 152 insertions(+)
>
> --
> 2.33.1.1570.g069344fdd45
Sorry for not responding earlier; you sent this on the day of our big
migration (consolidating Git servers, reworking CI pipelines, etc.) at
$DAYJOB. Anyway, I've now read through this series. It's simple and
straightforward, helps us on our journey to clean up leaks and
ensuring they stay clean (nice to see that this covers almost 5k
tests), and I couldn't spot any problems. Thanks for working on this!
Reviewed-by: Elijah Newren <newren@gmail.com>
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2021-11-15 23:12 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-30 22:24 [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 01/15] leak tests: mark a read-tree test as passing SANITIZE=leak Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 02/15] leak tests: mark "sort" " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 03/15] leak tests: mark most gettext tests as passing with SANITIZE=leak Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 04/15] leak tests: mark some misc " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 05/15] leak tests: mark some rev-list " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 06/15] leak tests: mark some rev-parse " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 07/15] leak tests: mark some update-index " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 08/15] leak tests: mark some notes " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 09/15] leak tests: mark some apply " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 10/15] leak tests: mark some diff " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 11/15] leak tests: mark some add " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 12/15] leak tests: mark some clone " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 13/15] leak tests: mark some status " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 14/15] leak tests: mark some config " Ævar Arnfjörð Bjarmason
2021-10-30 22:24 ` [PATCH 15/15] leak tests: mark some fast-import " Ævar Arnfjörð Bjarmason
2021-11-15 22:14 ` [PATCH 00/15] leak tests: remaining non-svn non-p4 tests as passing Elijah Newren
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).