feat: add bats::on_failure hook

Contrary to setup/teardown this has the bats:: prefix to avoid name collisions due to t he generic name.

In the long run, the other functions should be named bats::setup/bats::teardown/bats::... as well.

lib/bats-core/tracing.bash

@@ -386,8 +386,14 @@ bats_setup_tracing() {
   trap 'bats_error_trap' ERR
 }
 
+# predefine to avoid problems when the user does not declare one
+bats::on_failure() {
+  :
+}
+
 bats_error_trap() {
   bats_check_status_from_trap
+  bats::on_failure "$BATS_ERROR_STATUS"
 
   # If necessary, undo the most recent stack trace captured by bats_debug_trap.
   # See bats_debug_trap for details.

test/bats.bats

@@ -1500,4 +1500,50 @@ END_OF_ERR_MSG
   bats_require_minimum_version 1.5.0
 
   reentrant_run -0 bats --print-output-on-failure "$FIXTURE_ROOT/preserve_IFS" --filter-tags ''
+}
+
+@test "failure callback in test" {
+  bats_require_minimum_version 1.5.0
+
+  reentrant_run -1 bats --show-output-of-passing-tests --print-output-on-failure "$FIXTURE_ROOT/failure_callback.bats"
+
+  [ "${lines[0]}" = 1..3 ]
+  [ "${lines[1]}" = 'not ok 1 failure callback is called on failure' ]
+  [ "${lines[2]}" = "# (in test file $RELATIVE_FIXTURE_ROOT/failure_callback.bats, line 6)" ]
+  [ "${lines[3]}" = "#   \`false' failed" ]
+  [ "${lines[4]}" = "# called failure callback" ]
+  [ "${lines[5]}" = 'ok 2 failure callback is not called on success' ]
+  [ "${lines[6]}" = '# passed' ]
+  # this test should not contain: called failure callback
+  [ "${lines[7]}" = 'not ok 3 failure callback can be overriden locally' ]
+  [ "${lines[8]}" = "# (in test file $RELATIVE_FIXTURE_ROOT/failure_callback.bats, line 17)" ]
+  [ "${lines[9]}" = "#   \`false' failed" ]
+  [ "${lines[10]}" = "# override failure callback" ]
+  [ ${#lines[@]} -eq 11 ]
+}
+
+@test "failure callback in setup_file" {
+  bats_require_minimum_version 1.5.0
+
+  reentrant_run -1 bats --print-output-on-failure "$FIXTURE_ROOT/failure_callback_setup_file.bats"
+
+  [ "${lines[0]}" = 1..1 ]
+  [ "${lines[1]}" = 'not ok 1 setup_file failed' ]
+  [ "${lines[2]}" = "# (from function \`setup_file' in test file $RELATIVE_FIXTURE_ROOT/failure_callback_setup_file.bats, line 6)" ]
+  [ "${lines[3]}" = "#   \`false' failed" ] 
+  [ "${lines[4]}" = '# failure callback' ]
+  [ ${#lines[@]} -eq 5 ]
+}
+
+@test "failure callback in setup_suite" {
+  bats_require_minimum_version 1.5.0
+
+  reentrant_run -1 bats --print-output-on-failure "$FIXTURE_ROOT/failure_callback_setup_suite"
+
+  [ "${lines[0]}" = 1..1 ]
+  [ "${lines[1]}" = 'not ok 1 setup_suite' ]
+  [ "${lines[2]}" = "# (from function \`setup_suite' in test file $RELATIVE_FIXTURE_ROOT/failure_callback_setup_suite/setup_suite.bash, line 6)" ]
+  [ "${lines[3]}" = "#   \`false' failed" ]
+  [ "${lines[4]}" = '# failure callback' ]
+  [ ${#lines[@]} -eq 5 ]
 }

test/fixtures/bats/failure_callback.bats

@@ -0,0 +1,18 @@
+bats::on_failure() {
+    echo "called failure callback"
+}
+
+@test failure callback is called on failure {
+    false
+}
+
+@test failure callback is not called on success {
+    echo passed
+}
+
+@test failure callback can be overriden locally {
+    bats::on_failure() {
+        echo "override failure callback"
+    }
+    false
+}

test/fixtures/bats/failure_callback_setup_file.bats

@@ -0,0 +1,11 @@
+bats::on_failure() {
+    echo "failure callback"
+}
+
+setup_file() {
+    false
+}
+
+@test dummy {
+    true
+}

test/fixtures/bats/failure_callback_setup_suite/dummy.bats

@@ -0,0 +1,3 @@
+@test dummy {
+    true
+}

test/fixtures/bats/failure_callback_setup_suite/setup_suite.bash

@@ -0,0 +1,7 @@
+bats::on_failure() {
+    echo "failure callback"
+}
+
+setup_suite() {
+    false
+}