Skip to content

Commit

Permalink
Merge pull request #2355 from SchoNie/fix-test_restart_while_missing_…
Browse files Browse the repository at this point in the history
…cert.py-xfails-xpass

Fix xpass and xfails test_restart_while_missing_cert.py
  • Loading branch information
buchdag committed Dec 23, 2023
2 parents 39db465 + 33ceec0 commit db27a07
Showing 1 changed file with 3 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@

script_dir = os.path.dirname(__file__)

pytestmark = pytest.mark.xfail() # TODO delete this marker once those issues are fixed

@pytest.fixture(scope="module", autouse=True)
def certs():
"""
Expand Down Expand Up @@ -45,28 +43,24 @@ def test_https_web_is_200(docker_compose, nginxproxy):
assert "answer from port 81\n" in r.text


@pytest.mark.incremental
def test_delete_cert_and_restart_reverseproxy(docker_compose):
os.remove(join(script_dir, "tmp_certs", "web.nginx-proxy.crt"))
docker_compose.containers.get("reverseproxy").restart()
sleep(3) # give time for the container to initialize
sleep(5) # give time for the container to initialize
assert "running" == docker_compose.containers.get("reverseproxy").status


@pytest.mark.incremental
def test_unknown_virtual_host_is_still_503(nginxproxy):
def test_unknown_virtual_host_is_still_503(docker_compose, nginxproxy):
r = nginxproxy.get("http://foo.nginx-proxy/")
assert r.status_code == 503


@pytest.mark.incremental
def test_http_web_is_now_200(nginxproxy):
def test_http_web_is_now_200(docker_compose, nginxproxy):
r = nginxproxy.get("http://web.nginx-proxy/port", allow_redirects=False)
assert r.status_code == 200
assert "answer from port 81\n" == r.text


@pytest.mark.incremental
def test_https_web_is_now_broken_since_there_is_no_cert(nginxproxy):
with pytest.raises(ConnectionError):
nginxproxy.get("https://web.nginx-proxy/port")

0 comments on commit db27a07

Please sign in to comment.