github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge pull request #3947 from jayjun/tests

Browse Source 
fix: scope init tests to container processes only
This commit is contained in:
Jose Diaz-Gonzalez
2020-04-22 14:53:59 -04:00
committed by GitHub
parent 9b48a386fc 14934a9479
commit 9e9aa75fbb
3 changed files with 18 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
tests/unit/50_init_1.bats
View File

@@ -13,8 +13,12 @@ teardown() {
} }
@test "(init) buildpack" { @test "(init) buildpack" {
deploy_app zombies-buildpack source "$PLUGIN_CORE_AVAILABLE_PATH/common/functions"
run ps ax local APP="zombies-buildpack"
deploy_app "$APP"
local CIDS=$(get_app_container_ids "$APP")
run "$DOCKER_BIN" container top "$CIDS"
echo "output: $output" echo "output: $output"
assert_output_contains "<defunct>" "0" assert_output_contains "<defunct>" "0"
} }

8
tests/unit/50_init_2.bats
View File

@@ -13,8 +13,12 @@ teardown() {
} }
@test "(init) dockerfile no tini" { @test "(init) dockerfile no tini" {
deploy_app zombies-dockerfile-no-tini source "$PLUGIN_CORE_AVAILABLE_PATH/common/functions"
run ps auxf local APP="zombies-dockerfile-no-tini"
deploy_app "$APP"
local CIDS=$(get_app_container_ids "$APP")
run "$DOCKER_BIN" container top "$CIDS"
echo "output: $output" echo "output: $output"
assert_output_contains "<defunct>" "0" assert_output_contains "<defunct>" "0"
} }

8
tests/unit/50_init_3.bats
View File

@@ -13,8 +13,12 @@ teardown() {
} }
@test "(init) dockerfile with tini" { @test "(init) dockerfile with tini" {
deploy_app zombies-dockerfile-tini source "$PLUGIN_CORE_AVAILABLE_PATH/common/functions"
run ps ax local APP="zombies-dockerfile-tini"
deploy_app "$APP"
local CIDS=$(get_app_container_ids "$APP")
run "$DOCKER_BIN" container top "$CIDS"
echo "output: $output" echo "output: $output"
assert_output_contains "<defunct>" "0" assert_output_contains "<defunct>" "0"
} }