Refactor CircleCI suite selection to improve input handling and remove debug output

Author: Tomasz Wojcikowski

tools/test.sh

@@ -86,12 +86,11 @@ select_circleci_suites() {
   local glob_pattern="$1"
   local output_file="$2"
 
-  echo "DEBUG: select_circleci_suites glob_pattern='$glob_pattern' output_file='$output_file'"
   rm -f "$output_file"
   # Use xargs to normalize input to newlines, then sed to process
   # This follows CircleCI docs recommendation to use xargs for input handling
   if ! circleci tests glob "$glob_pattern" | \
-      circleci tests run --command="tee /dev/stderr | xargs -n1 echo | sed 's|.*/||; s|\.erl$||' >>$output_file" --verbose
+      circleci tests run --command="xargs -n1 echo | sed 's|.*/||; s|\.erl$||' >>$output_file" --verbose
   then
     echo "circleci tests run failed for pattern $glob_pattern"
     return 1
@@ -101,8 +100,6 @@ select_circleci_suites() {
     echo "circleci tests run returned no suites for pattern $glob_pattern"
     return 1
   fi
-  echo "DEBUG: Content of $output_file:"
-  cat "$output_file"
 }
 
 maybe_select_small_test_suites() {
@@ -200,7 +197,7 @@ run_test_preset() {
 }
 
 maybe_select_suites() {
-  if circleci_tests_available && select_circleci_suites tests/*_SUITE.erl selected_suites; then
+  if circleci_tests_available && select_circleci_suites "tests/*_SUITE.erl" selected_suites; then
     if [ -s selected_suites ]; then
       escript ../tools/select_suites_to_run.erl $TESTSPEC $(<selected_suites)
     else