From abf831b5fa54e0e8a534075098e747f0e9142c97 Mon Sep 17 00:00:00 2001 From: Victoria Dye Date: Thu, 5 Aug 2021 19:11:59 -0400 Subject: [PATCH] ci: run `contrib/subtree` tests in CI builds Because `git subtree` (unlike most other `contrib` modules) is included as part of the standard release of Git for Windows, its stability should be verified as consistently as it is for the rest of git. By including the `git subtree` tests in the CI workflow, these tests are as much of a gate to merging and indicator of stability as the standard test suite. Signed-off-by: Victoria Dye --- ci/run-build-and-tests.sh | 6 +++--- ci/run-test-slice.sh | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/ci/run-build-and-tests.sh b/ci/run-build-and-tests.sh index 3ce81ffee941b2..a14a6307c09aa9 100755 --- a/ci/run-build-and-tests.sh +++ b/ci/run-build-and-tests.sh @@ -27,13 +27,13 @@ linux-gcc) export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=master export GIT_TEST_WRITE_REV_INDEX=1 export GIT_TEST_CHECKOUT_WORKERS=2 - make test + make test && make -C contrib/subtree test || exit 1 ;; linux-clang) export GIT_TEST_DEFAULT_HASH=sha1 make test export GIT_TEST_DEFAULT_HASH=sha256 - make test + make test && make -C contrib/subtree test || exit 1 ;; linux-gcc-4.8) # Don't run the tests; we only care about whether Git can be @@ -41,7 +41,7 @@ linux-gcc-4.8) # constructs that newer compilers seem to quietly accept. ;; *) - make test + make test && make -C contrib/subtree test || exit 1 ;; esac diff --git a/ci/run-test-slice.sh b/ci/run-test-slice.sh index f8c2c3106a2ef4..e7f0f923cc5a51 100755 --- a/ci/run-test-slice.sh +++ b/ci/run-test-slice.sh @@ -14,4 +14,7 @@ make --quiet -C t T="$(cd t && ./helper/test-tool path-utils slice-tests "$1" "$2" t[0-9]*.sh | tr '\n' ' ')" +# Run the git subtree tests only if main tests succeeded +test 0 != "$1" || make -C contrib/subtree test + check_unignored_build_artifacts