Merge pull request #437 from TypedDevs/fix/436-clock-time

Fix process time always shows as '0 ms'

Author: Chemaclass

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Fix process time always shows as 0 ms
+
 ## [0.21.0](https://github.com/TypedDevs/bashunit/compare/0.20.0...0.21.0) - 2025-06-18
 
 - Fix typo "to has been called"

src/clock.sh

@@ -4,22 +4,13 @@ function clock::now() {
   local shell_time
   local attempts=()
 
-  # 1. Try using native shell EPOCHREALTIME (if available)
-  attempts+=("EPOCHREALTIME")
-  if shell_time="$(clock::shell_time)"; then
-    local seconds="${shell_time%%.*}"
-    local microseconds="${shell_time#*.}"
-    math::calculate "($seconds * 1000000000) + ($microseconds * 1000)"
-    return 0
-  fi
-
-  # 2. Try Perl with Time::HiRes
+  # 1. Try Perl with Time::HiRes
   attempts+=("Perl")
   if dependencies::has_perl && perl -MTime::HiRes -e "" &>/dev/null; then
     perl -MTime::HiRes -e 'printf("%.0f\n", Time::HiRes::time() * 1000000000)' && return 0
   fi
 
-  # 3. Try Python 3 with time module
+  # 2. Try Python 3 with time module
   attempts+=("Python")
   if dependencies::has_python; then
     python - <<'EOF'
@@ -29,12 +20,12 @@ EOF
     return 0
   fi
 
-  # 4. Try Node.js
+  # 3. Try Node.js
   attempts+=("Node")
   if dependencies::has_node; then
     node -e 'process.stdout.write((BigInt(Date.now()) * 1000000n).toString())' && return 0
   fi
-  # 5. Windows fallback with PowerShell
+  # 4. Windows fallback with PowerShell
   attempts+=("PowerShell")
   if check_os::is_windows && dependencies::has_powershell; then
     powershell -Command "
@@ -46,7 +37,7 @@ EOF
     " && return 0
   fi
 
-  # 6. Unix fallback using `date +%s%N` (if not macOS or Alpine)
+  # 5. Unix fallback using `date +%s%N` (if not macOS or Alpine)
   attempts+=("date")
   if ! check_os::is_macos && ! check_os::is_alpine; then
     local result
@@ -57,6 +48,15 @@ EOF
     fi
   fi
 
+  # 6. Try using native shell EPOCHREALTIME (if available)
+  attempts+=("EPOCHREALTIME")
+  if shell_time="$(clock::shell_time)"; then
+    local seconds="${shell_time%%.*}"
+    local microseconds="${shell_time#*.}"
+    math::calculate "($seconds * 1000000000) + ($microseconds * 1000)"
+    return 0
+  fi
+
   # 7. All methods failed
   printf "clock::now implementations tried: %s\n" "${attempts[*]}" >&2
   echo ""

tests/unit/clock_test.sh

@@ -91,6 +91,25 @@ function test_runtime_in_milliseconds_when_not_empty_time() {
   assert_not_empty "$(clock::total_runtime_in_milliseconds)"
 }
 
+function test_now_prefers_perl_over_shell_time() {
+  mock clock::shell_time echo "1234.0"
+  mock perl echo "999999999999"
+  mock dependencies::has_python mock_false
+  mock dependencies::has_node mock_false
+
+  assert_same "999999999999" "$(clock::now)"
+}
+
+function test_now_prefers_python_over_node() {
+  mock perl mock_non_existing_fn
+  mock dependencies::has_python mock_true
+  mock python echo "777777777777"
+  mock dependencies::has_node mock_true
+  mock node echo "888888888888"
+
+  assert_same "777777777777" "$(clock::now)"
+}
+
 function test_runtime_in_milliseconds_when_empty_time() {
   mock_macos
   mock perl mock_non_existing_fn