Merge pull request #183 from jscotka/tempfile

fix tempfile handling

Reviewed-by: None <None>

Author: softwarefactory-project-zuul[bot]

requre/helpers/tempfile.py

@@ -27,14 +27,36 @@
 from typing import Optional, Any
 from requre.cassette import Cassette, CassetteExecution
 from requre.objects import ObjectStorage
+from requre.helpers.simple_object import Simple
+from warnings import warn
 
 logger = logging.getLogger(__name__)
 
 
+class MkTemp(Simple):
+    """
+    decorate mktemp method wrapper
+    """
+
+    pass
+
+
+class MkDTemp(Simple):
+    """
+    decorate mkdtemp method wrapper
+    """
+
+    def from_serializable(self, data: Any) -> Any:
+        os.makedirs(data, exist_ok=True)
+        return data
+
+
 class TempFile(ObjectStorage):
     """
     replace system tempfile module with own predictable names implementation
      of temp files for mocking
+
+     Warn: Replaced by new implementations MkDTemp and MkTemp classes
     """
 
     root = "/tmp"
@@ -81,6 +103,7 @@ def mktemp(cls, cassette: Optional[Cassette] = None) -> Any:
         :return: CassetteExecution class with function and cassette instance
 
         """
+        warn("Please replace it by MkTemp.decorator_plain()")
         casex = CassetteExecution()
         casex.cassette = cassette or cls.get_cassette()
         casex.obj_cls = cls
@@ -105,6 +128,7 @@ def mkdtemp(cls, cassette: Optional[Cassette] = None) -> Any:
         :return: CassetteExecution class with function and cassette instance
 
         """
+        warn("Please replace it by class mkdtemp MkDTemp.decorator_plain()")
         casex = CassetteExecution()
         casex.cassette = cassette or cls.get_cassette()
         casex.obj_cls = cls

requre/modules_decorate_all_methods.py

@@ -4,7 +4,7 @@
     apply_decorator_to_all_methods,
     record_requests_for_all_methods,
 )
-from requre.helpers.tempfile import TempFile
+from requre.helpers.tempfile import MkTemp, MkDTemp
 from requre.cassette import Cassette
 from requre.constants import TEST_METHOD_REGEXP
 from requre.helpers.git.repo import Repo
@@ -47,8 +47,8 @@ def record_tempfile_module(
     regexp_method_pattern=TEST_METHOD_REGEXP,
 ):
     decorators = [
-        ("tempfile.mkdtemp", TempFile.mkdtemp()),
-        ("tempfile.mktemp", TempFile.mktemp()),
+        ("tempfile.mkdtemp", MkDTemp.decorator_plain()),
+        ("tempfile.mktemp", MkTemp.decorator_plain()),
     ]
     return __replace_module_match_with_multiple_decorators(
         *decorators,