diff options
| author | Joona Hoikkala <joohoi@users.noreply.github.com> | 2019-03-27 20:10:52 +0300 |
|---|---|---|
| committer | Brad Warren <bmw@users.noreply.github.com> | 2019-03-27 20:10:52 +0300 |
| commit | b30a5e5b738f812c23937ea63eedb6443f7b98fd (patch) | |
| tree | 785a6a25b45ce5e2917770676922c2467cf9ecfe /tox.cover.py | |
| parent | 491d6c8f4538c28f7c05d780db9a700982460242 (diff) | |
Add a test to ensure test coverage regardless of the vhost order (#6873)
Add a new test to make sure that we are covering all the branches of get_virtual_hosts() regardless of the order that Augeas returns the found VirtualHost paths.
Fixes: #6813
* Add a test to ensure test coverage regardless of the order of returned vhosts
* Use deepcopy instead, and increase coverage requirement back to 100%
Diffstat (limited to 'tox.cover.py')
| -rwxr-xr-x | tox.cover.py | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/tox.cover.py b/tox.cover.py index 6f5392b71..d0f97626a 100755 --- a/tox.cover.py +++ b/tox.cover.py @@ -14,10 +14,7 @@ DEFAULT_PACKAGES = [ COVER_THRESHOLDS = { 'certbot': {'linux': 98, 'windows': 93}, 'acme': {'linux': 100, 'windows': 99}, - # certbot_apache coverage not being at 100% is a workaround for - # https://github.com/certbot/certbot/issues/6813. We should increase - # the minimum coverage back to 100% when this issue is resolved. - 'certbot_apache': {'linux': 99, 'windows': 99}, + 'certbot_apache': {'linux': 100, 'windows': 100}, 'certbot_dns_cloudflare': {'linux': 98, 'windows': 98}, 'certbot_dns_cloudxns': {'linux': 99, 'windows': 99}, 'certbot_dns_digitalocean': {'linux': 98, 'windows': 98}, |