Remove FISSION_ROUTER for fn test (#1465)

pkg/fission-cli/cmd/function/test.go

@@ -58,18 +58,17 @@ func (opts *TestSubCommand) do(input cli.Input) error {
 	}
 
 	routerURL := os.Getenv("FISSION_ROUTER")
-	if len(routerURL) == 0 {
+	if len(routerURL) != 0 {
-		// Portforward to the fission router
+		console.Warn("The environment variable FISSION_ROUTER is no longer supported for this command")
-		localRouterPort, err := util.SetupPortForward(util.GetFissionNamespace(), "application=fission-router")
-		if err != nil {
-			return err
-		}
-		routerURL = "127.0.0.1:" + localRouterPort
-	} else {
-		console.Verbose(2, "Env FISSION_ROUTER: %v", routerURL)
-		routerURL = strings.TrimPrefix(routerURL, "http://")
 	}
 
+	// Portforward to the fission router
+	localRouterPort, err := util.SetupPortForward(util.GetFissionNamespace(), "application=fission-router")
+	if err != nil {
+		return err
+	}
+	routerURL = "127.0.0.1:" + localRouterPort
+
 	fnUri := m.Name
 	if m.Namespace != metav1.NamespaceDefault {
 		fnUri = fmt.Sprintf("%v/%v", m.Namespace, m.Name)