better handle symlinks in Directories

Author: mr-c

cwltool/process.py

@@ -356,9 +356,9 @@ def _relocate(src: str, dst: str) -> None:
             return
 
         # If the source is not contained in source_directories we're not allowed to delete it
-        src = fs_access.realpath(src)
+        real_src = fs_access.realpath(src)
         src_can_deleted = any(
-            os.path.commonprefix([p, src]) == p for p in source_directories
+            os.path.commonprefix([p, real_src]) == p for p in source_directories
         )
 
         _action = "move" if action == "move" and src_can_deleted else "copy"

tests/symlinks.cwl

@@ -0,0 +1,14 @@
+#!/usr/bin/env cwl-runner
+cwlVersion: v1.0
+class: CommandLineTool
+inputs: []
+baseCommand: [ bash, -c ]
+arguments:
+# - "mkdir foo; echo 42 > foo/bar; ln -s bar foo/baz"
+ - "mkdir foo; echo 42 > foo/bar; ln -s $PWD/foo/bar foo/baz"
+# - "mkdir foo; ln -s $PWD/foo/bar foo/baz ; echo 42 > foo/bar"
+outputs:
+  result:
+    type: Directory
+    outputBinding:
+      glob: foo

tests/test_relocate.py

@@ -6,10 +6,7 @@
 
 from .util import get_data, needs_docker
 
-if sys.version_info[0] < 3:
-    from StringIO import StringIO
-else:
-    from io import StringIO
+from io import StringIO
 
 
 @needs_docker
@@ -18,6 +15,20 @@ def test_for_910() -> None:
     assert main([get_data("tests/wf/910.cwl")]) == 0
 
 
+def test_symlinks_with_absolute_paths(tmp_path: Path) -> None:
+    assert (
+        main(
+            [
+                "--debug",
+                f"--outdir={tmp_path}/result",
+                "--tmpdir-prefix={tmp_path}/tmp",
+                get_data("tests/symlinks.cwl"),
+            ]
+        )
+        == 0
+    )
+
+
 @needs_docker
 def test_for_conflict_file_names(tmp_path: Path) -> None:
     stream = StringIO()