refactor: remove duplicate vesting tests (cosmos#15844)

notional-labs · Apr 14, 2023 · c4489d9 · c4489d9
1 parent a7d8477
commit c4489d9
Show file tree

Hide file tree

Showing 5 changed files with 298 additions and 240 deletions.
diff --git a/tests/e2e/auth/vesting/cli_test.go b/tests/e2e/auth/vesting/cli_test.go
diff --git a/tests/e2e/auth/vesting/suite.go b/tests/e2e/auth/vesting/suite.go
diff --git a/x/auth/vesting/client/cli/test.json b/x/auth/vesting/client/cli/test.json
@@ -0,0 +1,13 @@
+{
+  "start_time": 1625204910,
+  "period": [
+    {
+      "coins": "10test",
+      "length_seconds": 2592000
+    },
+    {
+      "coins": "10test",
+      "length_seconds": 2592000
+    }
+  ]
+}
diff --git a/x/auth/vesting/client/cli/tx.go b/x/auth/vesting/client/cli/tx.go
@@ -2,6 +2,7 @@ package cli
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"os"
 	"strconv"
@@ -61,6 +62,10 @@ timestamp.`,
 				return err
 			}
 
+			if args[1] == "" {
+				return errors.New("amount is empty")
+			}
+
 			amount, err := sdk.ParseCoinsNormalized(args[1])
 			if err != nil {
 				return err
@@ -104,6 +109,10 @@ tokens.`,
 				return err
 			}
 
+			if args[1] == "" {
+				return errors.New("amount is empty")
+			}
+
 			amount, err := sdk.ParseCoinsNormalized(args[1])
 			if err != nil {
 				return err