Merge pull request #455 from TypedDevs/feat/add-line-number-to-failing-tests

Add line number to failing tests

Author: Chemaclass

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Fix broken core snapshot tests
 - Improve NixOS support
+- Add line number to failing tests
 
 ## [0.22.1](https://github.com/TypedDevs/bashunit/compare/0.22.0...0.22.1) - 2025-07-23
 

src/helpers.sh

@@ -279,3 +279,18 @@ function helper::load_bench_files() {
 
   printf "%s\n" "${bench_files[@]}"
 }
+
+#
+# @param $1 string function name
+# @return number line number of the function in the source file
+#
+function helper::get_function_line_number() {
+  local fn_name=$1
+
+  shopt -s extdebug
+  local line_number
+  line_number=$(declare -F "$fn_name" | awk '{print $2}')
+  shopt -u extdebug
+
+  echo "$line_number"
+}

src/runner.sh

@@ -295,15 +295,15 @@ function runner::run_test() {
     state::add_tests_failed
     console_results::print_error_test "$fn_name" "$runtime_error"
     reports::add_test_failed "$test_file" "$fn_name" "$duration" "$total_assertions"
-    runner::write_failure_result_output "$test_file" "$runtime_error"
+    runner::write_failure_result_output "$test_file" "$fn_name" "$runtime_error"
     internal_log "Test error" "$fn_name" "$runtime_error"
     return
   fi
 
   if [[ "$current_assertions_failed" != "$(state::get_assertions_failed)" ]]; then
     state::add_tests_failed
     reports::add_test_failed "$test_file" "$fn_name" "$duration" "$total_assertions"
-    runner::write_failure_result_output "$test_file" "$subshell_output"
+    runner::write_failure_result_output "$test_file" "$fn_name" "$subshell_output"
 
     internal_log "Test failed" "$fn_name"
 
@@ -466,14 +466,18 @@ function runner::parse_result_sync() {
 
 function runner::write_failure_result_output() {
   local test_file=$1
-  local error_msg=$2
+  local fn_name=$2
+  local error_msg=$3
+
+  local line_number
+  line_number=$(helper::get_function_line_number "$fn_name")
 
   local test_nr="*"
   if ! parallel::is_enabled; then
     test_nr=$(state::get_tests_failed)
   fi
 
-  echo -e "$test_nr) $test_file\n$error_msg" >> "$FAILURES_OUTPUT_PATH"
+  echo -e "$test_nr) $test_file:$line_number\n$error_msg" >> "$FAILURES_OUTPUT_PATH"
 }
 
 function runner::run_set_up() {

tests/acceptance/snapshots/bashunit_exit_code_test_sh.test_bashunit_when_a_test_returns_non_zero.snapshot

@@ -4,7 +4,7 @@
 
 There was 1 failure:
 
-|1) tests/acceptance/fixtures/test_bashunit_when_a_test_returns_non_zero.sh
+|1) tests/acceptance/fixtures/test_bashunit_when_a_test_returns_non_zero.sh:3
 
 Tests:      1 failed, 1 total
 Assertions: 0 failed, 0 total

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_when_a_test_fail_simple_output_env.snapshot

@@ -2,7 +2,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh:12
 |✗ Failed: Assert failing
 |    Expected '1'
 |    but got  '0'

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_when_a_test_fail_simple_output_option.snapshot

@@ -2,7 +2,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh:12
 |✗ Failed: Assert failing
 |    Expected '1'
 |    but got  '0'

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_when_a_test_fail_verbose_output_env.snapshot

@@ -9,7 +9,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh:12
 |✗ Failed: Assert failing
 |    Expected '1'
 |    but got  '0'

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_when_a_test_fail_verbose_output_option.snapshot

@@ -9,7 +9,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_when_a_test_fail.sh:12
 |✗ Failed: Assert failing
 |    Expected '1'
 |    but got  '0'

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_with_a_test_fail_and_exit_immediately.snapshot

@@ -4,7 +4,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_when_exit_immediately_after_execution_error.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_when_exit_immediately_after_execution_error.sh:3
 
 Tests:      1 failed, 1 total
 Assertions: 0 failed, 0 total

tests/acceptance/snapshots/bashunit_fail_test_sh.test_bashunit_with_multiple_failing_tests.snapshot

@@ -13,7 +13,7 @@
 
 There was 1 failure:
 
-|1) ./tests/acceptance/fixtures/test_bashunit_with_multiple_failing_tests.sh
+|1) ./tests/acceptance/fixtures/test_bashunit_with_multiple_failing_tests.sh:7
 |✗ Failed: Assert failing
 |    Expected '1'
 |    but got  '2'