feat(input_file): test using context

Author: MaxNumerique

src/opengeodeweb_back/utils_functions.py

@@ -145,14 +145,20 @@ def handle_exception(e):
     return response
 
 
-def create_unique_data_folder(base_data_folder: str) -> tuple[str, str]:
+def create_unique_data_folder() -> tuple[str, str]:
+    base_data_folder = flask.current_app.config["DATA_FOLDER_PATH"]
     generated_id = str(uuid.uuid4()).replace("-", "")
     data_path = os.path.join(base_data_folder, generated_id)
     os.makedirs(data_path, exist_ok=True)
     return generated_id, data_path
 
 
-def save_all_viewables_and_return_info(geode_object, data, generated_id, data_path, additional_files=None):
+def save_all_viewables_and_return_info(
+        geode_object, 
+        data, generated_id, 
+        data_path, 
+        additional_files=None
+    ):
     saved_native_file_path = geode_functions.save(
         geode_object,
         data,
@@ -179,14 +185,19 @@ def save_all_viewables_and_return_info(geode_object, data, generated_id, data_pa
         "input_files": additional_files or [],
     }
 
-def generate_native_viewable_and_light_viewable_from_object(geode_object, data):
+def generate_native_viewable_and_light_viewable_from_object(
+        geode_object, 
+        data
+    ):
     base_data_folder = flask.current_app.config["DATA_FOLDER_PATH"]
     generated_id, data_path = create_unique_data_folder(base_data_folder)
-
     return save_all_viewables_and_return_info(geode_object, data, generated_id, data_path)
 
 
-def generate_native_viewable_and_light_viewable_from_file(geode_object, input_filename):
+def generate_native_viewable_and_light_viewable_from_file(
+        geode_object, 
+        input_filename
+    ):
     base_data_folder = flask.current_app.config["DATA_FOLDER_PATH"]
     generated_id, data_path = create_unique_data_folder(base_data_folder)
 

tests/test_utils_functions.py

@@ -72,16 +72,16 @@ def test_handle_exception(client):
     assert type(data["code"]) is int
 
 
-def test_create_unique_data_folder():
-    base_data_folder = "./tests/data"
-    generated_id, data_path = utils_functions.create_unique_data_folder(base_data_folder)
-    assert isinstance(generated_id, str)
-    assert re.fullmatch(r"[0-9a-f]{32}", generated_id)
-    assert os.path.exists(data_path)
-    assert data_path.startswith(base_data_folder)
-    assert generated_id in data_path
-    shutil.rmtree(data_path, ignore_errors=True)
-    assert not os.path.exists(data_path)
+def test_create_unique_data_folder(client):
+    app = client.application
+    with app.app_context():
+        generated_id, data_path = utils_functions.create_unique_data_folder()
+        assert isinstance(generated_id, str)
+        assert re.fullmatch(r"[0-9a-f]{32}", generated_id)
+        assert os.path.isdir(data_path)
+        assert generated_id in data_path
+        shutil.rmtree(data_path, ignore_errors=True)
+        assert not os.path.exists(data_path)
 
 
 
@@ -98,7 +98,6 @@ def test_save_all_viewables_and_return_info(client):
         result = utils_functions.save_all_viewables_and_return_info(
             geode_object, data, generated_id, data_path, additional_files
         )
-
     assert isinstance(result, dict)
     assert result["name"] == data.name()
     assert result["native_file_name"].startswith("native.")
@@ -115,13 +114,11 @@ def test_generate_native_viewable_and_light_viewable_from_object(client):
     with app.app_context():
         geode_object = "BRep"
         data = geode_functions.load(geode_object, "./tests/data/test.og_brep")
-
         result = (
             utils_functions.generate_native_viewable_and_light_viewable_from_object(
                 geode_object, data
             )
         )
-
     assert isinstance(result, dict)
     assert isinstance(result["name"], str)
     assert isinstance(result["native_file_name"], str)
@@ -144,7 +141,6 @@ def test_generate_native_viewable_and_light_viewable_from_file(client):
         result = utils_functions.generate_native_viewable_and_light_viewable_from_file(
             geode_object, input_filename
         )
-
     assert isinstance(result, dict)
     assert isinstance(result["name"], str)
     assert isinstance(result["native_file_name"], str)