mdeuser commented on a change in pull request #2461: Get Action URL from CLI
(Review)
URL:
https://github.com/apache/incubator-openwhisk/pull/2461#discussion_r126004176
##########
File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala
##########
@@ -695,6 +696,59 @@ class WskBasicUsageTests
}
}
+ it should "get an action URL" in withAssetCleaner(wskprops) {
+ (wp, assetHelper) =>
+ val actionName = "action name"
+ val packageName = "package name"
+ val defaultPackageName = "default"
+ val webActionName = "web action name"
+ val nonExistentActionName = "non-existence action"
Review comment:
Also try an action name that has a bunch of special characters to validate
the url encoding is correct
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
