From f9631f3b18fad89e6c57fdd2ad3a841e4fd73040 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Thu, 26 Sep 2024 13:27:27 +0700
Subject: [PATCH 01/10] coding

---
 tests/systemtests/cli.go      |   2 +
 tests/systemtests/gov_test.go | 227 ++++++++++++++++++++++++++++++++++
 tests/systemtests/system.go   |   9 ++
 tests/systemtests/test        |   6 +
 tests/systemtests/test.json   |   0
 5 files changed, 244 insertions(+)
 create mode 100644 tests/systemtests/gov_test.go
 create mode 100644 tests/systemtests/test
 create mode 100644 tests/systemtests/test.json

diff --git a/tests/systemtests/cli.go b/tests/systemtests/cli.go
index d7ac2368ae79..117e42588503 100644
--- a/tests/systemtests/cli.go
+++ b/tests/systemtests/cli.go
@@ -244,6 +244,8 @@ func (c CLIWrapper) runWithInput(args []string, input io.Reader) (output string,
 		return cmd.CombinedOutput()
 	}()
 	gotOut = filterProtoNoise(gotOut)
+	fmt.Println("assertError", gotErr)
+	fmt.Println("assertError", string(gotOut))
 	ok = c.assertErrorFn(c.t, gotErr, string(gotOut))
 	return strings.TrimSpace(string(gotOut)), ok
 }
diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
new file mode 100644
index 000000000000..08156b07400b
--- /dev/null
+++ b/tests/systemtests/gov_test.go
@@ -0,0 +1,227 @@
+//go:build system_test
+
+package systemtests
+
+import (
+	"encoding/base64"
+	"fmt"
+	"os"
+	"testing"
+
+	"cosmossdk.io/math"
+	"github.com/cosmos/cosmos-sdk/client/flags"
+	"github.com/cosmos/cosmos-sdk/testutil"
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/stretchr/testify/require"
+	"github.com/tidwall/gjson"
+)
+
+func TestSubmitProposal(t *testing.T) {
+	// given a running chain
+
+	sut.ResetChain(t)
+	cli := NewCLIWrapper(t, sut, verbose)
+
+	// get validator address
+	valAddr := gjson.Get(cli.Keys("keys", "list"), "0.address").String()
+	require.NotEmpty(t, valAddr)
+
+	sut.StartChain(t)
+
+	// get gov module address
+	resp := cli.CustomQuery("q", "auth", "module-account", "gov")
+	govAddress := gjson.Get(resp, "account.value.address").String()
+
+	invalidProp := `{
+	"title": "",
+	"description": "Where is the title!?",
+	"type": "Text",
+	"deposit": "-324foocoin"
+}`
+	invalidPropFile := testutil.WriteToNewTempFile(t, invalidProp)
+	os.WriteFile("test", []byte(invalidProp), 0o600)
+	// defer invalidPropFile.Close()
+
+
+	// Create a valid new proposal JSON.
+	propMetadata := []byte{42}
+	validProp := fmt.Sprintf(`
+{
+	"messages": [
+		{
+			"@type": "/cosmos.gov.v1.MsgExecLegacyContent",
+			"authority": "%s",
+			"content": {
+				"@type": "/cosmos.gov.v1beta1.TextProposal",
+				"title": "My awesome title",
+				"description": "My awesome description"
+			}
+		}
+	],
+	"title": "My awesome title",
+	"summary": "My awesome description",
+	"metadata": "%s",
+	"deposit": "%s"
+}`, govAddress, base64.StdEncoding.EncodeToString(propMetadata), sdk.NewCoin("stake", math.NewInt(100000)))
+	validPropFile := testutil.WriteToNewTempFile(t, validProp)
+	// defer validPropFile.Close()
+
+	testCases := []struct {
+		name         string
+		args         []string
+		expectErr    bool
+	}{
+		{
+			"invalid proposal",
+			[]string{
+				"tx", "gov", "submit-proposal",
+				invalidPropFile.Name(),
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			true,
+		},
+		{
+			"valid proposal",
+			[]string{
+				"tx", "gov", "submit-proposal",
+				validPropFile.Name(),
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false,
+		},
+	}
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			if tc.expectErr {
+				rsp := cli.Run(tc.args...)
+				fmt.Println("respppp", rsp)
+				txResult, found := cli.AwaitTxCommitted(rsp)
+				fmt.Println("resulttttt", txResult)
+				require.False(t, found)
+			} else {
+				rsp := cli.Run(tc.args...)
+				txResult, found := cli.AwaitTxCommitted(rsp)
+				require.True(t, found)
+				RequireTxSuccess(t, txResult)
+			}
+		})
+	}
+}
+
+func TestSubmitLegacyProposal(t *testing.T) {
+	// given a running chain
+
+	sut.ResetChain(t)
+	cli := NewCLIWrapper(t, sut, verbose)
+
+	// get validator address
+	valAddr := gjson.Get(cli.Keys("keys", "list"), "0.address").String()
+	require.NotEmpty(t, valAddr)
+
+	sut.StartChain(t)
+
+	// get gov module address
+	// resp := cli.CustomQuery("q", "auth", "module-account", "gov")
+	// govAddress := gjson.Get(resp, "account.value.address").String()
+
+	invalidProp := `{
+	"title": "",
+		"description": "Where is the title!?",
+		"type": "Text",
+	"deposit": "-324foocoin"
+	}`
+	invalidPropFile := testutil.WriteToNewTempFile(t, invalidProp)
+	defer invalidPropFile.Close()
+
+
+	validProp := fmt.Sprintf(`{
+		"title": "Text Proposal",
+		  "description": "Hello, World!",
+		  "type": "Text",
+		"deposit": "%s"
+	  }`, sdk.NewCoin("stake", math.NewInt(154310)))
+	  validPropFile := testutil.WriteToNewTempFile(t, validProp)
+	  defer validPropFile.Close()
+
+	testCases := []struct {
+		name         string
+		args         []string
+		expectErr    bool
+	}{
+		{
+			"invalid proposal (file)",
+			[]string{
+				"tx", "gov", "submit-legacy-proposal",
+				fmt.Sprintf("--%s=%s", "proposal", invalidPropFile.Name()), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			true,
+		},
+		{
+			"invalid proposal",
+			[]string{
+				"tx", "gov", "submit-legacy-proposal",
+				fmt.Sprintf("--%s='Where is the title!?'", "description"),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "type", "Text"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10000)).String()),
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false,
+		},
+		{
+			"valid transaction (file)",
+			[]string{
+				"tx", "gov", "submit-legacy-proposal",
+				fmt.Sprintf("--%s=%s", "proposal", validPropFile.Name()),
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false,
+		},
+		{
+			"valid transaction",
+			[]string{
+				"tx", "gov", "submit-legacy-proposal",
+				fmt.Sprintf("--%s='Text Proposal'", "title"),
+				fmt.Sprintf("--%s='Where is the title!?'", "description"),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "type", "Text"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(100000)).String()),
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false,
+		},
+	}
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			out, ok := cli.run(tc.args)			
+			if tc.expectErr {
+				require.Equal(t, ok, false)
+			} else {
+				require.Equal(t, ok, true)
+				resp, committed := cli.AwaitTxCommitted(out, DefaultWaitTime)
+				require.Equal(t, cli.expTXCommitted, committed, "expected tx committed: %v", cli.expTXCommitted)
+
+				txResult, found := cli.AwaitTxCommitted(resp)
+				require.True(t, found)
+				RequireTxSuccess(t, txResult)
+			}
+		})
+	}
+}
diff --git a/tests/systemtests/system.go b/tests/systemtests/system.go
index d76d482b89e2..d2cc174ec5cd 100644
--- a/tests/systemtests/system.go
+++ b/tests/systemtests/system.go
@@ -132,6 +132,15 @@ func (s *SystemUnderTest) SetupChain() {
 	if err != nil {
 		panic(fmt.Sprintf("failed set block max gas: %s", err))
 	}
+	// Short period for gov
+	genesisBz, err = sjson.SetRawBytes(genesisBz, "app_state.gov.params.max_deposit_period", []byte(fmt.Sprintf(`"%s"`, "20s")))
+	if err != nil {
+		panic(fmt.Sprintf("failed set block max gas: %s", err))
+	}
+	genesisBz, err = sjson.SetRawBytes(genesisBz, "app_state.gov.params.voting_period", []byte(fmt.Sprintf(`"%s"`, "8s")))
+	if err != nil {
+		panic(fmt.Sprintf("failed set block max gas: %s", err))
+	}
 	s.withEachNodeHome(func(i int, home string) {
 		if err := saveGenesis(home, genesisBz); err != nil {
 			panic(err)
diff --git a/tests/systemtests/test b/tests/systemtests/test
new file mode 100644
index 000000000000..e0a9fd6936f1
--- /dev/null
+++ b/tests/systemtests/test
@@ -0,0 +1,6 @@
+{
+	"title": "",
+	"description": "Where is the title!?",
+	"type": "Text",
+	"deposit": "-324foocoin"
+}
\ No newline at end of file
diff --git a/tests/systemtests/test.json b/tests/systemtests/test.json
new file mode 100644
index 000000000000..e69de29bb2d1

From d644a6adb80f172af5b3b99547a7e35b8f29e7bc Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Thu, 26 Sep 2024 13:27:44 +0700
Subject: [PATCH 02/10] coding

---
 scripts/local-testnet.sh   | 2 +-
 x/gov/keeper/msg_server.go | 1 +
 x/gov/keeper/proposal.go   | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/scripts/local-testnet.sh b/scripts/local-testnet.sh
index c73710dbab1e..04ed9919521f 100644
--- a/scripts/local-testnet.sh
+++ b/scripts/local-testnet.sh
@@ -10,7 +10,7 @@ SIMD="$ROOT/build/simdv2"
 
 COSMOS_BUILD_OPTIONS=v2 make build     
 
-$SIMD testnet init-files --chain-id=testing --output-dir="$HOME/.testnet" --validator-count=3 --keyring-backend=test --minimum-gas-prices=0.000001stake --commit-timeout=900ms --single-host
+$SIMD testnet init-files --chain-id=testing --output-dir="$HOME/.testnet" --validator-count=3 --keyring-backend=test --server.minimum-gas-prices=0.000001stake --commit-timeout=900ms --single-host
 
 $SIMD start --log_level=info --home "$HOME/.testnet/node0/simdv2" &
 $SIMD start --log_level=info --home "$HOME/.testnet/node1/simdv2" &
diff --git a/x/gov/keeper/msg_server.go b/x/gov/keeper/msg_server.go
index 8dca8c094fe1..eda432f255b1 100644
--- a/x/gov/keeper/msg_server.go
+++ b/x/gov/keeper/msg_server.go
@@ -204,6 +204,7 @@ func (k msgServer) ExecLegacyContent(ctx context.Context, msg *v1.MsgExecLegacyC
 	}
 
 	// Ensure that the content has a respective handler
+	fmt.Println("legacyRouter", k.Keeper.legacyRouter)
 	if !k.Keeper.legacyRouter.HasRoute(content.ProposalRoute()) {
 		return nil, errors.Wrap(govtypes.ErrNoProposalHandlerExists, content.ProposalRoute())
 	}
diff --git a/x/gov/keeper/proposal.go b/x/gov/keeper/proposal.go
index c68ef10c165f..66d09ed07437 100644
--- a/x/gov/keeper/proposal.go
+++ b/x/gov/keeper/proposal.go
@@ -127,6 +127,7 @@ func (k Keeper) SubmitProposal(ctx context.Context, messages []sdk.Msg, metadata
 		}
 
 		// Ensure that the content has a respective handler
+		fmt.Println(k.legacyRouter)
 		if !k.legacyRouter.HasRoute(content.ProposalRoute()) {
 			return v1.Proposal{}, errorsmod.Wrap(types.ErrNoProposalHandlerExists, content.ProposalRoute())
 		}

From 8200720a826b6958ee1a9cc3028e3feda4fad66b Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Thu, 26 Sep 2024 17:32:18 +0700
Subject: [PATCH 03/10] test submit gov

---
 tests/systemtests/gov_test.go | 68 ++++++++++++++++++++---------------
 1 file changed, 39 insertions(+), 29 deletions(-)

diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index 08156b07400b..9fcf4c12d351 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -12,6 +12,7 @@ import (
 	"github.com/cosmos/cosmos-sdk/client/flags"
 	"github.com/cosmos/cosmos-sdk/testutil"
 	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"github.com/tidwall/gjson"
 )
@@ -40,8 +41,7 @@ func TestSubmitProposal(t *testing.T) {
 }`
 	invalidPropFile := testutil.WriteToNewTempFile(t, invalidProp)
 	os.WriteFile("test", []byte(invalidProp), 0o600)
-	// defer invalidPropFile.Close()
-
+	defer invalidPropFile.Close()
 
 	// Create a valid new proposal JSON.
 	propMetadata := []byte{42}
@@ -64,12 +64,13 @@ func TestSubmitProposal(t *testing.T) {
 	"deposit": "%s"
 }`, govAddress, base64.StdEncoding.EncodeToString(propMetadata), sdk.NewCoin("stake", math.NewInt(100000)))
 	validPropFile := testutil.WriteToNewTempFile(t, validProp)
-	// defer validPropFile.Close()
+	defer validPropFile.Close()
 
 	testCases := []struct {
-		name         string
-		args         []string
-		expectErr    bool
+		name      string
+		args      []string
+		expectErr bool
+		errMsg    string
 	}{
 		{
 			"invalid proposal",
@@ -82,6 +83,7 @@ func TestSubmitProposal(t *testing.T) {
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
 			true,
+			"invalid character in coin string",
 		},
 		{
 			"valid proposal",
@@ -94,17 +96,19 @@ func TestSubmitProposal(t *testing.T) {
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
 			false,
+			"",
 		},
 	}
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
 			if tc.expectErr {
-				rsp := cli.Run(tc.args...)
-				fmt.Println("respppp", rsp)
-				txResult, found := cli.AwaitTxCommitted(rsp)
-				fmt.Println("resulttttt", txResult)
-				require.False(t, found)
+				assertOutput := func(_ assert.TestingT, gotErr error, gotOutputs ...interface{}) bool {
+					require.Contains(t, gotOutputs[0], tc.errMsg)
+					return false
+				}
+
+				cli.WithRunErrorMatcher(assertOutput).Run(tc.args...)
 			} else {
 				rsp := cli.Run(tc.args...)
 				txResult, found := cli.AwaitTxCommitted(rsp)
@@ -140,20 +144,20 @@ func TestSubmitLegacyProposal(t *testing.T) {
 	invalidPropFile := testutil.WriteToNewTempFile(t, invalidProp)
 	defer invalidPropFile.Close()
 
-
 	validProp := fmt.Sprintf(`{
 		"title": "Text Proposal",
 		  "description": "Hello, World!",
 		  "type": "Text",
 		"deposit": "%s"
 	  }`, sdk.NewCoin("stake", math.NewInt(154310)))
-	  validPropFile := testutil.WriteToNewTempFile(t, validProp)
-	  defer validPropFile.Close()
+	validPropFile := testutil.WriteToNewTempFile(t, validProp)
+	defer validPropFile.Close()
 
 	testCases := []struct {
-		name         string
-		args         []string
-		expectErr    bool
+		name      string
+		args      []string
+		expectErr bool
+		errMsg    string
 	}{
 		{
 			"invalid proposal (file)",
@@ -165,19 +169,21 @@ func TestSubmitLegacyProposal(t *testing.T) {
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
 			true,
+			"proposal title is required",
 		},
 		{
 			"invalid proposal",
 			[]string{
 				"tx", "gov", "submit-legacy-proposal",
-				fmt.Sprintf("--%s='Where is the title!?'", "description"),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", "type", "Text"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
 				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10000)).String()),
 				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
-			false,
+			true,
+			"proposal title is required",
 		},
 		{
 			"valid transaction (file)",
@@ -190,14 +196,15 @@ func TestSubmitLegacyProposal(t *testing.T) {
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
 			false,
+			"",
 		},
 		{
 			"valid transaction",
 			[]string{
 				"tx", "gov", "submit-legacy-proposal",
 				fmt.Sprintf("--%s='Text Proposal'", "title"),
-				fmt.Sprintf("--%s='Where is the title!?'", "description"),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", "type", "Text"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
 				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(100000)).String()),
 				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
@@ -205,20 +212,23 @@ func TestSubmitLegacyProposal(t *testing.T) {
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
 			},
 			false,
+			"",
 		},
 	}
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			out, ok := cli.run(tc.args)			
 			if tc.expectErr {
-				require.Equal(t, ok, false)
-			} else {
-				require.Equal(t, ok, true)
-				resp, committed := cli.AwaitTxCommitted(out, DefaultWaitTime)
-				require.Equal(t, cli.expTXCommitted, committed, "expected tx committed: %v", cli.expTXCommitted)
+				assertOutput := func(_ assert.TestingT, gotErr error, gotOutputs ...interface{}) bool {
+					fmt.Println("gotOut", gotOutputs)
+					require.Contains(t, gotOutputs[0], tc.errMsg)
+					return false
+				}
 
-				txResult, found := cli.AwaitTxCommitted(resp)
+				cli.WithRunErrorMatcher(assertOutput).Run(tc.args...)
+			} else {
+				rsp := cli.Run(tc.args...)
+				txResult, found := cli.AwaitTxCommitted(rsp)
 				require.True(t, found)
 				RequireTxSuccess(t, txResult)
 			}

From 97e20e0bd3cb012d64380a992cede9e5bb8808f2 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Sun, 29 Sep 2024 22:16:39 +0700
Subject: [PATCH 04/10] vote tests

---
 tests/systemtests/gov_test.go | 140 +++++++++++++++++++++++++++++++++-
 1 file changed, 136 insertions(+), 4 deletions(-)

diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index 9fcf4c12d351..9318a87e16b4 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -131,10 +131,6 @@ func TestSubmitLegacyProposal(t *testing.T) {
 
 	sut.StartChain(t)
 
-	// get gov module address
-	// resp := cli.CustomQuery("q", "auth", "module-account", "gov")
-	// govAddress := gjson.Get(resp, "account.value.address").String()
-
 	invalidProp := `{
 	"title": "",
 		"description": "Where is the title!?",
@@ -235,3 +231,139 @@ func TestSubmitLegacyProposal(t *testing.T) {
 		})
 	}
 }
+
+func TestNewCmdWeightedVote(t *testing.T) {
+	// given a running chain
+
+	sut.ResetChain(t)
+	cli := NewCLIWrapper(t, sut, verbose)
+
+	// get validator address
+	valAddr := gjson.Get(cli.Keys("keys", "list"), "0.address").String()
+	require.NotEmpty(t, valAddr)
+
+	sut.StartChain(t)
+
+	// Submit a new proposal for voting
+	proposalArgs := []string{
+		"tx", "gov", "submit-legacy-proposal",
+		fmt.Sprintf("--%s='Text Proposal'", "title"),
+		fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10_000_000)).String()),
+		fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+		fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+		fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+		fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String())}
+	rsp := cli.Run(proposalArgs...)
+	txResult, found := cli.AwaitTxCommitted(rsp)
+	require.True(t, found)
+	RequireTxSuccess(t, txResult)
+
+	proposalsResp := cli.CustomQuery("q", "gov", "proposals")
+	proposals := gjson.Get(proposalsResp, "proposals.#.id").Array()
+	require.NotEmpty(t, proposals)
+
+	proposal1 := cli.CustomQuery("q", "gov", "proposal", "1")
+	fmt.Println("first proposal", proposal1)
+
+	testCases := []struct {
+		name         string
+		args         []string
+		expectErr    bool
+		expectedCode uint32
+		broadcasted  bool
+		errMsg       string
+	}{
+		{
+			"vote for invalid proposal",
+			[]string{
+				"tx", "gov", "weighted-vote",
+				"10",
+				"yes",
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			true, 3, true,
+			"inactive proposal",
+		},
+		{
+			"valid vote",
+			[]string{
+				"tx", "gov", "weighted-vote",
+				"1",
+				"yes",
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false, 0, true,
+			"",
+		},
+		{
+			"valid vote with metadata",
+			[]string{
+				"tx", "gov", "weighted-vote",
+				"1",
+				"yes",
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--metadata=%s", "AQ=="),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false, 0, true,
+			"",
+		},
+		{
+			"invalid valid split vote string",
+			[]string{
+				"tx", "gov", "weighted-vote",
+				"1",
+				"yes/0.6,no/0.3,abstain/0.05,no_with_veto/0.05",
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			true, 0, false,
+			"is not a valid vote option",
+		},
+		{
+			"valid split vote",
+			[]string{
+				"tx", "gov", "weighted-vote",
+				"1",
+				"yes=0.6,no=0.3,abstain=0.05,no_with_veto=0.05",
+				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
+			},
+			false, 0, true,
+			"",
+		},
+	}
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			if !tc.broadcasted {
+				assertOutput := func(_ assert.TestingT, gotErr error, gotOutputs ...interface{}) bool {
+					require.Contains(t, gotOutputs[0], tc.errMsg)
+					return false
+				}
+				cli.WithRunErrorMatcher(assertOutput).Run(tc.args...)
+			} else {
+				rsp := cli.Run(tc.args...)
+				if tc.expectErr {
+					RequireTxFailure(t, rsp)
+				} else {
+					cli.AwaitTxCommitted(rsp)
+				}
+			}
+		})
+	}
+}

From 374f231af8bec72d30e8e46042d99f7cf9e3aa2a Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 15:52:17 +0700
Subject: [PATCH 05/10] all tests

---
 tests/systemtests/gov_test.go | 44 +++++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index 9318a87e16b4..c209cd3517fb 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"os"
 	"testing"
+	"time"
 
 	"cosmossdk.io/math"
 	"github.com/cosmos/cosmos-sdk/client/flags"
@@ -367,3 +368,46 @@ func TestNewCmdWeightedVote(t *testing.T) {
 		})
 	}
 }
+
+func TestQueryDeposit(t *testing.T) {
+	// given a running chain
+
+	sut.ResetChain(t)
+	cli := NewCLIWrapper(t, sut, verbose)
+
+	// get validator address
+	valAddr := gjson.Get(cli.Keys("keys", "list"), "0.address").String()
+	require.NotEmpty(t, valAddr)
+
+	sut.StartChain(t)
+
+	// Submit a new proposal for voting
+	proposalArgs := []string{
+		"tx", "gov", "submit-legacy-proposal",
+		fmt.Sprintf("--%s='Text Proposal'", "title"),
+		fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10_000_000)).String()),
+		fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
+		fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
+		fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
+		fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String())}
+	rsp := cli.Run(proposalArgs...)
+	txResult, found := cli.AwaitTxCommitted(rsp)
+	require.True(t, found)
+	RequireTxSuccess(t, txResult)
+
+	// Query initial deposit
+	resp := cli.CustomQuery("q", "gov", "deposit", "1", valAddr)
+	depositAmount := gjson.Get(resp, "deposit.amount.0.amount").Int()
+	require.Equal(t, depositAmount, int64(10_000_000))
+
+	resp = cli.CustomQuery("q", "gov", "deposits", "1")
+	deposits := gjson.Get(resp, "deposits").Array()
+	require.Equal(t, len(deposits), 1)
+
+	time.Sleep(time.Second * 8)
+	resp = cli.CustomQuery("q", "gov", "deposits", "1")
+	deposits = gjson.Get(resp, "deposits").Array()
+	require.Equal(t, len(deposits), 0)
+}

From 39a78bf349b6b2f920666f2ede783f415ab225f8 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 15:57:55 +0700
Subject: [PATCH 06/10] clean up

---
 tests/systemtests/cli.go      | 2 --
 tests/systemtests/gov_test.go | 2 +-
 tests/systemtests/system.go   | 4 ----
 tests/systemtests/test        | 6 ------
 tests/systemtests/test.json   | 0
 x/gov/keeper/msg_server.go    | 1 -
 x/gov/keeper/proposal.go      | 1 -
 7 files changed, 1 insertion(+), 15 deletions(-)
 delete mode 100644 tests/systemtests/test
 delete mode 100644 tests/systemtests/test.json

diff --git a/tests/systemtests/cli.go b/tests/systemtests/cli.go
index 117e42588503..d7ac2368ae79 100644
--- a/tests/systemtests/cli.go
+++ b/tests/systemtests/cli.go
@@ -244,8 +244,6 @@ func (c CLIWrapper) runWithInput(args []string, input io.Reader) (output string,
 		return cmd.CombinedOutput()
 	}()
 	gotOut = filterProtoNoise(gotOut)
-	fmt.Println("assertError", gotErr)
-	fmt.Println("assertError", string(gotOut))
 	ok = c.assertErrorFn(c.t, gotErr, string(gotOut))
 	return strings.TrimSpace(string(gotOut)), ok
 }
diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index c209cd3517fb..1d83e9daf42b 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -1,4 +1,4 @@
-//go:build system_test
+//go:build !system_test
 
 package systemtests
 
diff --git a/tests/systemtests/system.go b/tests/systemtests/system.go
index d2cc174ec5cd..028645956d85 100644
--- a/tests/systemtests/system.go
+++ b/tests/systemtests/system.go
@@ -133,10 +133,6 @@ func (s *SystemUnderTest) SetupChain() {
 		panic(fmt.Sprintf("failed set block max gas: %s", err))
 	}
 	// Short period for gov
-	genesisBz, err = sjson.SetRawBytes(genesisBz, "app_state.gov.params.max_deposit_period", []byte(fmt.Sprintf(`"%s"`, "20s")))
-	if err != nil {
-		panic(fmt.Sprintf("failed set block max gas: %s", err))
-	}
 	genesisBz, err = sjson.SetRawBytes(genesisBz, "app_state.gov.params.voting_period", []byte(fmt.Sprintf(`"%s"`, "8s")))
 	if err != nil {
 		panic(fmt.Sprintf("failed set block max gas: %s", err))
diff --git a/tests/systemtests/test b/tests/systemtests/test
deleted file mode 100644
index e0a9fd6936f1..000000000000
--- a/tests/systemtests/test
+++ /dev/null
@@ -1,6 +0,0 @@
-{
-	"title": "",
-	"description": "Where is the title!?",
-	"type": "Text",
-	"deposit": "-324foocoin"
-}
\ No newline at end of file
diff --git a/tests/systemtests/test.json b/tests/systemtests/test.json
deleted file mode 100644
index e69de29bb2d1..000000000000
diff --git a/x/gov/keeper/msg_server.go b/x/gov/keeper/msg_server.go
index eda432f255b1..8dca8c094fe1 100644
--- a/x/gov/keeper/msg_server.go
+++ b/x/gov/keeper/msg_server.go
@@ -204,7 +204,6 @@ func (k msgServer) ExecLegacyContent(ctx context.Context, msg *v1.MsgExecLegacyC
 	}
 
 	// Ensure that the content has a respective handler
-	fmt.Println("legacyRouter", k.Keeper.legacyRouter)
 	if !k.Keeper.legacyRouter.HasRoute(content.ProposalRoute()) {
 		return nil, errors.Wrap(govtypes.ErrNoProposalHandlerExists, content.ProposalRoute())
 	}
diff --git a/x/gov/keeper/proposal.go b/x/gov/keeper/proposal.go
index 66d09ed07437..c68ef10c165f 100644
--- a/x/gov/keeper/proposal.go
+++ b/x/gov/keeper/proposal.go
@@ -127,7 +127,6 @@ func (k Keeper) SubmitProposal(ctx context.Context, messages []sdk.Msg, metadata
 		}
 
 		// Ensure that the content has a respective handler
-		fmt.Println(k.legacyRouter)
 		if !k.legacyRouter.HasRoute(content.ProposalRoute()) {
 			return v1.Proposal{}, errorsmod.Wrap(types.ErrNoProposalHandlerExists, content.ProposalRoute())
 		}

From f917fb8e1e591b6b3e7aac7cbdd3abe8931af906 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 15:59:28 +0700
Subject: [PATCH 07/10] clean up

---
 scripts/local-testnet.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/local-testnet.sh b/scripts/local-testnet.sh
index 04ed9919521f..c73710dbab1e 100644
--- a/scripts/local-testnet.sh
+++ b/scripts/local-testnet.sh
@@ -10,7 +10,7 @@ SIMD="$ROOT/build/simdv2"
 
 COSMOS_BUILD_OPTIONS=v2 make build     
 
-$SIMD testnet init-files --chain-id=testing --output-dir="$HOME/.testnet" --validator-count=3 --keyring-backend=test --server.minimum-gas-prices=0.000001stake --commit-timeout=900ms --single-host
+$SIMD testnet init-files --chain-id=testing --output-dir="$HOME/.testnet" --validator-count=3 --keyring-backend=test --minimum-gas-prices=0.000001stake --commit-timeout=900ms --single-host
 
 $SIMD start --log_level=info --home "$HOME/.testnet/node0/simdv2" &
 $SIMD start --log_level=info --home "$HOME/.testnet/node1/simdv2" &

From 985a32d1c2369922ef768ad009643e08b3ba15f2 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 16:04:21 +0700
Subject: [PATCH 08/10] tag

---
 tests/systemtests/gov_test.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index 1d83e9daf42b..c209cd3517fb 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -1,4 +1,4 @@
-//go:build !system_test
+//go:build system_test
 
 package systemtests
 

From 6a70abd430d19ce24b18dc1003efa7a6b3f9cc21 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 16:05:46 +0700
Subject: [PATCH 09/10] remove nolint cmt

---
 tests/systemtests/gov_test.go | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index c209cd3517fb..d89efc4ee314 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -160,7 +160,7 @@ func TestSubmitLegacyProposal(t *testing.T) {
 			"invalid proposal (file)",
 			[]string{
 				"tx", "gov", "submit-legacy-proposal",
-				fmt.Sprintf("--%s=%s", "proposal", invalidPropFile.Name()), //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s=%s", "proposal", invalidPropFile.Name()),
 				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
 				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin("stake", math.NewInt(10))).String()),
@@ -172,8 +172,8 @@ func TestSubmitLegacyProposal(t *testing.T) {
 			"invalid proposal",
 			[]string{
 				"tx", "gov", "submit-legacy-proposal",
-				fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s='Where is the title!?'", "description"),
+				fmt.Sprintf("--%s=%s", "type", "Text"),
 				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10000)).String()),
 				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
@@ -200,8 +200,8 @@ func TestSubmitLegacyProposal(t *testing.T) {
 			[]string{
 				"tx", "gov", "submit-legacy-proposal",
 				fmt.Sprintf("--%s='Text Proposal'", "title"),
-				fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+				fmt.Sprintf("--%s='Where is the title!?'", "description"),
+				fmt.Sprintf("--%s=%s", "type", "Text"),
 				fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(100000)).String()),
 				fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
@@ -249,8 +249,8 @@ func TestNewCmdWeightedVote(t *testing.T) {
 	proposalArgs := []string{
 		"tx", "gov", "submit-legacy-proposal",
 		fmt.Sprintf("--%s='Text Proposal'", "title"),
-		fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-		fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s='Where is the title!?'", "description"),
+		fmt.Sprintf("--%s=%s", "type", "Text"),
 		fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10_000_000)).String()),
 		fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 		fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
@@ -385,8 +385,8 @@ func TestQueryDeposit(t *testing.T) {
 	proposalArgs := []string{
 		"tx", "gov", "submit-legacy-proposal",
 		fmt.Sprintf("--%s='Text Proposal'", "title"),
-		fmt.Sprintf("--%s='Where is the title!?'", "description"), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-		fmt.Sprintf("--%s=%s", "type", "Text"),                    //nolint:staticcheck // we are intentionally using a deprecated flag here.
+		fmt.Sprintf("--%s='Where is the title!?'", "description"),
+		fmt.Sprintf("--%s=%s", "type", "Text"),
 		fmt.Sprintf("--%s=%s", "deposit", sdk.NewCoin("stake", math.NewInt(10_000_000)).String()),
 		fmt.Sprintf("--%s=%s", flags.FlagFrom, valAddr),
 		fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),

From dab01a243cfec297c074f660b718f028ee6aa3d8 Mon Sep 17 00:00:00 2001
From: Hieu Vu <72878483+hieuvubk@users.noreply.github.com>
Date: Mon, 30 Sep 2024 16:19:01 +0700
Subject: [PATCH 10/10] migrate

---
 tests/e2e/gov/cli_test.go     |  37 ----
 tests/e2e/gov/deposits.go     | 164 ---------------
 tests/e2e/gov/tx.go           | 372 ----------------------------------
 tests/systemtests/gov_test.go |   2 -
 4 files changed, 575 deletions(-)
 delete mode 100644 tests/e2e/gov/cli_test.go
 delete mode 100644 tests/e2e/gov/deposits.go
 delete mode 100644 tests/e2e/gov/tx.go

diff --git a/tests/e2e/gov/cli_test.go b/tests/e2e/gov/cli_test.go
deleted file mode 100644
index be8e7afc82e3..000000000000
--- a/tests/e2e/gov/cli_test.go
+++ /dev/null
@@ -1,37 +0,0 @@
-//go:build e2e
-// +build e2e
-
-package gov
-
-import (
-	"testing"
-	"time"
-
-	"github.com/stretchr/testify/require"
-	"github.com/stretchr/testify/suite"
-
-	"cosmossdk.io/simapp"
-	v1 "cosmossdk.io/x/gov/types/v1"
-
-	"github.com/cosmos/cosmos-sdk/testutil/network"
-)
-
-func TestE2ETestSuite(t *testing.T) {
-	cfg := network.DefaultConfig(simapp.NewTestNetworkFixture)
-	cfg.NumValidators = 1
-	suite.Run(t, NewE2ETestSuite(cfg))
-}
-
-func TestDepositTestSuite(t *testing.T) {
-	cfg := network.DefaultConfig(simapp.NewTestNetworkFixture)
-	cfg.NumValidators = 1
-	genesisState := v1.DefaultGenesisState()
-	maxDepPeriod := time.Duration(20) * time.Second
-	votingPeriod := time.Duration(8) * time.Second
-	genesisState.Params.MaxDepositPeriod = &maxDepPeriod
-	genesisState.Params.VotingPeriod = &votingPeriod
-	bz, err := cfg.Codec.MarshalJSON(genesisState)
-	require.NoError(t, err)
-	cfg.GenesisState["gov"] = bz
-	suite.Run(t, NewDepositTestSuite(cfg))
-}
diff --git a/tests/e2e/gov/deposits.go b/tests/e2e/gov/deposits.go
deleted file mode 100644
index b685fa08bc62..000000000000
--- a/tests/e2e/gov/deposits.go
+++ /dev/null
@@ -1,164 +0,0 @@
-package gov
-
-import (
-	"fmt"
-	"strconv"
-	"time"
-
-	"github.com/stretchr/testify/suite"
-
-	"cosmossdk.io/math"
-	"cosmossdk.io/x/gov/client/cli"
-	govclitestutil "cosmossdk.io/x/gov/client/testutil"
-	v1 "cosmossdk.io/x/gov/types/v1"
-	"cosmossdk.io/x/gov/types/v1beta1"
-
-	"github.com/cosmos/cosmos-sdk/testutil"
-	"github.com/cosmos/cosmos-sdk/testutil/network"
-	sdk "github.com/cosmos/cosmos-sdk/types"
-)
-
-type DepositTestSuite struct {
-	suite.Suite
-
-	cfg     network.Config
-	network network.NetworkI
-}
-
-func NewDepositTestSuite(cfg network.Config) *DepositTestSuite {
-	return &DepositTestSuite{cfg: cfg}
-}
-
-func (s *DepositTestSuite) SetupSuite() {
-	s.T().Log("setting up test suite")
-
-	var err error
-	s.network, err = network.New(s.T(), s.T().TempDir(), s.cfg)
-	s.Require().NoError(err)
-}
-
-func (s *DepositTestSuite) submitProposal(val network.ValidatorI, initialDeposit sdk.Coin, name string) uint64 {
-	var exactArgs []string
-
-	if !initialDeposit.IsZero() {
-		exactArgs = append(exactArgs, fmt.Sprintf("--%s=%s", cli.FlagDeposit, initialDeposit.String()))
-	}
-
-	_, err := govclitestutil.MsgSubmitLegacyProposal(
-		val.GetClientCtx(),
-		val.GetAddress().String(),
-		fmt.Sprintf("Text Proposal %s", name),
-		"Where is the title!?",
-		v1beta1.ProposalTypeText,
-		exactArgs...,
-	)
-	s.Require().NoError(err)
-	s.Require().NoError(s.network.WaitForNextBlock())
-
-	// query proposals, return the last's id
-	res, err := testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals", val.GetAPIAddress()))
-	s.Require().NoError(err)
-	var proposals v1.QueryProposalsResponse
-	err = s.cfg.Codec.UnmarshalJSON(res, &proposals)
-	s.Require().NoError(err)
-	s.Require().GreaterOrEqual(len(proposals.Proposals), 1)
-
-	return proposals.Proposals[len(proposals.Proposals)-1].Id
-}
-
-func (s *DepositTestSuite) TearDownSuite() {
-	s.T().Log("tearing down test suite")
-	s.network.Cleanup()
-}
-
-func (s *DepositTestSuite) TestQueryDepositsWithInitialDeposit() {
-	val := s.network.GetValidators()[0]
-	depositAmount := sdk.NewCoin(s.cfg.BondDenom, v1.DefaultMinDepositTokens)
-
-	// submit proposal with an initial deposit
-	id := s.submitProposal(val, depositAmount, "TestQueryDepositsWithInitialDeposit")
-	proposalID := strconv.FormatUint(id, 10)
-
-	// query deposit
-	deposit := s.queryDeposit(val, proposalID, false, "")
-	s.Require().NotNil(deposit)
-	s.Require().Equal(depositAmount.String(), sdk.Coins(deposit.Deposit.Amount).String())
-
-	// query deposits
-	deposits := s.queryDeposits(val, proposalID, false, "")
-	s.Require().NotNil(deposits)
-	s.Require().Len(deposits.Deposits, 1)
-	// verify initial deposit
-	s.Require().Equal(depositAmount.String(), sdk.Coins(deposits.Deposits[0].Amount).String())
-}
-
-func (s *DepositTestSuite) TestQueryProposalAfterVotingPeriod() {
-	val := s.network.GetValidators()[0]
-	depositAmount := sdk.NewCoin(s.cfg.BondDenom, v1.DefaultMinDepositTokens.Sub(math.NewInt(50)))
-
-	// submit proposal with an initial deposit
-	id := s.submitProposal(val, depositAmount, "TestQueryProposalAfterVotingPeriod")
-	proposalID := strconv.FormatUint(id, 10)
-
-	resp, err := testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals", val.GetAPIAddress()))
-	s.Require().NoError(err)
-	var proposals v1.QueryProposalsResponse
-	err = s.cfg.Codec.UnmarshalJSON(resp, &proposals)
-	s.Require().NoError(err)
-	s.Require().GreaterOrEqual(len(proposals.Proposals), 1)
-
-	// query proposal
-	resp, err = testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals/%s", val.GetAPIAddress(), proposalID))
-	s.Require().NoError(err)
-	var proposal v1.QueryProposalResponse
-	err = s.cfg.Codec.UnmarshalJSON(resp, &proposal)
-	s.Require().NoError(err)
-
-	// waiting for deposit and voting period to end
-	time.Sleep(25 * time.Second)
-
-	// query proposal
-	resp, err = testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals/%s", val.GetAPIAddress(), proposalID))
-	s.Require().NoError(err)
-	s.Require().Contains(string(resp), fmt.Sprintf("proposal %s doesn't exist", proposalID))
-
-	// query deposits
-	deposits := s.queryDeposits(val, proposalID, false, "")
-	s.Require().Len(deposits.Deposits, 0)
-}
-
-func (s *DepositTestSuite) queryDeposits(val network.ValidatorI, proposalID string, exceptErr bool, message string) *v1.QueryDepositsResponse {
-	s.Require().NoError(s.network.WaitForNextBlock())
-
-	resp, err := testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals/%s/deposits", val.GetAPIAddress(), proposalID))
-	s.Require().NoError(err)
-
-	if exceptErr {
-		s.Require().Contains(string(resp), message)
-		return nil
-	}
-
-	var depositsRes v1.QueryDepositsResponse
-	err = val.GetClientCtx().Codec.UnmarshalJSON(resp, &depositsRes)
-	s.Require().NoError(err)
-
-	return &depositsRes
-}
-
-func (s *DepositTestSuite) queryDeposit(val network.ValidatorI, proposalID string, exceptErr bool, message string) *v1.QueryDepositResponse {
-	s.Require().NoError(s.network.WaitForNextBlock())
-
-	resp, err := testutil.GetRequest(fmt.Sprintf("%s/cosmos/gov/v1/proposals/%s/deposits/%s", val.GetAPIAddress(), proposalID, val.GetAddress().String()))
-	s.Require().NoError(err)
-
-	if exceptErr {
-		s.Require().Contains(string(resp), message)
-		return nil
-	}
-
-	var depositRes v1.QueryDepositResponse
-	err = val.GetClientCtx().Codec.UnmarshalJSON(resp, &depositRes)
-	s.Require().NoError(err)
-
-	return &depositRes
-}
diff --git a/tests/e2e/gov/tx.go b/tests/e2e/gov/tx.go
deleted file mode 100644
index 9d6f84ce15aa..000000000000
--- a/tests/e2e/gov/tx.go
+++ /dev/null
@@ -1,372 +0,0 @@
-package gov
-
-import (
-	"encoding/base64"
-	"fmt"
-
-	"github.com/cosmos/gogoproto/proto"
-	"github.com/stretchr/testify/suite"
-
-	"cosmossdk.io/math"
-	"cosmossdk.io/x/gov/client/cli"
-	govclitestutil "cosmossdk.io/x/gov/client/testutil"
-	"cosmossdk.io/x/gov/types"
-	v1 "cosmossdk.io/x/gov/types/v1"
-	"cosmossdk.io/x/gov/types/v1beta1"
-
-	"github.com/cosmos/cosmos-sdk/client/flags"
-	"github.com/cosmos/cosmos-sdk/testutil"
-	clitestutil "github.com/cosmos/cosmos-sdk/testutil/cli"
-	"github.com/cosmos/cosmos-sdk/testutil/network"
-	sdk "github.com/cosmos/cosmos-sdk/types"
-	authtypes "github.com/cosmos/cosmos-sdk/x/auth/types"
-)
-
-type E2ETestSuite struct {
-	suite.Suite
-
-	cfg     network.Config
-	network network.NetworkI
-}
-
-func NewE2ETestSuite(cfg network.Config) *E2ETestSuite {
-	return &E2ETestSuite{cfg: cfg}
-}
-
-func (s *E2ETestSuite) SetupSuite() {
-	s.T().Log("setting up e2e test suite")
-
-	var err error
-	s.network, err = network.New(s.T(), s.T().TempDir(), s.cfg)
-	s.Require().NoError(err)
-	s.Require().NoError(s.network.WaitForNextBlock())
-
-	val := s.network.GetValidators()[0]
-	clientCtx := val.GetClientCtx()
-	var resp sdk.TxResponse
-
-	// create a proposal with deposit
-	out, err := govclitestutil.MsgSubmitLegacyProposal(val.GetClientCtx(), val.GetAddress().String(),
-		"Text Proposal 1", "Where is the title!?", v1beta1.ProposalTypeText,
-		fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, v1.DefaultMinDepositTokens).String()))
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-
-	// vote for proposal
-	out, err = govclitestutil.MsgVote(val.GetClientCtx(), val.GetAddress().String(), "1", "yes")
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-
-	// create a proposal with a small deposit
-	minimumAcceptedDep := v1.DefaultMinDepositTokens.ToLegacyDec().Mul(v1.DefaultMinDepositRatio).Ceil().TruncateInt()
-	out, err = govclitestutil.MsgSubmitLegacyProposal(val.GetClientCtx(), val.GetAddress().String(),
-		"Text Proposal 2", "Where is the title!?", v1beta1.ProposalTypeText,
-		fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, minimumAcceptedDep).String()))
-
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-
-	// create a proposal3 with deposit
-	out, err = govclitestutil.MsgSubmitLegacyProposal(val.GetClientCtx(), val.GetAddress().String(),
-		"Text Proposal 3", "Where is the title!?", v1beta1.ProposalTypeText,
-		fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, v1.DefaultMinDepositTokens).String()))
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-
-	// create a proposal4 with deposit to check the cancel proposal cli tx
-	out, err = govclitestutil.MsgSubmitLegacyProposal(val.GetClientCtx(), val.GetAddress().String(),
-		"Text Proposal 4", "Where is the title!?", v1beta1.ProposalTypeText,
-		fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, v1.DefaultMinDepositTokens).String()))
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-
-	// vote for proposal3 as val
-	out, err = govclitestutil.MsgVote(val.GetClientCtx(), val.GetAddress().String(), "3", "yes=0.6,no=0.3,abstain=0.05,no_with_veto=0.05")
-	s.Require().NoError(err)
-	s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &resp), out.String())
-	s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, resp.TxHash, 0))
-}
-
-func (s *E2ETestSuite) TearDownSuite() {
-	s.T().Log("tearing down e2e test suite")
-	s.network.Cleanup()
-}
-
-func (s *E2ETestSuite) TestNewCmdSubmitProposal() {
-	val := s.network.GetValidators()[0]
-
-	// Create a legacy proposal JSON, make sure it doesn't pass this new CLI
-	// command.
-	invalidProp := `{
-	"title": "",
-	"description": "Where is the title!?",
-	"type": "Text",
-	"deposit": "-324foocoin"
-}`
-	invalidPropFile := testutil.WriteToNewTempFile(s.T(), invalidProp)
-	defer invalidPropFile.Close()
-
-	// Create a valid new proposal JSON.
-	propMetadata := []byte{42}
-	validProp := fmt.Sprintf(`
-{
-	"messages": [
-		{
-			"@type": "/cosmos.gov.v1.MsgExecLegacyContent",
-			"authority": "%s",
-			"content": {
-				"@type": "/cosmos.gov.v1beta1.TextProposal",
-				"title": "My awesome title",
-				"description": "My awesome description"
-			}
-		}
-	],
-	"title": "My awesome title",
-	"summary": "My awesome description",
-	"metadata": "%s",
-	"deposit": "%s"
-}`, authtypes.NewModuleAddress(types.ModuleName), base64.StdEncoding.EncodeToString(propMetadata), sdk.NewCoin(s.cfg.BondDenom, math.NewInt(100000)))
-	validPropFile := testutil.WriteToNewTempFile(s.T(), validProp)
-	defer validPropFile.Close()
-
-	testCases := []struct {
-		name         string
-		args         []string
-		expectErr    bool
-		expectedCode uint32
-		respType     proto.Message
-	}{
-		{
-			"invalid proposal",
-			[]string{
-				invalidPropFile.Name(),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			true, 0, nil,
-		},
-		{
-			"valid proposal",
-			[]string{
-				validPropFile.Name(),
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0, &sdk.TxResponse{},
-		},
-	}
-
-	for _, tc := range testCases {
-		s.Run(tc.name, func() {
-			cmd := cli.NewCmdSubmitProposal()
-			clientCtx := val.GetClientCtx()
-
-			out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
-			if tc.expectErr {
-				s.Require().Error(err)
-			} else {
-				s.Require().NoError(err)
-				s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
-				txResp := tc.respType.(*sdk.TxResponse)
-				s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, txResp.TxHash, tc.expectedCode))
-			}
-		})
-	}
-}
-
-func (s *E2ETestSuite) TestNewCmdSubmitLegacyProposal() {
-	val := s.network.GetValidators()[0]
-	invalidProp := `{
-	  "title": "",
-		"description": "Where is the title!?",
-		"type": "Text",
-	  "deposit": "-324foocoin"
-	}`
-	invalidPropFile := testutil.WriteToNewTempFile(s.T(), invalidProp)
-	defer invalidPropFile.Close()
-	validProp := fmt.Sprintf(`{
-	  "title": "Text Proposal",
-		"description": "Hello, World!",
-		"type": "Text",
-	  "deposit": "%s"
-	}`, sdk.NewCoin(s.cfg.BondDenom, math.NewInt(154310)))
-	validPropFile := testutil.WriteToNewTempFile(s.T(), validProp)
-	defer validPropFile.Close()
-
-	testCases := []struct {
-		name         string
-		args         []string
-		expectErr    bool
-		expectedCode uint32
-		respType     proto.Message
-	}{
-		{
-			"invalid proposal (file)",
-			[]string{
-				fmt.Sprintf("--%s=%s", cli.FlagProposal, invalidPropFile.Name()), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			true, 0, nil,
-		},
-		{
-			"invalid proposal",
-			[]string{
-				fmt.Sprintf("--%s='Where is the title!?'", cli.FlagDescription),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", cli.FlagProposalType, v1beta1.ProposalTypeText), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10000)).String()),
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			true, 0, nil,
-		},
-		{
-			"valid transaction (file)",
-			//nolint:staticcheck // we are intentionally using a deprecated flag here.
-			[]string{
-				fmt.Sprintf("--%s=%s", cli.FlagProposal, validPropFile.Name()),
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0, &sdk.TxResponse{},
-		},
-		{
-			"valid transaction",
-			[]string{
-				fmt.Sprintf("--%s='Text Proposal'", cli.FlagTitle),
-				fmt.Sprintf("--%s='Where is the title!?'", cli.FlagDescription),        //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", cli.FlagProposalType, v1beta1.ProposalTypeText), //nolint:staticcheck // we are intentionally using a deprecated flag here.
-				fmt.Sprintf("--%s=%s", cli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, math.NewInt(100000)).String()),
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0, &sdk.TxResponse{},
-		},
-	}
-
-	for _, tc := range testCases {
-		s.Run(tc.name, func() {
-			cmd := cli.NewCmdSubmitLegacyProposal()
-			clientCtx := val.GetClientCtx()
-
-			out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
-			if tc.expectErr {
-				s.Require().Error(err)
-			} else {
-				s.Require().NoError(err)
-				s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
-				txResp := tc.respType.(*sdk.TxResponse)
-				s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, txResp.TxHash, tc.expectedCode))
-			}
-		})
-	}
-}
-
-func (s *E2ETestSuite) TestNewCmdWeightedVote() {
-	val := s.network.GetValidators()[0]
-
-	testCases := []struct {
-		name         string
-		args         []string
-		expectErr    bool
-		expectedCode uint32
-	}{
-		{
-			"invalid vote",
-			[]string{},
-			true, 0,
-		},
-		{
-			"vote for invalid proposal",
-			[]string{
-				"10",
-				"yes",
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 3,
-		},
-		{
-			"valid vote",
-			[]string{
-				"1",
-				"yes",
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0,
-		},
-		{
-			"valid vote with metadata",
-			[]string{
-				"1",
-				"yes",
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--metadata=%s", "AQ=="),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0,
-		},
-		{
-			"invalid valid split vote string",
-			[]string{
-				"1",
-				"yes/0.6,no/0.3,abstain/0.05,no_with_veto/0.05",
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			true, 0,
-		},
-		{
-			"valid split vote",
-			[]string{
-				"1",
-				"yes=0.6,no=0.3,abstain=0.05,no_with_veto=0.05",
-				fmt.Sprintf("--%s=%s", flags.FlagFrom, val.GetAddress().String()),
-				fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
-				fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastSync),
-				fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, math.NewInt(10))).String()),
-			},
-			false, 0,
-		},
-	}
-
-	for _, tc := range testCases {
-		s.Run(tc.name, func() {
-			cmd := cli.NewCmdWeightedVote()
-			clientCtx := val.GetClientCtx()
-			var txResp sdk.TxResponse
-
-			out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
-
-			if tc.expectErr {
-				s.Require().Error(err)
-			} else {
-				s.Require().NoError(err)
-				s.Require().NoError(clientCtx.Codec.UnmarshalJSON(out.Bytes(), &txResp), out.String())
-				s.Require().NoError(clitestutil.CheckTxCode(s.network, clientCtx, txResp.TxHash, tc.expectedCode))
-			}
-		})
-	}
-}
diff --git a/tests/systemtests/gov_test.go b/tests/systemtests/gov_test.go
index d89efc4ee314..73bc7fd96efc 100644
--- a/tests/systemtests/gov_test.go
+++ b/tests/systemtests/gov_test.go
@@ -5,7 +5,6 @@ package systemtests
 import (
 	"encoding/base64"
 	"fmt"
-	"os"
 	"testing"
 	"time"
 
@@ -41,7 +40,6 @@ func TestSubmitProposal(t *testing.T) {
 	"deposit": "-324foocoin"
 }`
 	invalidPropFile := testutil.WriteToNewTempFile(t, invalidProp)
-	os.WriteFile("test", []byte(invalidProp), 0o600)
 	defer invalidPropFile.Close()
 
 	// Create a valid new proposal JSON.