[apache#5830] fix(client): add error handling for no tag in cli. (apa…

…che#5857)

### What changes were proposed in this pull request?

Add error handle with a friendly output for no tags command line.

### Why are the changes needed?

Now it throws an exception and may make customer confused.

Fix: apache#5830 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Unit tests have been attached.

clients/cli/src/main/java/org/apache/gravitino/cli/commands/CreateTag.java

@@ -52,11 +52,15 @@ public CreateTag(
   /** Create tags. */
   @Override
   public void handle() {
-    boolean hasOnlyOneTag = tags.length == 1;
-    if (hasOnlyOneTag) {
-      handleOnlyOneTag();
+    if (tags == null || tags.length == 0) {
+      System.err.println(ErrorMessages.TAG_EMPTY);
     } else {
-      handleMultipleTags();
+      boolean hasOnlyOneTag = tags.length == 1;
+      if (hasOnlyOneTag) {
+        handleOnlyOneTag();
+      } else {
+        handleMultipleTags();
+      }
     }
   }
 

clients/cli/src/main/java/org/apache/gravitino/cli/commands/DeleteTag.java

@@ -57,11 +57,16 @@ public void handle() {
     if (!AreYouSure.really(force)) {
       return;
     }
-    boolean hasOnlyOneTag = tags.length == 1;
-    if (hasOnlyOneTag) {
-      handleOnlyOneTag();
+
+    if (tags == null || tags.length == 0) {
+      System.err.println(ErrorMessages.TAG_EMPTY);
     } else {
-      handleMultipleTags();
+      boolean hasOnlyOneTag = tags.length == 1;
+      if (hasOnlyOneTag) {
+        handleOnlyOneTag();
+      } else {
+        handleMultipleTags();
+      }
     }
   }
 

clients/cli/src/test/java/org/apache/gravitino/cli/TestMain.java

@@ -143,4 +143,24 @@ public void catalogWithOneArg() throws ParseException {
     String entity = Main.resolveEntity(line);
     assertEquals(CommandEntities.CATALOG, entity);
   }
+
+  @Test
+  @SuppressWarnings("DefaultCharset")
+  public void CreateTagWithNoTag() {
+    String[] args = {"tag", "create", "--metalake", "metalake_test_no_tag"};
+
+    Main.main(args);
+
+    assertTrue(errContent.toString().contains(ErrorMessages.TAG_EMPTY)); // Expect error
+  }
+
+  @Test
+  @SuppressWarnings("DefaultCharset")
+  public void DeleteTagWithNoTag() {
+    String[] args = {"tag", "delete", "--metalake", "metalake_test_no_tag", "-f"};
+
+    Main.main(args);
+
+    assertTrue(errContent.toString().contains(ErrorMessages.TAG_EMPTY)); // Expect error
+  }
 }