Fix #216 by considering the case of job field with null value

common-workflow-language · Jul 14, 2024 · 713f6e2 · 713f6e2
1 parent 7208701
commit 713f6e2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cwltest/plugin.py b/cwltest/plugin.py
@@ -67,7 +67,7 @@ def _run_test_hook_or_plain(
     start_time = time.time()
     reltool = os.path.relpath(test["tool"], start=config.test_basedir)
     tooluri = urljoin(config.test_baseuri, reltool)
-    if "job" in test:
+    if test.get("job", None):
         reljob = os.path.relpath(test["job"], start=config.test_basedir)
         joburi = urljoin(config.test_baseuri, reljob)
     else:

diff --git a/cwltest/utils.py b/cwltest/utils.py
@@ -440,7 +440,7 @@ def run_test_plain(
 
     reltool = os.path.relpath(test["tool"], start=config.test_basedir)
     tooluri = urljoin(config.test_baseuri, reltool)
-    if "job" in test:
+    if test.get("job", None):
         reljob = os.path.relpath(test["job"], start=config.test_basedir)
         joburi = urljoin(config.test_baseuri, reljob)
     else: