git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 04/15] leak tests: mark some misc tests as passing with SANITIZE=leak

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Oct 30, 2021, 22:24 UTC
Message-ID
<patch-04.15-ab9f891a3fe-20211030T221945Z-avarab@gmail.com>
In-Reply-To
<cover-00.15-00000000000-20211030T221945Z-avarab@gmail.com>

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
Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 6 of 17 in “leak tests: remaining non-svn non-p4 tests as passing”
  1. 00/15 leak tests: remaining non-svn non-p4 tests as passingÆvar Arnfjörð Bjarmason, Oct 30, 2021
  2. 02/15 leak tests: mark "sort" test as passing SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  3. 01/15 leak tests: mark a read-tree test as passing SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  4. 03/15 leak tests: mark most gettext tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  5. 05/15 leak tests: mark some rev-list tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  6. 04/15 leak tests: mark some misc tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  7. 07/15 leak tests: mark some update-index tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  8. 06/15 leak tests: mark some rev-parse tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  9. 08/15 leak tests: mark some notes tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  10. 09/15 leak tests: mark some apply tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  11. 10/15 leak tests: mark some diff tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  12. 11/15 leak tests: mark some add tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  13. 12/15 leak tests: mark some clone tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  14. 13/15 leak tests: mark some status tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  15. 14/15 leak tests: mark some config tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  16. 15/15 leak tests: mark some fast-import tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 30, 2021
  17. Elijah NewrenNov 15, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.