Move remote command to cloud as subcommand

Author: iv-p

docs/SUMMARY.md

@@ -12,8 +12,8 @@
   - [Jenkins](integration/jenkins.md)
 - [CLI](cli/README.md)
   - [check](cli/check.md)
-  - [remote](cli/remote.md)
   - [cloud](cli/cloud/README.md)
+    - [check](cli/cloud/check.md)
     - [list](cli/cloud/list.md)
     - [upsert](cli/cloud/upsert.md)
     - [enable](cli/cloud/enable.md)

docs/cli/README.md

@@ -3,5 +3,4 @@
 In this section we will discuss the different CLI commands available for all users.
 
 - [check](check.md)
-- [remote](remote.md)
 - [cloud](cloud/README.md)

docs/cli/cloud/README.md

@@ -2,6 +2,7 @@
 
 In this section we will discuss the different CLI commands available for communication with APId cloud.
 
+- [check](check.md)
 - [list](list.md)
 - [upsert](upsert.md)
 - [enable](enable.md)

docs/cli/remote.md docs/cli/cloud/check.mddocs/cli/remote.md renamed to docs/cli/cloud/check.md

@@ -1,10 +1,10 @@
-# Remote
+# Check
 
-The remote command uses exactly the same configs as `check`, but executes requests from the APId cloud. This allows you to test your API from around the world. Remote will run all the transactions defined in `apid.yaml` in the current directory or optionally take a path to the config via the `-c` or `--config` flag. To get familiar with the syntax of a config file, see [Reference](../reference/README.md)
+The cloud check command uses exactly the same configs as `check`, but executes requests from the APId cloud. This allows you to test your API from around the world. Remote will run all the transactions defined in `apid.yaml` in the current directory or optionally take a path to the config via the `-c` or `--config` flag. To get familiar with the syntax of a config file, see [Reference](../reference/README.md)
 
 # Details
 
-Remote takes all the transactions that you have specified in the yaml. The steps in each transaction are executed sequentially. If a step fails, then the whole transaction is aborted and the rest of the steps are ignored. Each step is executed on a remote server, in a location specified via the command line flags. If a transaction fails, this will be reported in the console and the next transaction will be started.
+Check takes all the transactions that you have specified in the yaml. The steps in each transaction are executed sequentially. If a step fails, then the whole transaction is aborted and the rest of the steps are ignored. Each step is executed on a remote server, in a location specified via the command line flags. If a transaction fails, this will be reported in the console and the next transaction will be started.
 
 ## Flags
 
@@ -18,7 +18,7 @@ Remote takes all the transactions that you have specified in the yaml. The steps
 # Examples
 
 ```bash
-apid remote --key <access-key>
-apid remote --config ./tests/e2e/apid.yaml --key <access-key>
-apid remote --config ./tests/e2e/apid.yaml --key <access-key> --region washington
+apid cloud check --key <access-key>
+apid cloud check --config ./tests/e2e/apid.yaml --key <access-key>
+apid cloud check --config ./tests/e2e/apid.yaml --key <access-key> --region washington
 ```

docs/cloud/remote.md

@@ -17,7 +17,7 @@ In order to use the power of the cloud you will need a personal access key. To g
 
 Once you have your key you will need to [install the APId CLI](../installation/cli.md) (if you haven't already) or use our [official docker image](../installation/docker.md).
 
-A reference on how use the CLI after installation for remote execution can be found [here](../cli/remote.md).
+A reference on how use the CLI after installation for remote execution can be found [here](../cli/cloud/check.md).
 
 ## Timeouts
 

svc/cli/cmd/remote_check.go svc/cli/cmd/cloud/check.gosvc/cli/cmd/remote_check.go renamed to svc/cli/cmd/cloud/check.go

@@ -1,4 +1,4 @@
-package cmd
+package cloud
 
 import (
 	"fmt"
@@ -16,41 +16,36 @@ import (
 	"github.com/spf13/cobra"
 )
 
-const (
-	apiKeyEnvKey = "APID_KEY"
-)
-
 var (
-	apiKey = ""
-	region = ""
+	region      = ""
+	showTimings = false
 )
 
-var remoteCmd = &cobra.Command{
-	Use:   "remote",
+var checkCmd = &cobra.Command{
+	Use:   "check",
 	Short: "Executes a config on apid remote infrastructure",
 	Long: `Loads either a config file, or if a directory is provided it 
 recursively loads all .yaml files and executes them. It will run all the 
 transactions in you config, verify the responses, and record the time it 
 took to action each request. Runs all steps in each transaction on a 
-public cloud infrastructure.`,
+public cloud infrastructure. See https://docs.getapid.com for more inforamtion`,
 	Example: `
-	apid remote	--key <apid access key>
-	apid remote --config my-api.yaml --key <apid access key> --region us-east
-	apid remote -c ./e2e-tests/ -k <apid access key>
-	apid remote -c ./e2e-tests/ -k <apid access key> -r us-east`,
+	apid cloud check --key <apid access key>
+	apid cloud check --config my-api.yaml --key <apid access key> --region dublin
+	apid cloud check -c ./e2e-tests/ -k <apid access key>
+	apid cloud check -c ./e2e-tests/ -k <apid access key> -r dublin`,
 	Args: cobra.NoArgs,
-	RunE: remoteRun,
+	RunE: remoteCheck,
 }
 
 func init() {
-	rootCmd.AddCommand(remoteCmd)
-	remoteCmd.Flags().StringVarP(&configFilepath, "config", "c", "./apid.yaml", "file with config to run")
-	remoteCmd.Flags().BoolVarP(&showTimings, "timings", "t", false, "output the durations of requests")
-	remoteCmd.Flags().StringVarP(&apiKey, "key", "k", os.Getenv(apiKeyEnvKey), "apid access key")
-	remoteCmd.Flags().StringVarP(&region, "region", "r", "washington", "location to run the tests from")
+	RootCommand.AddCommand(checkCmd)
+	checkCmd.Flags().StringVarP(&configFilepath, "config", "c", "./apid.yaml", "file with config to run")
+	checkCmd.Flags().BoolVarP(&showTimings, "timings", "t", false, "output the durations of requests")
+	checkCmd.Flags().StringVarP(&region, "region", "r", "washington", "location to run the tests from")
 }
 
-func remoteRun(cmd *cobra.Command, args []string) error {
+func remoteCheck(cmd *cobra.Command, args []string) error {
 	c, err := config.Load(configFilepath)
 	if err != nil {
 		return fmt.Errorf("could not load config file: %v", err)