This is an automated email from the ASF dual-hosted git repository.

dubeejw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-cli.git


The following commit(s) were added to refs/heads/master by this push:
     new 9740d5d  golint package path update (#374)
9740d5d is described below

commit 9740d5dfe0084a978c812982187cbb4eab0b3690
Author: Mark Deuser <mdeu...@us.ibm.com>
AuthorDate: Mon Oct 15 19:30:18 2018 -0400

    golint package path update (#374)
    
    * golint package path update
    
    * use controller.protocol instead of hardcoding protocol in test
    
    * remove controller http port now that the http api is no longer exposed
---
 build.gradle                                                  | 2 +-
 tests/src/test/scala/whisk/core/cli/test/WskConfigTests.scala | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/build.gradle b/build.gradle
index 68302d4..a3e2296 100644
--- a/build.gradle
+++ b/build.gradle
@@ -211,7 +211,7 @@ resolveBuildDependencies.dependsOn(goI18n)
        The get step is needed to be sure a golint binary is available to run.
  */
 task getGoLint(type: com.github.blindpirate.gogradle.Go) {
-    go 'get -u github.com/golang/lint/golint'
+    go 'get -u golang.org/x/lint/golint'
 }
 
 task goLint(type: com.github.blindpirate.gogradle.Go, dependsOn: getGoLint) {
diff --git a/tests/src/test/scala/whisk/core/cli/test/WskConfigTests.scala 
b/tests/src/test/scala/whisk/core/cli/test/WskConfigTests.scala
index aa49ded..039692c 100644
--- a/tests/src/test/scala/whisk/core/cli/test/WskConfigTests.scala
+++ b/tests/src/test/scala/whisk/core/cli/test/WskConfigTests.scala
@@ -265,11 +265,13 @@ class WskConfigTests extends TestHelpers with 
WskTestHelpers {
     }
   }
 
-  it should "show api build using http apihost" in {
+  it should "show api build using explicit protocol for apihost" in {
     val tmpwskprops = File.createTempFile("wskprops", ".tmp")
     try {
       val env = Map("WSK_CONFIG_FILE" -> tmpwskprops.getAbsolutePath())
-      val apihost = s"http://${WhiskProperties.getBaseControllerAddress()}"
+      val controllerProtocol = 
WhiskProperties.getProperty("controller.protocol")
+      val apihost =
+        s"${controllerProtocol}://${WhiskProperties.getBaseControllerHost()}"
       wsk.cli(Seq("property", "set", "--apihost", apihost), env = env)
       val rr = wsk.cli(Seq("property", "get", "--apibuild", "-i"), env = env)
       rr.stdout should not include regex("""whisk API build\s*Unknown""")

Reply via email to