mirror of
https://github.com/certbot/certbot.git
synced 2026-06-06 15:22:38 -04:00
95557fa9b4
Fixes #6585. I wrote up three suggestions for fixing this at https://github.com/certbot/certbot/issues/6585#issuecomment-448054502. I took the middle approach of requiring the user to provide an ACME server to use. I like this better than the other approaches which were: > Resolve #5938 instead of this issue. There is value in these tests as is over the compatibility tests in that they don't use Docker and run on different OSes. > Spin up a local Python server to return the directory object. Trying to set up a dummy ACME server seemed hacky and error prone. Other notes about this PR are: * I put the Pebble setup in `tox.ini` rather than `.travis.yml` as this seems much cleaner and more natural. * I created a new `tox` environment called `apacheconftest-with-pebble` that reuses the code from `testenv:apacheconftest` so `apacheconftest` can continue to be used with servers other than Pebble like is done in our test farm tests. * I chose the environment variable `SERVER` for consistency with our integration tests. I chose to not give this environment variable a default but to fail fast when it is not set. * I ran test farm tests on this PR and they passed. |
||
|---|---|---|
| .. | ||
| boulder_config.sh | ||
| boulder_install.sh | ||
| test_apache2.sh | ||
| test_leauto_upgrades.sh | ||
| test_letsencrypt_auto_certonly_standalone.sh | ||
| test_letsencrypt_auto_venv_only.sh | ||
| test_renew_standalone.sh | ||
| test_sdists.sh | ||
| test_tests.sh | ||
| test_tox.sh | ||