rabbah commented on a change in pull request #2591: User-Agent CLI/version 
header
URL: 
https://github.com/apache/incubator-openwhisk/pull/2591#discussion_r136485499
 
 

 ##########
 File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala
 ##########
 @@ -119,6 +119,10 @@ class WskBasicUsageTests
             wsk.action.get(fullQualifiedName).stdout should include(s"ok: got 
action ${packageName}/${actionName}")
     }
 
+    it should "include CLI user agent headers with outbound requests" in {
+        val stdout = wsk.action.list(verbose = true).stdout
+        stdout should include regex ("""\bUser-Agent\b": 
\[\s+"OpenWhisk\-CLI/1.\d+.*""")
 
 Review comment:
   since the goal is just to run a cli command with verbose on, i'd rather you 
don't change the `list` method and install use `wsk.cli(Seq("list"), verbose = 
true)` directly.
 
----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to