Merge pull request #63 from csantanapr/os_chdir

makes current directory for action correct

Author: dubee

core/CHANGELOG.md

@@ -22,6 +22,7 @@
 ## Apache 1.13.0 (next release)
 Changes:
   - Update base python image to `python:3.6-alpine`
+  - Update current directory for action to be root of zip
 
 ## 1.3.3 (Apache 1.12.0)
 Changes:

core/actionProxy/actionproxy.py

@@ -52,6 +52,7 @@ def __init__(self, source=None, binary=None, zipdest=None):
         self.source = source if source else defaultBinary
         self.binary = binary if binary else defaultBinary
         self.zipdest = zipdest if zipdest else os.path.dirname(self.source)
+        os.chdir(os.path.dirname(self.source))
 
     def preinit(self):
         return

tests/src/test/scala/runtime/actionContainers/ActionProxyContainerTests.scala

@@ -252,4 +252,20 @@ class ActionProxyContainerTests extends BasicActionRunnerTests with WskActorSyst
         e shouldBe empty
     })
   }
+
+  it should "support current directory be action location" in {
+    withActionContainer() { c =>
+      val code = """
+                   |#!/bin/bash
+                   |echo "{\"pwd_env\":\"$PWD\",\"pwd_cmd\":\"$(pwd)\"}"
+                 """.stripMargin.trim
+
+      val (initCode, initRes) = c.init(initPayload(code))
+      initCode should be(200)
+
+      val (_, runRes) = c.run(runPayload(JsObject()))
+      runRes.get.fields.get("pwd_env") shouldBe Some(JsString("/action"))
+      runRes.get.fields.get("pwd_cmd") shouldBe Some(JsString("/action"))
+    }
+  }
 }