Phillip Wood [off-list ref] writes:

The patch looks good to me, it is very convenient that we can put the 
test in the library files rather than each test file. Should we drop 
these tests from the ASan job as well?

That's probably a good idea.

I also was wondering if we want a blanket NO_FOO_TESTS that we can
use instead listing all.  That FOO should not be SCM, though, as the
reason why we exclude the tests is not because they are test about
foreign SCM.  We exclude them as low value because testing them
exercises little code of ours that we may make mistakes these checks
are trying to uncover and that we can fix when they do.
NO_FOREIGN_CODE_TESTS?  I dunno.

Thanks.

Thanks

Phillip

quoted

Suggested-by: Phillip Wood <redacted>
Signed-off-by: Junio C Hamano <redacted>
---
  ci/lib.sh       | 2 ++
  t/lib-cvs.sh    | 6 ++++++
  t/lib-git-p4.sh | 5 +++++
  3 files changed, 13 insertions(+)

diff --git a/ci/lib.sh b/ci/lib.sh
index a165c7f268..3ecbf147db 100755
--- a/ci/lib.sh
+++ b/ci/lib.sh

@@ -356,7 +356,9 @@ linux-musl-meson)
  	;;
  linux-leaks|linux-reftable-leaks)
  	export SANITIZE=leak
+	export NO_CVS_TESTS=LetsSaveSomeTime
  	export NO_SVN_TESTS=LetsSaveSomeTime
+	export NO_P4_TESTS=LetsSaveSomeTime
  	;;
  linux-asan-ubsan)
  	export SANITIZE=address,undefined

diff --git a/t/lib-cvs.sh b/t/lib-cvs.sh
index 57b9b2db9b..c8b4404888 100644
--- a/t/lib-cvs.sh
+++ b/t/lib-cvs.sh

@@ -2,6 +2,12 @@
  
  . ./test-lib.sh
  
+if test -n "$NO_CVS_TESTS"
+then
+	skip_all='skipping git cvs tests, NO_CVS_TESTS defined'
+	test_done
+fi
+
  unset CVS_SERVER
  
  if ! type cvs >/dev/null 2>&1

diff --git a/t/lib-git-p4.sh b/t/lib-git-p4.sh
index 2a5b8738ea..d22e9c684a 100644
--- a/t/lib-git-p4.sh
+++ b/t/lib-git-p4.sh

@@ -16,6 +16,11 @@ P4D_TIMEOUT=300
  
  . ./test-lib.sh
  
+if test -n "$NO_P4_TESTS"
+then
+	skip_all='skipping git p4 tests, NO_P4_TESTS defined'
+	test_done
+fi
  if ! test_have_prereq PYTHON
  then
  	skip_all='skipping git p4 tests; python not available'