Bug fix for unintentional variable section requirement. (#256)

Closes #255. A bug fix that corrects an unintentional assumption that the variable section in a specification will always exist.
LLNL · Apr 30, 2020 · aeda12f · aeda12f
1 parent 60d5175
commit aeda12f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/maestrowf/datastructures/schemas.json b/maestrowf/datastructures/schemas.json
@@ -62,7 +62,7 @@
     "properties": {
       "variables": {"type": "object"},
       "labels": {"type": "object"},
-      "sources": {"type": "object"},
+      "sources": {"type": "array"},
       "dependencies": {
         "type": "object",
         "properties": {

diff --git a/maestrowf/datastructures/yamlspecification.py b/maestrowf/datastructures/yamlspecification.py
@@ -197,6 +197,8 @@ def _verify_variables(self):
         :returns: A set of keys encountered in the variables section.
         """
         keys_seen = set()
+        if "variables" not in self.environment:
+            return keys_seen
         for key, value in self.environment["variables"].items():
             logger.debug("Verifying %s...", key)
             if not key: