Skip to content

Commit

Permalink
Separate integration coverage (#6113)
Browse files Browse the repository at this point in the history
* check coverage separately

* Add coverage minimums for integration tests.
  • Loading branch information
bmw committed Jun 15, 2018
1 parent 8b16a56 commit 3316eac
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions certbot-nginx/tests/boulder-integration.sh
Expand Up @@ -62,3 +62,5 @@ test_deployment_and_rollback nginx6.wtf
# note: not reached if anything above fails, hence "killall" at the
# top
nginx -c $nginx_root/nginx.conf -s stop

coverage report --fail-under 75 --include 'certbot-nginx/*' --show-missing
4 changes: 2 additions & 2 deletions tests/boulder-integration.sh
Expand Up @@ -486,11 +486,11 @@ if [ "${BOULDER_INTEGRATION:-v1}" = "v2" ]; then
--manual-cleanup-hook ./tests/manual-dns-cleanup.sh
fi

coverage report --fail-under 65 --include 'certbot/*' --show-missing

# Most CI systems set this variable to true.
# If the tests are running as part of CI, Nginx should be available.
if ${CI:-false} || type nginx;
then
. ./certbot-nginx/tests/boulder-integration.sh
fi

coverage report --fail-under 67 -m

0 comments on commit 3316eac

Please sign in to comment.