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

[PATCH 00/10] leak tests: mark more tests as passing

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Oct 6, 2021, 09:50 UTC
Message-ID
<cover-00.10-00000000000-20211006T094705Z-avarab@gmail.com>

This goes on top of ab/sanitize-leak-ci, in that topic I introduced a "linux-leaks" job that runs in CI and checks that we don't have SANITIZE=leak regressions, but it just marked one test file as passing under that mode.

That was out of an abundance of caution, and to not conflate any potential failures with the mode itself.

This series marks up a lot of tests as passing, ensuring that they won't regress when it comes to memory leaks.

I did a "git rebase -i --exec" of each of these commits where I merged with origin/seen, ran all tests under GIT_TEST_PASSING_SANITIZE_LEAK=true. They all pass, so hopefully these won't cause any disruption. I was careful to leave out any areas that are actively being worked on (e.g. I had to eject a test in the "sparse" area, since it would pass on master+ab/sanitize-leak-ci, but has a regression in "seen". I didn't look into why).

Ævar Arnfjörð Bjarmason (10):
  leak tests: run various built-in tests in t00*.sh SANITIZE=leak
  leak tests: run various "test-tool" tests in t00*.sh SANITIZE=leak
  leak tests: mark t0000-init.sh as passing with SANITIZE=leak
  leak tests: mark all ls-tree tests as passing with SANITIZE=leak
  leak tests: mark all trace2 tests as passing with SANITIZE=leak
  leak tests: mark all checkout-index tests as passing with
    SANITIZE=leak
  leak tests: mark some ls-files tests as passing with SANITIZE=leak
  leak tests: mark some read-tree tests as passing with SANITIZE=leak
  leak tests: mark various "generic" tests as passing with SANITIZE=leak
  leak tests: mark some misc tests as passing with SANITIZE=leak
 t/t0001-init.sh                         | 1 +
 t/t0002-gitfile.sh                      | 1 +
 t/t0003-attributes.sh                   | 1 +
 t/t0005-signals.sh                      | 2 ++
 t/t0007-git-var.sh                      | 2 ++
 t/t0008-ignores.sh                      | 1 +
 t/t0010-racy-git.sh                     | 1 +
 t/t0013-sha1dc.sh                       | 2 ++
 t/t0022-crlf-rename.sh                  | 1 +
 t/t0024-crlf-archive.sh                 | 1 +
 t/t0025-crlf-renormalize.sh             | 1 +
 t/t0026-eol-config.sh                   | 1 +
 t/t0029-core-unsetenvvars.sh            | 1 +
 t/t0052-simple-ipc.sh                   | 1 +
 t/t0055-beyond-symlinks.sh              | 1 +
 t/t0061-run-command.sh                  | 1 +
 t/t0065-strcmp-offset.sh                | 1 +
 t/t0066-dir-iterator.sh                 | 1 +
 t/t0067-parse_pathspec_file.sh          | 1 +
 t/t0210-trace2-normal.sh                | 2 ++
 t/t0211-trace2-perf.sh                  | 2 ++
 t/t0212-trace2-event.sh                 | 2 ++
 t/t1000-read-tree-m-3way.sh             | 2 ++
 t/t1003-read-tree-prefix.sh             | 1 +
 t/t1009-read-tree-new-index.sh          | 1 +
 t/t1010-mktree.sh                       | 1 +
 t/t1012-read-tree-df.sh                 | 1 +
 t/t1014-read-tree-confusing.sh          | 2 ++
 t/t1100-commit-tree-options.sh          | 1 +
 t/t1430-bad-ref-name.sh                 | 1 +
 t/t1504-ceiling-dirs.sh                 | 2 ++
 t/t1510-repo-setup.sh                   | 1 +
 t/t2002-checkout-cache-u.sh             | 1 +
 t/t2003-checkout-cache-mkdir.sh         | 1 +
 t/t2004-checkout-cache-temp.sh          | 1 +
 t/t2005-checkout-index-symlinks.sh      | 1 +
 t/t2050-git-dir-relative.sh             | 1 +
 t/t2081-parallel-checkout-collisions.sh | 1 +
 t/t2300-cd-to-toplevel.sh               | 1 +
 t/t3000-ls-files-others.sh              | 2 ++
 t/t3002-ls-files-dashpath.sh            | 2 ++
 t/t3003-ls-files-exclude.sh             | 2 ++
 t/t3004-ls-files-basic.sh               | 1 +
 t/t3006-ls-files-long.sh                | 2 ++
 t/t3008-ls-files-lazy-init-name-hash.sh | 1 +
 t/t3070-wildmatch.sh                    | 1 +
 t/t3100-ls-tree-restrict.sh             | 2 ++
 t/t3101-ls-tree-dirname.sh              | 2 ++
 t/t3102-ls-tree-wildcards.sh            | 1 +
 t/t3103-ls-tree-misc.sh                 | 1 +
 t/t3205-branch-color.sh                 | 1 +
 t/t3211-peel-ref.sh                     | 1 +
 t/t3300-funny-names.sh                  | 1 +
 t/t3601-rm-pathspec-file.sh             | 1 +
 t/t3902-quoted.sh                       | 1 +
 t/t4002-diff-basic.sh                   | 2 ++
 t/t4016-diff-quote.sh                   | 1 +
 t/t4019-diff-wserror.sh                 | 1 +
 t/t4025-hunk-header.sh                  | 1 +
 t/t4026-color.sh                        | 2 ++
 t/t4300-merge-tree.sh                   | 2 ++
 t/t5580-unc-paths.sh                    | 1 +
 t/t5615-alternate-env.sh                | 2 ++
 t/t7518-ident-corner-cases.sh           | 2 ++
 t/t7812-grep-icase-non-ascii.sh         | 1 +
 t/t7813-grep-icase-iso.sh               | 1 +
 t/t7816-grep-binary-pattern.sh          | 1 +
 67 files changed, 87 insertions(+)
-- 
2.33.0.1441.gbbcdb4c3c66
Next: Ævar Arnfjörð Bjarmason
Message 1 of 27 in “leak tests: mark more tests as passing”
  1. 00/10 leak tests: mark more tests as passingÆvar Arnfjörð Bjarmason, Oct 6, 2021
  2. 01/10 leak tests: run various built-in tests in t00*.sh SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  3. Junio C HamanoOct 7, 2021
  4. 02/10 leak tests: run various "test-tool" tests in t00*.sh SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  5. 03/10 leak tests: mark t0000-init.sh as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  6. Junio C HamanoOct 7, 2021
  7. Ævar Arnfjörð BjarmasonOct 12, 2021
  8. 04/10 leak tests: mark all ls-tree tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  9. 05/10 leak tests: mark all trace2 tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  10. 07/10 leak tests: mark some ls-files tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  11. 06/10 leak tests: mark all checkout-index tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  12. 09/10 leak tests: mark various "generic" tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  13. 08/10 leak tests: mark some read-tree tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  14. 10/10 leak tests: mark some misc tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 6, 2021
  15. Elijah NewrenOct 6, 2021
  16. Junio C HamanoOct 7, 2021
  17. Ævar Arnfjörð BjarmasonOct 12, 2021
  18. 0/9 leak tests: mark more tests as passingÆvar Arnfjörð Bjarmason, Oct 12, 2021
  19. 1/9 leak tests: run various built-in tests in t00*.sh SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  20. 2/9 leak tests: run various "test-tool" tests in t00*.sh SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  21. 3/9 leak tests: mark all ls-tree tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  22. 4/9 leak tests: mark all trace2 tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  23. 5/9 leak tests: mark all checkout-index tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  24. 6/9 leak tests: mark some ls-files tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  25. 7/9 leak tests: mark some read-tree tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  26. 8/9 leak tests: mark various "generic" tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 2021
  27. 9/9 leak tests: mark some misc tests as passing with SANITIZE=leakÆvar Arnfjörð Bjarmason, Oct 12, 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.