refactor for transferable

Signed-off-by: mgorsk1 <[email protected]>

Author: mgorsk1

core/testcontainers/core/container.py

@@ -1,7 +1,7 @@
 import contextlib
 import io
+import os
 import tarfile
-from pathlib import Path
 from platform import system
 from socket import socket
 from typing import TYPE_CHECKING, Optional
@@ -16,6 +16,7 @@
 from testcontainers.core.exceptions import ContainerStartException
 from testcontainers.core.labels import LABEL_SESSION_ID, SESSION_ID
 from testcontainers.core.network import Network
+from testcontainers.core.transferable import Transferable
 from testcontainers.core.utils import inside_container, is_arm, setup_logger
 from testcontainers.core.waiting_utils import wait_container_is_ready, wait_for_logs
 
@@ -55,7 +56,7 @@ def __init__(
         self._network: Optional[Network] = None
         self._network_aliases: Optional[list[str]] = None
         self._kwargs = kwargs
-        self._files: list[tuple[Path, Path]] = []
+        self._files: list[Transferable] = []
 
     def with_env(self, key: str, value: str) -> Self:
         self.env[key] = value
@@ -82,12 +83,12 @@ def with_kwargs(self, **kwargs) -> Self:
         self._kwargs = kwargs
         return self
 
-    def with_copy_file_to_container(self, source_file: Path, destination_file: Path) -> Self:
-        self._files.append((source_file, destination_file))
+    def with_copy_file_to_container(self, transferable: Transferable) -> Self:
+        self._files.append(transferable)
 
         return self
 
-    def copy_file_from_container(self, container_file: Path, destination_file: Path) -> Path:
+    def copy_file_from_container(self, container_file: os.PathLike, destination_file: os.PathLike) -> os.PathLike:
         tar_stream, _ = self._container.get_archive(container_file)
 
         for chunk in tar_stream:
@@ -99,11 +100,11 @@ def copy_file_from_container(self, container_file: Path, destination_file: Path)
         return destination_file
 
     @staticmethod
-    def _put_file_in_container(container, source_file: Path, destination_file: Path):
+    def _put_data_in_container(container, transferable: Transferable):
         data = io.BytesIO()
 
-        with tarfile.open(fileobj=data, mode="w") as tar:
-            tar.add(source_file, arcname=destination_file)
+        with transferable as f, tarfile.open(fileobj=data, mode="w") as tar:
+            tar.add(f.input_path, arcname=f.output_path)
 
         data.seek(0)
 
@@ -149,10 +150,8 @@ def start(self) -> Self:
         if self._network:
             self._network.connect(self._container.id, self._network_aliases)
 
-        for copy_spec in self._files:
-            source, destination = copy_spec[0], copy_spec[1]
-
-            DockerContainer._put_file_in_container(self._container, source, destination)
+        for transferable in self._files:
+            DockerContainer._put_data_in_container(self._container, transferable)
 
         return self
 

core/testcontainers/core/transferable.py

@@ -0,0 +1,33 @@
+import os
+import tempfile
+from typing import Union
+
+
+class Transferable:
+    def __init__(self, input_data: Union[os.PathLike, bytes], output_path: os.PathLike):
+        self._input = input_data
+        self._output_path = output_path
+
+        self._tmp_file: bool = False
+
+    def __enter__(self):
+        if isinstance(self._input, bytes):
+            tmp_file = tempfile.NamedTemporaryFile(delete=False)
+            tmp_file.write(self._input)
+
+            self._input = tmp_file.name
+            self._tmp_file = True
+
+        return self
+
+    def __exit__(self, *args):
+        if self._tmp_file:
+            os.remove(self._input)
+
+    @property
+    def input_path(self):
+        return self._input
+
+    @property
+    def output_path(self):
+        return self._output_path