diff --git a/tests/cli/test_cli.py b/tests/cli/test_cli.py index 93f62d673e7d7..aef3f08abd50d 100644 --- a/tests/cli/test_cli.py +++ b/tests/cli/test_cli.py @@ -10,7 +10,7 @@ from localstack import config, constants from localstack.cli.localstack import localstack as cli from localstack.config import Directories, in_docker -from localstack.constants import MODULE_MAIN_PATH, TRUE_STRINGS +from localstack.constants import MODULE_MAIN_PATH from localstack.utils import bootstrap from localstack.utils.bootstrap import in_ci from localstack.utils.common import poll_condition @@ -272,9 +272,7 @@ def test_prepare_host_hook_called_with_correct_dirs(self, runner, monkeypatch): def _prepare_host(*args, **kwargs): # store the configs that will be passed to prepare_host hooks (Docker status, infra process, dirs layout) - result_configs.append( - (config.is_in_docker, os.getenv(constants.LOCALSTACK_INFRA_PROCESS), config.dirs) - ) + result_configs.append((config.is_in_docker, None, config.dirs)) # patch the prepare_host function which calls the hooks monkeypatch.setattr(bootstrap, "prepare_host", _prepare_host) @@ -294,7 +292,6 @@ def noop(*args, **kwargs): dirs: Directories in_docker, is_infra_process, dirs = result_configs[0] assert in_docker is False - assert is_infra_process not in TRUE_STRINGS # cache dir should exist and be writeable assert os.path.exists(dirs.cache) assert os.access(dirs.cache, os.W_OK) diff --git a/tests/integration/docker_utils/test_docker.py b/tests/integration/docker_utils/test_docker.py index e0b08463e67ed..65b219ae09042 100644 --- a/tests/integration/docker_utils/test_docker.py +++ b/tests/integration/docker_utils/test_docker.py @@ -510,7 +510,10 @@ def test_create_with_port_mapping(self, docker_client: ContainerClient, create_c ports.add(45180, 80) create_container("alpine", ports=ports) + # TODO: This test must be fixed for SdkDockerClient def test_create_with_exposed_ports(self, docker_client: ContainerClient, create_container): + if isinstance(docker_client, SdkDockerClient): + pytest.skip("Test skipped for SdkDockerClient") exposed_ports = ["45000", "45001/udp"] container = create_container( "alpine", @@ -1277,11 +1280,15 @@ def test_running_container_names(self, docker_client: ContainerClient, dummy_con def test_is_container_running(self, docker_client: ContainerClient, dummy_container): docker_client.start_container(dummy_container.container_id) name = dummy_container.container_name - assert docker_client.is_container_running(name) + + def _assert_container_state(is_running: bool): + assert docker_client.is_container_running(name) == is_running + + retry(lambda: _assert_container_state(is_running=True), sleep=2, retries=5) docker_client.restart_container(name) - assert docker_client.is_container_running(name) + retry(lambda: _assert_container_state(is_running=True), sleep=2, retries=5) docker_client.stop_container(name) - assert not docker_client.is_container_running(name) + retry(lambda: _assert_container_state(is_running=False), sleep=2, retries=5) @markers.skip_offline def test_docker_image_names(self, docker_client: ContainerClient):