Fix #6: cluster name must be long enough

Author: icy

main.go

@@ -96,12 +96,12 @@ func main() {
 	}
 
 	cluster_name := os.Args[1]
+	if len(cluster_name) <= 1 {
+		log2exit(1, ":: Error: Cluster name must be something like :foo.\n")
+	}
 	if cluster_name[0:1] != ":" {
 		log2exit(1, fmt.Sprintf(":: Error: Cluster name (context) must be prefixed with `:', e.g., gk8s :%s.\n", cluster_name))
 	}
-	if len(cluster_name) == 1 {
-		log2exit(1, ":: Error: Cluster name is not provided.\n")
-	}
 	cluster_name = cluster_name[1:]
 
 	config_dir := os.Getenv("GK8S_HOME")

tests.sh

@@ -4,11 +4,14 @@
 # Author  : Ky-Anh Huynh
 # License : Public domain
 
-
 _gk8s() {
   "$(pwd -P)"/gk8s "${@}"
 }
 
+_fail_issue_6_with_empty_cluster_name() {
+  _gk8s ''
+}
+
 _fail_without_any_argument() {
   _gk8s
 }
@@ -167,6 +170,10 @@ default() {
       "Command not found" \
       "Exit immediately when requested kubectl command is not found."
 
+  _test _fail_issue_6_with_empty_cluster_name \
+      "Error: Cluster name must be something like :foo" \
+      "#6: Cluster name must not be empty."
+
   _test _fail_with_wrong_cluster_prefix \
       "must be prefixed with" \
       "Return error when cluster name is not started with :"
@@ -221,7 +228,8 @@ default() {
 
   _test _fail_with_helm_repo_list \
       "Error: no repositories to show" \
-      "helm would not need any separator (--)."
+      "helm would not need any separator (--). To get this test passed,
+      helm binary should be found on your local system."
 }
 
 ### main routines ######################################################