Use process substitution instead of here-string

This fixes an issue where if no test cases are found, an empty test case name would be output and counted towards the total number of test cases.
Beluga-lang · Jan 27, 2024 · 52cb6b1 · 52cb6b1
1 parent 8e17136
commit 52cb6b1
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/TEST.sh b/TEST.sh
@@ -133,7 +133,7 @@ function do_testing {
             start_test_case "${file_path}"
 
             check_example_test_case "${file_path}"
-        done <<< "$(find_compiler_tests_in "${STUDYDIR}" | sort -n)"
+        done < <(find_compiler_tests_in "${STUDYDIR}" | sort -n)
     fi
 
     if [[ -n "${RUN_EXAMPLES_TESTS}" ]]; then
@@ -143,7 +143,7 @@ function do_testing {
             start_test_case "${file_path}"
 
             check_example_test_case "${file_path}"
-        done <<< "$(find_compiler_tests_in "${EXAMPLEDIR}" | sort -n)"
+        done < <(find_compiler_tests_in "${EXAMPLEDIR}" | sort -n)
     fi
 
     if [[ -n "${RUN_COMPILER_TESTS}" ]]; then
@@ -153,7 +153,7 @@ function do_testing {
             start_test_case "${file_path}"
 
             check_compiler_test_case "${file_path}"
-        done <<< "$(find_compiler_tests_in "${TESTDIR}" | sort -n)"
+        done < <(find_compiler_tests_in "${TESTDIR}" | sort -n)
     fi
 
     if [[ -n "${RUN_INTERACTIVE_MODE_TESTS}" ]]; then
@@ -163,7 +163,7 @@ function do_testing {
             start_test_case "${file_path}"
 
             check_interactive "${file_path}"
-        done <<< "$(find "${INTERACTIVE_TESTDIR}" -type f | sort -n)"
+        done < <(find "${INTERACTIVE_TESTDIR}" -type f | sort -n)
     fi
 
     if [[ -n "${RUN_HARPOON_MODE_TESTS}" ]]; then
@@ -173,7 +173,7 @@ function do_testing {
             start_test_case "${file_path}"
 
             check_harpoon "${file_path}"
-        done <<< "$(find "${HARPOON_TESTDIR}" -type f -name "*.input" | sort -n)"
+        done < <(find "${HARPOON_TESTDIR}" -type f -name "*.input" | sort -n)
     fi
 
     echo