kubernetes
/
git-sync
mirror of https://github.com/kubernetes/git-sync.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

e2e: Don't manually "docker kill" 

It happens automatically on test cleanup
This commit is contained in:
Tim Hockin 2022-06-13 09:24:13 -07:00
parent 5e35650bc7
commit 5f6b04eae2
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test_e2e.sh
View File

@ -928,11 +928,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 'echo -e "HTTP/1.1 200 OK\r\n\r\nusername=my-username\npassword=my-password"') 80 'echo -e "HTTP/1.1 200 OK\r\n\r\nusername=my-username\npassword=my-password"')
@ -948,7 +948,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"
@ -1122,7 +1121,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"
} }
############################################## ##############################################
@ -1157,9 +1155,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" \
@ -1171,7 +1169,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"
} }
############################################## ##############################################
@ -1207,8 +1204,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"
} }
############################################## ##############################################
@ -1244,7 +1239,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"
} }
############################################## ##############################################
@ -1281,7 +1275,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"
} }
############################################## ##############################################