refactor: replace compgen with a safer globbing approach

Author: Chemaclass

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 ## Unreleased
 
-- Fix build issue with bashunit on NixOS
+- Fix parallel and `compgen` issue on NixOS
 
 ## [0.22.2](https://github.com/TypedDevs/bashunit/compare/0.22.1...0.22.2) - 2025-07-26
 

src/math.sh

@@ -1,12 +1,12 @@
 #!/usr/bin/env bash
 
-if dependencies::has_bc; then
-  # bc is better than awk because bc has no integer limits.
-  function math::calculate() {
+function math::calculate() {
+  if dependencies::has_bc; then
     echo "$*" | bc
-  }
-elif dependencies::has_awk; then
-  function math::calculate() {
-    awk "BEGIN { print ""$*"" }"
-  }
-fi
+  elif dependencies::has_awk; then
+    awk "BEGIN { print ($*) }"
+  else
+    local result=$(( $* ))
+    echo "$result"
+  fi
+}

src/parallel.sh

@@ -12,12 +12,16 @@ function parallel::aggregate_test_results() {
   local total_snapshot=0
 
   for script_dir in "$temp_dir_parallel_test_suite"/*; do
-    if ! compgen -G "$script_dir"/*.result > /dev/null; then
+    shopt -s nullglob
+    local result_files=("$script_dir"/*.result)
+    shopt -u nullglob
+
+    if [ ${#result_files[@]} -eq 0 ]; then
       printf "%sNo tests found%s" "$_COLOR_SKIPPED" "$_COLOR_DEFAULT"
       continue
     fi
 
-    for result_file in "$script_dir"/*.result; do
+    for result_file in "${result_files[@]}"; do
       local result_line
       result_line=$(tail -n 1 "$result_file")