Skip to content

Commit

Permalink
Replace curl by requests
Browse files Browse the repository at this point in the history
Replace the curl utility by the python requests library to avoid another
host-dependency.
  • Loading branch information
grisu48 committed Aug 17, 2023
1 parent 06fb816 commit d4bed9e
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
14 changes: 10 additions & 4 deletions tests/test_nginx.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
"""This module contains the tests for the nginx container, the image with nginx pre-installed.
"""
from bci_tester.data import NGINX_CONTAINER

import requests
import time

CONTAINER_IMAGES = (NGINX_CONTAINER,)

Expand All @@ -10,6 +11,11 @@ def test_nginx_welcome_page(auto_container, host):
"""test that the default welcome page is served by the container."""
host_port = auto_container.forwarded_ports[0].host_port

assert "Welcome to nginx" in host.check_output(
f"curl -sf --retry 5 --retry-connrefused http://localhost:{host_port}/"
)
# Retry 5 times with quadratic backoff delay
retries = 5
for i in range(retries):
resp = requests.get(f"http://localhost:{host_port}/", timeout=30)
if resp.status_code == 200 and "Welcome to nginx" in resp.text:
return
time.Sleep(i * i)
assert False, "invalid nginx response"
1 change: 1 addition & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ deps =
dataclasses ; python_version < "3.7"
pytest-rerunfailures
typing_extensions
requests
git+https://github.com/dcermak/pytest_container
doc: Sphinx

Expand Down

0 comments on commit d4bed9e

Please sign in to comment.