Merge pull request #495 from TypedDevs/fix/494-false-negative-from-test-doubles-with-pipe

Fix false negative from assert_have_been_called_with with pipes

Author: Chemaclass

CHANGELOG.md

@@ -12,6 +12,7 @@
     - tear_down
     - set_up_before_script
     - tear_down_after_script
+- Fix false negative from `assert_have_been_called_with` with pipes
 
 ## [0.24.0](https://github.com/TypedDevs/bashunit/compare/0.23.0...0.24.0) - 2025-09-14
 

src/test_doubles.sh

@@ -59,7 +59,7 @@ function spy() {
       serialized+=\"\$(printf '%q' \"\$arg\")$'\\x1f'\"
     done
     serialized=\${serialized%$'\\x1f'}
-    printf '%s|%s\\n' \"\$raw\" \"\$serialized\" >> '$params_file'
+    printf '%s\x1e%s\\n' \"\$raw\" \"\$serialized\" >> '$params_file'
     local _c=\$(cat '$times_file')
     _c=\$((_c+1))
     echo \"\$_c\" > '$times_file'
@@ -115,7 +115,7 @@ function assert_have_been_called_with() {
   fi
 
   local raw
-  IFS='|' read -r raw _ <<<"$line"
+  IFS=$'\x1e' read -r raw _ <<<"$line"
 
   if [[ "$expected" != "$raw" ]]; then
     state::add_assertions_failed

tests/unit/test_doubles_test.sh

@@ -166,3 +166,11 @@ function test_spy_unsuccessful_not_called() {
       "actual" "1 times")" \
     "$(assert_not_called ps)"
 }
+
+function test_spy_with_pipe_in_arguments() {
+  spy grep
+
+  grep -E 'foo|bar'
+
+  assert_have_been_called_with grep '-E foo|bar'
+}