- Fix Docker execution for ${PIN}file

  And adds comment for its use

Author: Alan Christie

src/groovy/ContainerExecutor.groovy

@@ -48,8 +48,13 @@ class ContainerExecutor {
         StringBuilder sout = new StringBuilder()
         StringBuilder serr = new StringBuilder()
 
+        // Note: PIN and POUT have trailing forward-slashed for now
+        //       to allow a migratory use of $PIN}file references
+        //       rather than insisting on ${PIN}/file which would fail if
+        //       PIN wasn't defined - it's about lowest risk changes.
+
         String cmd = "docker run -v $pin:/data -v $pout:/output" +
-                     " -w /output -e PIN=/data -e POUT=/output $imageName" +
+                     " -w /output -e PIN=/data/ -e POUT=/output/ $imageName" +
                      " sh -c '$command'"
 
         def proc = ['sh', '-c', cmd].execute(null, new File('.'))

src/groovy/Tester.groovy

@@ -676,7 +676,7 @@ class Tester {
 
             // Redirect output
             String outputFileBaseName = oOption[0][1]
-            String testOutputFile = '\\${POUT}' + File.separator + outputFileBaseName
+            String testOutputFile = '\\${POUT}' + outputFileBaseName
             // Now swap-out the original '-o'...
             String redirectedOutputOption = "-o ${testOutputFile}"
             pipelineCommand = pipelineCommand.replaceAll(/$outputRegex/,