[GitHub] [incubator-openwhisk-cli] sciabarracom commented on a change in pull request #348: Update cli to fail create/update sequences with limits

[GitBox <gi...@apache.org>]

sciabarracom commented on a change in pull request #348: Update cli to fail create/update sequences with limits
URL: https://github.com/apache/incubator-openwhisk-cli/pull/348#discussion_r281586675
 
 

 ##########
 File path: tests/src/test/scala/whisk/core/cli/test/WskCliBasicUsageTests.scala
 ##########
 @@ -173,6 +173,48 @@ class WskCliBasicUsageTests extends TestHelpers with WskTestHelpers {
                       expectedExitCode = MISUSE_EXIT)
   }
 
+  it should "reject create or update of an action sequence that sets a memory limit" in withAssetCleaner(wskprops) { (wp, assetHelper) =>
 
 Review comment:
   I built the cli and I tried with an existing configuration with a namespace:
   
   Old: 
   wsk -i action list
   error: Unable to obtain the list of actions for namespace 'neuronix_cloud': The supplied authentication is not authorized to access 'neuronix_cloud'. (code nVQD2dAZZbjgN4Z1fqsXkE7Q8aAFVFSi)
   Run 'wsk --help' for usage.
   
   New:
   ./wsk -i action list
   actions

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services