Fix setup_file marker logic
This commit is contained in:
parent
a20f2c4199
commit
3f9dd4180a
|
@ -34,13 +34,16 @@ function wait_for_finished_setup_in_container() {
|
||||||
repeat_until_success_or_timeout $TEST_TIMEOUT_IN_SECONDS sh -c "docker logs $1 | grep 'is up and running'"
|
repeat_until_success_or_timeout $TEST_TIMEOUT_IN_SECONDS sh -c "docker logs $1 | grep 'is up and running'"
|
||||||
}
|
}
|
||||||
|
|
||||||
SETUP_FILE_MARKER="$BATS_TMPDIR/`basename \"$BATS_TEST_FILENAME\".setup_file`"
|
SETUP_FILE_MARKER="$BATS_TMPDIR/`basename \"$BATS_TEST_FILENAME\"`.setup_file"
|
||||||
|
|
||||||
# use in setup() in conjunction with a `@test "first" {}` to trigger setup_file reliably
|
# use in setup() in conjunction with a `@test "first" {}` to trigger setup_file reliably
|
||||||
function run_setup_file_if_necessary() {
|
function run_setup_file_if_necessary() {
|
||||||
if [ "$BATS_TEST_NAME" == 'test_first' ]; then
|
if [ "$BATS_TEST_NAME" == 'test_first' ]; then
|
||||||
rm -f "$SETUP_FILE_MARKE"
|
# prevent old markers from marking success or get an error if we cannot remove due to permissions
|
||||||
|
rm -f "$SETUP_FILE_MARKER"
|
||||||
|
|
||||||
setup_file
|
setup_file
|
||||||
|
|
||||||
touch "$SETUP_FILE_MARKER"
|
touch "$SETUP_FILE_MARKER"
|
||||||
else
|
else
|
||||||
if [ ! -f "$SETUP_FILE_MARKER" ]; then
|
if [ ! -f "$SETUP_FILE_MARKER" ]; then
|
||||||
|
@ -53,7 +56,8 @@ function run_setup_file_if_necessary() {
|
||||||
# use in teardown() in conjunction with a `@test "last" {}` to trigger teardown_file reliably
|
# use in teardown() in conjunction with a `@test "last" {}` to trigger teardown_file reliably
|
||||||
function run_teardown_file_if_necessary() {
|
function run_teardown_file_if_necessary() {
|
||||||
if [ "$BATS_TEST_NAME" == 'test_last' ]; then
|
if [ "$BATS_TEST_NAME" == 'test_last' ]; then
|
||||||
rm -f "$SETUP_FILE_MARKE"
|
# cleanup setup file marker
|
||||||
|
rm -f "$SETUP_FILE_MARKER"
|
||||||
teardown_file
|
teardown_file
|
||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue