Skip to content

Commit 1f5bdbf

Browse files
committed
chore(test): add missing docker=True to TestFileTransferLimitsEnterprise fixture
The TestFileTransferLimitsEnterprise fixture was missing docker=True and ignore_existing=True in the prepare_env_for_connect() call. This was dropped in b5a8c23 (QA-1527) when migrating file transfer tests to the mender docker client. Without these flags, prepare_env_for_connect spawns a non-docker client in a docker-client environment, causing every file transfer limits test in the Enterprise class to time out with HTTP 408 instead of receiving the expected 403 from the device. The fix aligns the Limits Enterprise fixture with the Download Enterprise fixture, which already passes docker=True, ignore_existing=True. Signed-off-by: pasinskim <marcin.pasinski@northern.tech>
1 parent 23fae41 commit 1f5bdbf

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

tests/tests/test_filetransfer.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -771,7 +771,11 @@ class TestFileTransferLimitsEnterprise(BaseTestFileTransferLimits):
771771
@pytest.fixture(scope="function")
772772
def mender_device_setup(self, request, enterprise_one_docker_client_bootstrapped):
773773
env = enterprise_one_docker_client_bootstrapped
774-
devid, auth_token, auth, mender_device = prepare_env_for_connect(env)
774+
devid, auth_token, auth, mender_device = prepare_env_for_connect(
775+
env,
776+
docker=True,
777+
ignore_existing=True,
778+
)
775779
request.cls.devid = devid
776780
request.cls.auth_token = auth_token
777781
request.cls.auth = auth

0 commit comments

Comments
 (0)