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