Improve the output adding a space between each test file (#385)

Chemaclass · web-flow · commit af5beadfeca2 · 2024-11-09T13:25:31.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Improved output: adding a space between each test file
+
 ## [0.18.0](https://github.com/TypedDevs/bashunit/compare/0.17.0...0.18.0) - 2024-10-16
 
 - Added `-p|--parallel` to enable running tests in parallel
diff --git a/src/console_results.sh b/src/console_results.sh
@@ -12,7 +12,9 @@ function console_results::render_result() {
     return 1
   fi
 
-  echo ""
+  if env::is_simple_output_enabled; then
+    printf "\n\n"
+  fi
 
   local total_tests=0
   ((total_tests += $(state::get_tests_passed))) || true
@@ -262,7 +264,10 @@ function console_results::print_failing_tests_and_reset() {
     local total_failed
     total_failed=$(state::get_tests_failed)
 
-    echo ""
+    if env::is_simple_output_enabled; then
+      printf "\n\n"
+    fi
+
     if [[ "$total_failed" -eq 1 ]]; then
       echo -e "${_COLOR_BOLD}There was 1 failure:${_COLOR_DEFAULT}\n"
     else
@@ -271,5 +276,7 @@ function console_results::print_failing_tests_and_reset() {
 
     sed '${/^$/d;}' "$FAILURES_OUTPUT_PATH" | sed 's/^/|/'
     rm "$FAILURES_OUTPUT_PATH"
+
+    echo ""
   fi
 }
diff --git a/src/runner.sh b/src/runner.sh
@@ -105,6 +105,10 @@ function runner::call_test_functions() {
     done
     unset function_name
   done
+
+  if ! env::is_simple_output_enabled; then
+    echo ""
+  fi
 }
 
 function runner::render_running_file_header() {
diff --git a/tests/acceptance/snapshots/bashunit_find_tests_command_line_test_sh.test_all_tests_files_with_wildcard.snapshot b/tests/acceptance/snapshots/bashunit_find_tests_command_line_test_sh.test_all_tests_files_with_wildcard.snapshot
@@ -1,6 +1,7 @@
 [1mRunning ./tests/acceptance/fixtures/tests_path/a_test.sh[0m
 [32m✓ Passed[0m: Assert greater and less than
 [32m✓ Passed[0m: Assert empty
+
 [1mRunning ./tests/acceptance/fixtures/tests_path/other_test.sh[0m
 [32m✓ Passed[0m: Assert same
 [32m✓ Passed[0m: Assert contains
diff --git a/tests/acceptance/snapshots/bashunit_find_tests_command_line_test_sh.test_all_tests_files_within_a_directory.snapshot b/tests/acceptance/snapshots/bashunit_find_tests_command_line_test_sh.test_all_tests_files_within_a_directory.snapshot
@@ -1,6 +1,7 @@
 [1mRunning ./tests/acceptance/fixtures/tests_path/a_test.sh[0m
 [32m✓ Passed[0m: Assert greater and less than
 [32m✓ Passed[0m: Assert empty
+
 [1mRunning ./tests/acceptance/fixtures/tests_path/other_test.sh[0m
 [32m✓ Passed[0m: Assert same
 [32m✓ Passed[0m: Assert contains
diff --git a/tests/acceptance/snapshots/bashunit_pass_test_sh.test_bashunit_when_a_test_passes_simple_output_env.snapshot b/tests/acceptance/snapshots/bashunit_pass_test_sh.test_bashunit_when_a_test_passes_simple_output_env.snapshot
@@ -1,4 +1,5 @@
 ....
+
 [2mTests:     [0m [32m4 passed[0m, 4 total
 [2mAssertions:[0m [32m6 passed[0m, 6 total
 
diff --git a/tests/acceptance/snapshots/bashunit_pass_test_sh.test_bashunit_when_a_test_passes_simple_output_option.snapshot b/tests/acceptance/snapshots/bashunit_pass_test_sh.test_bashunit_when_a_test_passes_simple_output_option.snapshot
@@ -1,4 +1,5 @@
 ....
+
 [2mTests:     [0m [32m4 passed[0m, 4 total
 [2mAssertions:[0m [32m6 passed[0m, 6 total
 
diff --git a/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_argument_overloads_default_path.snapshot b/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_argument_overloads_default_path.snapshot
@@ -1,4 +1,3 @@
-
 [2mTests:     [0m 0 total
 [2mAssertions:[0m 0 total
 
diff --git a/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_with_argument_path.snapshot b/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_with_argument_path.snapshot
@@ -1,6 +1,7 @@
 [1mRunning tests/acceptance/fixtures/tests_path/a_test.sh[0m
 [32m✓ Passed[0m: Assert greater and less than
 [32m✓ Passed[0m: Assert empty
+
 [1mRunning tests/acceptance/fixtures/tests_path/other_test.sh[0m
 [32m✓ Passed[0m: Assert same
 [32m✓ Passed[0m: Assert contains
diff --git a/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_with_env_default_path.snapshot b/tests/acceptance/snapshots/bashunit_path_test_sh.test_bashunit_with_env_default_path.snapshot
@@ -1,6 +1,7 @@
 [1mRunning tests/acceptance/fixtures/tests_path/a_test.sh[0m
 [32m✓ Passed[0m: Assert greater and less than
 [32m✓ Passed[0m: Assert empty
+
 [1mRunning tests/acceptance/fixtures/tests_path/other_test.sh[0m
 [32m✓ Passed[0m: Assert same
 [32m✓ Passed[0m: Assert contains
diff --git a/tests/acceptance/snapshots/bashunit_stop_on_failure_test_sh.test_bashunit_when_stop_on_failure_env.snapshot b/tests/acceptance/snapshots/bashunit_stop_on_failure_test_sh.test_bashunit_when_stop_on_failure_env.snapshot
@@ -5,7 +5,6 @@
     [2mbut got [0m [1m'2'[0m
 
 [33mStop on failure enabled...[0m
-
 [1mThere was 1 failure:[0m
 
 |1) ./tests/acceptance/fixtures/test_bashunit_when_stop_on_failure.sh
diff --git a/tests/acceptance/snapshots/bashunit_stop_on_failure_test_sh.test_bashunit_when_stop_on_failure_option.snapshot b/tests/acceptance/snapshots/bashunit_stop_on_failure_test_sh.test_bashunit_when_stop_on_failure_option.snapshot
@@ -5,7 +5,6 @@
     [2mbut got [0m [1m'2'[0m
 
 [33mStop on failure enabled...[0m
-
 [1mThere was 1 failure:[0m
 
 |1) ./tests/acceptance/fixtures/test_bashunit_when_stop_on_failure.sh

-Original file line number
+Diff line change
@@ @@ -1,6 +1,7 @@ @@
 [1mRunning ./tests/acceptance/fixtures/tests_path/a_test.sh[0m
 [32m✓ Passed[0m: Assert greater and less than
 [32m✓ Passed[0m: Assert empty
++
 [1mRunning ./tests/acceptance/fixtures/tests_path/other_test.sh[0m
 [32m✓ Passed[0m: Assert same
 [32m✓ Passed[0m: Assert contains
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`....`
	`2`	`+`
`2`	`3`	`[2mTests: [0m [32m4 passed[0m, 4 total`
`3`	`4`	`[2mAssertions:[0m [32m6 passed[0m, 6 total`
`4`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-`
`2`	`1`	`[2mTests: [0m 0 total`
`3`	`2`	`[2mAssertions:[0m 0 total`
`4`	`3`