From b30fdb4b4e6608ccafdfa4a13893940b85a8a7e8 Mon Sep 17 00:00:00 2001 From: Taylor Blau Date: Thu, 23 Apr 2020 15:41:06 -0600 Subject: [PATCH] t/test-lib.sh: make ULIMIT_FILE_DESCRIPTORS available to tests In t1400 the prerequisite 'ULIMIT_FILE_DESCRIPTORS' is defined and used to effectively guard the helper function 'run_with_limited_open_files' from being used on systems that do not satisfy this prerequisite. In the subsequent patch, we will introduce another test outside of t1400 that would benefit from using this prerequisite. So, move it to 'test-lib.sh' instead so that it can be used by multiple tests. Signed-off-by: Taylor Blau Signed-off-by: Junio C Hamano --- t/t1400-update-ref.sh | 9 --------- t/test-lib.sh | 9 +++++++++ 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/t/t1400-update-ref.sh b/t/t1400-update-ref.sh index b815cdd1b8..65574e0f1c 100755 --- a/t/t1400-update-ref.sh +++ b/t/t1400-update-ref.sh @@ -1342,15 +1342,6 @@ test_expect_success 'fails with duplicate ref update via symref' ' test_cmp expect actual ' -run_with_limited_open_files () { - (ulimit -n 32 && "$@") -} - -test_lazy_prereq ULIMIT_FILE_DESCRIPTORS ' - test_have_prereq !MINGW,!CYGWIN && - run_with_limited_open_files true -' - test_expect_success ULIMIT_FILE_DESCRIPTORS 'large transaction creating branches does not burst open file limit' ' ( for i in $(test_seq 33) diff --git a/t/test-lib.sh b/t/test-lib.sh index 44df51be8f..cb0f93eb58 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -1618,6 +1618,15 @@ test_lazy_prereq ULIMIT_STACK_SIZE ' run_with_limited_stack true ' +run_with_limited_open_files () { + (ulimit -n 32 && "$@") +} + +test_lazy_prereq ULIMIT_FILE_DESCRIPTORS ' + test_have_prereq !MINGW,!CYGWIN && + run_with_limited_open_files true +' + build_option () { git version --build-options | sed -ne "s/^$1: //p"