Update tests

packaging/filereplacer.py

@@ -40,11 +40,10 @@ def convert(infile, outfile):
                     out += os.environ[name]
 
                 else:
-                    raise Exception(f"The variable {name} is not defined")
+                    raise KeyError(f"The variable {name} is not defined")
 
-    if out:
-        with open(outfile, 'w', encoding="utf-8") as fout:
-            fout.write(out)
+    with open(outfile, 'w', encoding="utf-8") as fout:
+        fout.write(out)
 
 
 if __name__ == '__main__':

sonar-project.properties

@@ -1,4 +1,4 @@
 sonar.projectKey=Laerdal_python-objdictgen
 sonar.organization=laerdal-foss
 sonar.python.coverage.reportPaths=coverage.xml
-sonar.exclusions = tests/**
+sonar.exclusions = tests/**, packaging/**

tests/test_installer.py

@@ -0,0 +1,38 @@
+
+import pytest
+from unittest import mock
+import os
+import sys
+
+@pytest.fixture
+def setenvvar(monkeypatch):
+    with mock.patch.dict(os.environ, {"TEST": "foobar"}):
+        yield
+
+
+def test_filereplacer(basepath, wd, setenvvar):
+
+    sys.path.append(str(basepath / "packaging"))
+    os.chdir(basepath)
+    from filereplacer import convert
+
+    tests = [
+        (1, "Test data", "Test data"),
+        (2, "@@{name}", "objdictgen"),
+        (3, "@@{TEST}", "foobar"),  # Read from the mocked environment variable
+        (4, "@@{nonexisting}", "non-existing"),
+    ]
+
+    for i, data, result in tests:
+        infile = wd / "test.txt"
+        outfile = wd / "out.txt"
+        with open(infile, "w", encoding="utf-8") as f:
+            f.write(data)
+        if i == 4:
+            with pytest.raises(KeyError):
+                convert(infile, outfile)
+            continue
+        else:
+            convert(infile, outfile)
+            with open(outfile, "r", encoding="utf-8") as f:
+                assert f.read() == result