Skip to content

Commit 3894eb1

Browse files
fix: check if self._kwargs is None before using it
1 parent 4c0ba18 commit 3894eb1

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

core/testcontainers/core/container.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ def start(self) -> Self:
216216
self.ports,
217217
self._name,
218218
self.volumes,
219-
str(tuple(sorted(self._kwargs.values()))),
219+
str(tuple(sorted(self._kwargs.values()))) if self._kwargs else None,
220220
]
221221
hash_ = str(zlib.crc32(pickle.dumps(args)))
222222
docker_client = self.get_docker_client()

core/tests/test_reusable_containers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ def test_docker_container_labels_hash_default():
114114
def test_docker_container_labels_hash(monkeypatch):
115115
tc_properties_mock = testcontainers_config.tc_properties | {"testcontainers.reuse.enable": "true"}
116116
monkeypatch.setattr(testcontainers_config, "tc_properties", tc_properties_mock)
117-
expected_hash = "1296172062"
117+
expected_hash = "380705419"
118118
with DockerContainer("hello-world").with_reuse() as container:
119119
assert container._container is not None
120120
assert container._container.labels["hash"] == expected_hash

0 commit comments

Comments
 (0)