Change working directory before loading test file

bash_unit changes the current working directory to the
one of the code under test just before running the tests.

But it was not changing the current working directory
while loading the test file. Hence, if one had static
code in its test file, this code was run in an unknown
working directory.

Know bash_unit changes the current working directory
before loading the test file.

Author: pgrange

README.adoc

@@ -86,7 +86,6 @@ Running test_assert_shows_stdout_on_failure... SUCCESS
 Running test_assert_status_code_fails... SUCCESS
 Running test_assert_status_code_succeeds... SUCCESS
 Running test_assert_succeeds... SUCCESS
-Running test_bash_unit_changes_cwd_to_current_test_file_directory... SUCCESS
 Running test_fail_fails... SUCCESS
 Running test_fail_prints_failure_message... SUCCESS
 Running test_fail_prints_where_is_error... SUCCESS
@@ -143,7 +142,6 @@ ok - test_assert_shows_stdout_on_failure
 ok - test_assert_status_code_fails
 ok - test_assert_status_code_succeeds
 ok - test_assert_succeeds
-ok - test_bash_unit_changes_cwd_to_current_test_file_directory
 ok - test_fail_fails
 ok - test_fail_prints_failure_message
 ok - test_fail_prints_where_is_error

bash_unit

@@ -348,8 +348,13 @@ for test_file in "$@"
 do
   notify_suite_starting "$test_file"
   (
-    source "$test_file"
-    cd "$(dirname "$test_file")"
+    if [[ "${STICK_TO_CWD}" != true ]]
+    then
+      cd "$(dirname "$test_file")"
+      source "$(basename "$test_file")"
+    else
+      source "$test_file"
+    fi
     run_test_suite
   )
   failure=$(( $? || failure))

tests/test_cli.sh

@@ -1,17 +1,18 @@
 #!/bin/bash
 
 test_run_all_tests_even_in_case_of_failure() {
-  bash_unit_output=$($BASH_UNIT <(cat << EOF
-function test_succeed() { assert true ; }
-function test_fails()   { assert false ; }
-EOF
-) | sed -e 's:/dev/fd/[0-9]*:test_file:')
-
-  assert_equals "\
-Running tests in test_file
+  assert_equals \
+"\
+Running tests in code
 Running test_fails... FAILURE
-test_file:2:test_fails()
-Running test_succeed... SUCCESS" "$bash_unit_output" 
+code:2:test_fails()
+Running test_succeed... SUCCESS\
+" \
+"$(bash_unit_out_for_code << EOF
+  function test_succeed() { assert true ; }
+  function test_fails()   { assert false ; }
+EOF
+)"
 }
 
 test_exit_code_not_0_in_case_of_failure() {
@@ -29,11 +30,14 @@ test_run_all_file_parameters() {
     | sed -e 's:/dev/fd/[0-9]*:test_file:' \
   )
 
-  assert_equals "\
+  assert_equals \
+"\
 Running tests in test_file
 Running test_one... SUCCESS
 Running tests in test_file
-Running test_two... SUCCESS" "$bash_unit_output" 
+Running test_two... SUCCESS\
+" \
+"$bash_unit_output"
 }
 
 test_run_only_tests_that_match_pattern() {
@@ -109,4 +113,8 @@ line() {
   tail -n +$line_nb | head -1
 }
 
+bash_unit_out_for_code() {
+  $BASH_UNIT <(cat) | sed -e 's:/dev/fd/[0-9]*:code:' -e 's/[0-9]*:/code:/'
+}
+
 BASH_UNIT="eval FORCE_COLOR=false ../bash_unit"

tests/test_core.sh

@@ -138,10 +138,19 @@ EOF
   assert_equals 2 $(ps | grep pts | wc -l)
 }
 
-test_bash_unit_changes_cwd_to_current_test_file_directory() {
+if [[ "${STICK_TO_CWD}" != true ]]
+then
+  # do not test for cwd if STICK_TO_CWD is true
+  test_bash_unit_changes_cwd_to_current_test_file_directory() {
+    assert "ls ../tests/$(basename "$BASH_SOURCE")" \
+      "bash_unit should change current working directory to match the directory of the currenlty running test"
+  }
+
+  #the following assertion is out of any test on purpose
   assert "ls ../tests/$(basename "$BASH_SOURCE")" \
-    "bash_unit should change current working directory to match the directory of the currenlty running test"
-}
+  "bash_unit should change current working directory to match the directory of the currenlty running test before sourcing test file"
+fi
+
 
 line() {
   line_nb=$1

tests/test_doc.sh

@@ -4,7 +4,10 @@ TEST_PATTERN='```bash|```test'
 OUTPUT_PATTERN='```output'
 LANG=C.UTF-8
 
-BASH_UNIT="eval FORCE_COLOR=false ./bash_unit"
+export FORCE_COLOR=false
+export STICK_TO_CWD=true
+BASH_UNIT="eval ./bash_unit"
+#BASH_UNIT="eval FORCE_COLOR=false ./bash_unit"
 
 prepare_tests() {
   mkdir /tmp/$$
@@ -62,5 +65,7 @@ function _next_quote_section() {
   '
 }
 
-cd "$(dirname "$0")"
+# change to bash_unit source directory since we should be in
+# test subdirectory
+cd ..
 prepare_tests

tests/test_tap_format

@@ -102,7 +102,7 @@ EOF
 }
 
 bash_unit_out_for_code() {
-  $BASH_UNIT -f tap <(cat) | sed -e 's:/dev/fd/[0-9]*:code:g'
+  $BASH_UNIT -f tap <(cat) | sed -e 's:/dev/fd/[0-9]*:code:' -e 's/[0-9]*:/code:/'
 }
 
 BASH_UNIT="eval FORCE_COLOR=false ../bash_unit"