fix(core): multiple container start invocations with custom labels (testcontainers#769)

When invoking `.start()` multiple times on the same `DockerContainer`
instance, the call fails with `ValueError: The org.testcontainers
namespace is reserved for internal use` error.

Example code:

```
from testcontainers.core.container import DockerContainer
container = DockerContainer("alpine:latest").with_kwargs(labels={})
container.start()
container.stop()
container.start()
```

The fix is to update labels for the container in a copy of the
user-provided dictionary, so that:
* the code doesn't mutate user structures
* avoid side effects, allowing for multiple .start() invocations

Author: neykov

core/testcontainers/core/labels.py

@@ -21,12 +21,15 @@ def create_labels(image: str, labels: Optional[dict[str, str]]) -> dict[str, str
             if k.startswith(TESTCONTAINERS_NAMESPACE):
                 raise ValueError("The org.testcontainers namespace is reserved for internal use")
 
-    labels[LABEL_LANG] = "python"
-    labels[LABEL_TESTCONTAINERS] = "true"
-    labels[LABEL_VERSION] = importlib.metadata.version("testcontainers")
+    tc_labels = {
+        **labels,
+        LABEL_LANG: "python",
+        LABEL_TESTCONTAINERS: "true",
+        LABEL_VERSION: importlib.metadata.version("testcontainers"),
+    }
 
     if image == c.ryuk_image:
-        return labels
+        return tc_labels
 
-    labels[LABEL_SESSION_ID] = SESSION_ID
-    return labels
+    tc_labels[LABEL_SESSION_ID] = SESSION_ID
+    return tc_labels

core/tests/test_labels.py

@@ -56,3 +56,10 @@ def test_session_are_module_import_scoped():
     assert LABEL_SESSION_ID in first_labels
     assert LABEL_SESSION_ID in second_labels
     assert first_labels[LABEL_SESSION_ID] == second_labels[LABEL_SESSION_ID]
+
+
+def test_create_no_side_effects():
+    input_labels = {"key": "value"}
+    expected_labels = input_labels.copy()
+    create_labels("not-ryuk", {"key": "value"})
+    assert input_labels == expected_labels, input_labels