Merge pull request #546 from thockin/v3-dont-docker-kill
v3: e2e: Don't manually "docker kill"
This commit is contained in:
commit
43fde5ff1d
11
test_e2e.sh
11
test_e2e.sh
|
|
@ -941,11 +941,11 @@ function e2e::askpass_url() {
|
||||||
--root="$ROOT" \
|
--root="$ROOT" \
|
||||||
--dest="link" \
|
--dest="link" \
|
||||||
>> "$1" 2>&1 || true
|
>> "$1" 2>&1 || true
|
||||||
docker_kill "$CTR"
|
|
||||||
# check for failure
|
# check for failure
|
||||||
assert_file_absent "$ROOT"/link/file
|
assert_file_absent "$ROOT"/link/file
|
||||||
|
|
||||||
# run with askpass_url service with correct password
|
# run with askpass_url service with correct password
|
||||||
|
docker_kill "$CTR"
|
||||||
CTR=$(docker_run \
|
CTR=$(docker_run \
|
||||||
e2e/test/test-ncsvr \
|
e2e/test/test-ncsvr \
|
||||||
80 '
|
80 '
|
||||||
|
|
@ -966,7 +966,6 @@ function e2e::askpass_url() {
|
||||||
--root="$ROOT" \
|
--root="$ROOT" \
|
||||||
--dest="link" \
|
--dest="link" \
|
||||||
>> "$1" 2>&1
|
>> "$1" 2>&1
|
||||||
docker_kill "$CTR"
|
|
||||||
assert_link_exists "$ROOT"/link
|
assert_link_exists "$ROOT"/link
|
||||||
assert_file_exists "$ROOT"/link/file
|
assert_file_exists "$ROOT"/link/file
|
||||||
assert_file_eq "$ROOT"/link/file "$FUNCNAME 1"
|
assert_file_eq "$ROOT"/link/file "$FUNCNAME 1"
|
||||||
|
|
@ -1140,7 +1139,6 @@ function e2e::webhook_success() {
|
||||||
if [[ "$HITS" < 1 ]]; then
|
if [[ "$HITS" < 1 ]]; then
|
||||||
fail "webhook 2 called $HITS times"
|
fail "webhook 2 called $HITS times"
|
||||||
fi
|
fi
|
||||||
docker_kill "$CTR"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
##############################################
|
##############################################
|
||||||
|
|
@ -1175,9 +1173,9 @@ function e2e::webhook_fail_retry() {
|
||||||
if [[ "$HITS" < 1 ]]; then
|
if [[ "$HITS" < 1 ]]; then
|
||||||
fail "webhook 1 called $HITS times"
|
fail "webhook 1 called $HITS times"
|
||||||
fi
|
fi
|
||||||
docker_kill "$CTR"
|
|
||||||
|
|
||||||
# Now return 200, ensure that it gets called
|
# Now return 200, ensure that it gets called
|
||||||
|
docker_kill "$CTR"
|
||||||
cat /dev/null > "$HITLOG"
|
cat /dev/null > "$HITLOG"
|
||||||
CTR=$(docker_run \
|
CTR=$(docker_run \
|
||||||
--ip="$IP" \
|
--ip="$IP" \
|
||||||
|
|
@ -1189,7 +1187,6 @@ function e2e::webhook_fail_retry() {
|
||||||
if [[ "$HITS" < 1 ]]; then
|
if [[ "$HITS" < 1 ]]; then
|
||||||
fail "webhook 2 called $HITS times"
|
fail "webhook 2 called $HITS times"
|
||||||
fi
|
fi
|
||||||
docker_kill "$CTR"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
##############################################
|
##############################################
|
||||||
|
|
@ -1225,8 +1222,6 @@ function e2e::webhook_success_once() {
|
||||||
if [[ "$HITS" != 1 ]]; then
|
if [[ "$HITS" != 1 ]]; then
|
||||||
fail "webhook called $HITS times"
|
fail "webhook called $HITS times"
|
||||||
fi
|
fi
|
||||||
|
|
||||||
docker_kill "$CTR"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
##############################################
|
##############################################
|
||||||
|
|
@ -1262,7 +1257,6 @@ function e2e::webhook_fail_retry_once() {
|
||||||
if [[ "$HITS" != 1 ]]; then
|
if [[ "$HITS" != 1 ]]; then
|
||||||
fail "webhook called $HITS times"
|
fail "webhook called $HITS times"
|
||||||
fi
|
fi
|
||||||
docker_kill "$CTR"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
##############################################
|
##############################################
|
||||||
|
|
@ -1299,7 +1293,6 @@ function e2e::webhook_fire_and_forget() {
|
||||||
if [[ "$HITS" < 1 ]]; then
|
if [[ "$HITS" < 1 ]]; then
|
||||||
fail "webhook called $HITS times"
|
fail "webhook called $HITS times"
|
||||||
fi
|
fi
|
||||||
docker_kill "$CTR"
|
|
||||||
}
|
}
|
||||||
|
|
||||||
##############################################
|
##############################################
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue