update pathmapper entries for non-Docker as well

Author: mr-c

cwltool/job.py

@@ -18,26 +18,23 @@
 
 import shellescape
 from prov.model import PROV
-from schema_salad.sourceline import SourceLine
 from six import PY2, with_metaclass
 from typing_extensions import (TYPE_CHECKING,  # pylint: disable=unused-import
                                Text)
-# move to a regular typing import when Python 3.3-3.6 is no longer supported
+from schema_salad.sourceline import SourceLine
 
 from .builder import Builder, HasReqsHints  # pylint: disable=unused-import
 from .context import RuntimeContext  # pylint: disable=unused-import
 from .context import getdefault
 from .errors import WorkflowException
 from .loghandler import _logger
-from .pathmapper import (MapperEnt, PathMapper, ensure_writable,
-                         ensure_non_writable)
-from .process import UnsupportedRequirement, stageFiles
+from .pathmapper import (MapperEnt, PathMapper,  # pylint: disable=unused-import
+                         ensure_writable, ensure_non_writable)
+from .process import UnsupportedRequirement, stage_files
 from .secrets import SecretStore  # pylint: disable=unused-import
-from .utils import \
-    bytes2str_in_dicts  # pylint: disable=unused-import; pylint: disable=unused-import
-from .utils import (DEFAULT_TMP_PREFIX, Directory, copytree_with_merge,
-                    json_dump, json_dumps, onWindows, processes_to_kill,
-                    subprocess)
+from .utils import (DEFAULT_TMP_PREFIX, Directory, bytes2str_in_dicts,
+                    copytree_with_merge, json_dump, json_dumps, onWindows,
+                    processes_to_kill, subprocess)
 
 if TYPE_CHECKING:
     from .provenance import CreateProvProfile  # pylint: disable=unused-import
@@ -409,14 +406,17 @@ def run(self,
         if "PATH" not in env:
             env["PATH"] = str(os.environ["PATH"]) if onWindows() else os.environ["PATH"]
         if "SYSTEMROOT" not in env and "SYSTEMROOT" in os.environ:
-            env["SYSTEMROOT"] = str(os.environ["SYSTEMROOT"]) if onWindows() else os.environ["SYSTEMROOT"]
+            env["SYSTEMROOT"] = str(os.environ["SYSTEMROOT"]) if onWindows() \
+                    else os.environ["SYSTEMROOT"]
 
-        stageFiles(self.pathmapper, ignoreWritable=True, symLink=True, secret_store=runtimeContext.secret_store)
+        stage_files(self.pathmapper, ignore_writable=True, symlink=True,
+                    secret_store=runtimeContext.secret_store)
         if self.generatemapper is not None:
-            stageFiles(self.generatemapper, ignoreWritable=self.inplace_update,
-                       symLink=True, secret_store=runtimeContext.secret_store)
-            relink_initialworkdir(self.generatemapper, self.outdir,
-                                  self.builder.outdir, inplace_update=self.inplace_update)
+            stage_files(self.generatemapper, ignore_writable=self.inplace_update,
+                        symlink=True, secret_store=runtimeContext.secret_store)
+            relink_initialworkdir(
+                self.generatemapper, self.outdir, self.builder.outdir,
+                inplace_update=self.inplace_update)
 
         self._execute([], env, runtimeContext)
 
@@ -538,12 +538,11 @@ def add_volumes(self,
                 new_path = self.create_file_and_add_volume(
                     runtime, vol, host_outdir_tgt, secret_store)
                 key = pathmapper.reversemap(vol.target)
-                if key:
+                if key and new_path:
                     pathmapper.update(
                         key[0], new_path, vol.target, vol.type, vol.staged)
 
-    def run(self, runtimeContext):
-        # type: (RuntimeContext) -> None
+    def run(self, runtimeContext):  # type: (RuntimeContext) -> None
         if not os.path.exists(self.tmpdir):
             os.makedirs(self.tmpdir)
         (docker_req, docker_is_req) = self.get_requirement("DockerRequirement")