Merge pull request #178 from Sahil-u07/add-unit-tests

Add unit tests for core Python components

Author: pradeeban

.gitignore

@@ -1 +1,21 @@
-**/.DS_Store
+# Python
+__pycache__/
+*.py[cod]
+*.class
+*.so
+.Python
+venv/
+env/
+ENV/
+
+# IDE
+.vscode/
+.idea/
+
+# Testing
+.pytest_cache/
+htmlcov/
+.coverage
+
+# Concore specific
+concorekill.bat

pytest.ini

@@ -0,0 +1,6 @@
+[pytest]
+testpaths = tests
+python_files = test_*.py
+python_classes = Test*
+python_functions = test_*
+addopts = -v --tb=short

requirements-dev.txt

@@ -0,0 +1,2 @@
+pytest>=7.0.0
+pytest-cov>=4.0.0

tests/__init__.py

@@ -0,0 +1,15 @@
+import pytest
+import os
+import sys
+import tempfile
+import shutil
+
+sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
+
+
+@pytest.fixture
+def temp_dir():
+    dirpath = tempfile.mkdtemp()
+    yield dirpath
+    if os.path.exists(dirpath):
+        shutil.rmtree(dirpath)

tests/conftest.py

@@ -0,0 +1,87 @@
+import pytest
+import os
+
+class TestSafeLiteralEval:
+
+    def test_reads_dictionary_from_file(self, temp_dir):
+        test_file = os.path.join(temp_dir, "config.txt")
+        with open(test_file, "w") as f:
+            f.write("{'name': 'test', 'value': 123}")
+        
+        from concore import safe_literal_eval
+        result = safe_literal_eval(test_file, {})
+        
+        assert result == {'name': 'test', 'value': 123}
+
+    def test_returns_default_when_file_missing(self):
+        from concore import safe_literal_eval
+        result = safe_literal_eval("nonexistent_file.txt", "fallback")
+        
+        assert result == "fallback"
+
+    def test_returns_default_for_empty_file(self, temp_dir):
+        test_file = os.path.join(temp_dir, "empty.txt")
+        with open(test_file, "w") as f:
+            pass
+        
+        from concore import safe_literal_eval
+        result = safe_literal_eval(test_file, "default")
+        
+        assert result == "default"
+
+
+class TestTryparam:
+
+    @pytest.fixture(autouse=True)
+    def reset_params(self):
+        from concore import params
+        original_params = params.copy()
+        yield
+        params.clear()
+        params.update(original_params)
+
+    def test_returns_existing_parameter(self):
+        from concore import tryparam, params
+        params['my_setting'] = 'custom_value'
+        
+        result = tryparam('my_setting', 'default_value')
+        
+        assert result == 'custom_value'
+
+    def test_returns_default_for_missing_parameter(self):
+        from concore import tryparam
+        result = tryparam('missing_param', 'fallback')
+        
+        assert result == 'fallback'
+
+
+class TestZeroMQPort:
+
+    def test_class_is_defined(self):
+        from concore import ZeroMQPort
+        assert ZeroMQPort is not None
+
+
+class TestDefaultConfiguration:
+
+    def test_default_input_path(self):
+        from concore import inpath
+        assert inpath == "./in"
+
+    def test_default_output_path(self):
+        from concore import outpath
+        assert outpath == "./out"
+
+
+class TestPublicAPI:
+
+    def test_module_imports_successfully(self):
+        from concore import safe_literal_eval
+        assert safe_literal_eval is not None
+
+    def test_core_functions_exist(self):
+        from concore import safe_literal_eval, tryparam, default_maxtime
+        
+        assert callable(safe_literal_eval)
+        assert callable(tryparam)
+        assert callable(default_maxtime)