Merge pull request #30 from IGNF/fix-decorator

Fix copy_and_hack_decorator + version 1.4.1

CHANGELOG.md

@@ -1,3 +1,6 @@
+# 1.4.1
+- fix copy_and_hack_decorator (was not returning the decorated function output)
+
 # 1.4.0
 - count_occurences / replace_value: add copy_and_hack decorator to run on tscan output files
 - Update to pdal 2.6+ to better handle classification values and flags in replace_attribute_in_las

pdaltools/_version.py

@@ -1,4 +1,4 @@
-__version__ = "1.4.0"
+__version__ = "1.4.1"
 
 
 if __name__ == "__main__":

pdaltools/unlock_file.py

@@ -43,7 +43,8 @@ def newfn(*args, **kwargs):
                 with tempfile.NamedTemporaryFile(suffix=os.path.splitext(in_file)[-1]) as tmp:
                     copy_and_hack_input_file(in_file, tmp.name)
                     args[0] = tmp.name
-                    func(*args, **kwargs)
+
+                    return func(*args, **kwargs)
 
             else:
                 raise e

test/test_unlock.py

@@ -28,7 +28,8 @@ def decorated_pdal_info_json(input_file: str):
 def test_copy_and_hack_decorator_simple():
     TMP_FILE = os.path.join(TMPDIR, "copy_and_hack_simple.laz")
     shutil.copy(LAZ_FILE, TMP_FILE)
-    decorated_pdal_info_json(TMP_FILE)
+    ret = decorated_pdal_info_json(TMP_FILE)
+    assert ret  # Check that the return value of the decorated function is returned
 
 
 @pytest.mark.geoportail