fix issue where manifest could be empty (#191)

BlueBrain · Mar 22, 2023 · 839c910 · 839c910
1 parent 201a3d3
commit 839c910
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/bluepysnap/config.py b/bluepysnap/config.py
@@ -65,8 +65,8 @@ def __init__(self, config, config_dir):
              Parser: A Parser object.
         """
         content = config.copy()
-
-        self.manifest = Parser._resolve_manifest(content.pop("manifest", {}), config_dir)
+        manifest = content.pop("manifest", None) or {}
+        self.manifest = Parser._resolve_manifest(manifest, config_dir)
         self.content = content
 
     @staticmethod

diff --git a/tests/test_circuit.py b/tests/test_circuit.py
@@ -96,3 +96,20 @@ def test_pickle(tmp_path):
 
     assert pickle_path.stat().st_size < 200
     assert list(circuit.edges) == ["default", "default2"]
+
+
+def test_empty_manifest(tmp_path):
+    path = tmp_path / "circuit_config.json"
+    with open(path, "w") as fd:
+        json.dump(
+            {
+                "manifest": {},
+                "networks": {
+                    "nodes": {},
+                    "edges": {},
+                },
+            },
+            fd,
+        )
+
+    test_module.Circuit(path)