Merge branch 'main' into carlos/issue#2266-modulesapps27-interchain-a…

…ccountscontrollertypes-paramsvalidate-suspiciously-performs-no-checks-yet-unnecessary-conversions-from-bool-to-interface-then-checks-if-value-is-a-bool-and-not-if-it-is-enabled
cosmos · Oct 5, 2022 · b840f1d · b840f1d
2 parents e18b6f2 + 8c7c655
commit b840f1d
Show file tree

Hide file tree

Showing 7 changed files with 26 additions and 27 deletions.
diff --git a/.github/workflows/e2e-manual-icad.yaml b/.github/workflows/e2e-manual-icad.yaml
@@ -24,7 +24,7 @@ on:
         default: master
         options:
           - master
-          - v0.3.3
+          - v0.3.4
           - v0.2.3
           - v0.1.4
       chain-b-tag:
@@ -34,7 +34,7 @@ on:
         type: choice
         options:
           - master
-          - v0.3.3
+          - v0.3.4
           - v0.2.3
           - v0.1.4
       relayer-tag:

diff --git a/e2e/scripts/test-matricies/icad/test-matrix.json b/e2e/scripts/test-matricies/icad/test-matrix.json
@@ -4,24 +4,24 @@
     "chain-binary": "icad",
     "tests": [
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.1.4"
       },
       {
         "chain-a-tag": "v0.1.4",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.2.3"
       },
       {
-        "chain-a-tag": "v0.3.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-a-tag": "v0.3.4",
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",
@@ -39,15 +39,15 @@
     "tests": [
       {
         "chain-a-tag": "v0.2.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.2.3"
       },
       {
-        "chain-a-tag": "v0.3.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-a-tag": "v0.3.4",
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",

diff --git a/e2e/scripts/test-matricies/main/test-matrix.json b/e2e/scripts/test-matricies/main/test-matrix.json
@@ -120,24 +120,24 @@
     "chain-binary": "icad",
     "tests": [
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.1.4"
       },
       {
         "chain-a-tag": "v0.1.4",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.2.3"
       },
       {
-        "chain-a-tag": "v0.3.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-a-tag": "v0.3.4",
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",
@@ -155,15 +155,15 @@
     "tests": [
       {
         "chain-a-tag": "v0.2.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-b-tag": "v0.3.4"
       },
       {
-        "chain-a-tag": "v0.3.3",
+        "chain-a-tag": "v0.3.4",
         "chain-b-tag": "v0.2.3"
       },
       {
-        "chain-a-tag": "v0.3.3",
-        "chain-b-tag": "v0.3.3"
+        "chain-a-tag": "v0.3.4",
+        "chain-b-tag": "v0.3.4"
       },
       {
         "chain-a-tag": "v0.2.3",

diff --git a/e2e/tests/interchain_accounts/base_test.go b/e2e/tests/interchain_accounts/base_test.go
@@ -42,7 +42,8 @@ func (s *InterchainAccountsTestSuite) RegisterInterchainAccount(ctx context.Cont
 
 // RegisterCounterPartyPayee broadcasts a MsgRegisterCounterpartyPayee message.
 func (s *InterchainAccountsTestSuite) RegisterCounterPartyPayee(ctx context.Context, chain *cosmos.CosmosChain,
-	user *ibc.Wallet, portID, channelID, relayerAddr, counterpartyPayeeAddr string) (sdk.TxResponse, error) {
+	user *ibc.Wallet, portID, channelID, relayerAddr, counterpartyPayeeAddr string,
+) (sdk.TxResponse, error) {
 	msg := feetypes.NewMsgRegisterCounterpartyPayee(portID, channelID, relayerAddr, counterpartyPayeeAddr)
 	return s.BroadcastMessages(ctx, chain, user, msg)
 }
@@ -97,7 +98,6 @@ func (s *InterchainAccountsTestSuite) TestMsgSubmitTx_SuccessfulTransfer() {
 	})
 
 	t.Run("interchain account executes a bank transfer on behalf of the corresponding owner account", func(t *testing.T) {
-
 		t.Run("fund interchain account wallet", func(t *testing.T) {
 			// fund the host account account so it has some $$ to send
 			err := chainB.SendFunds(ctx, ibctest.FaucetAccountKeyName, ibc.WalletAmount{

diff --git a/e2e/tests/interchain_accounts/incentivized_test.go b/e2e/tests/interchain_accounts/incentivized_test.go
@@ -257,7 +257,6 @@ func (s *IncentivizedInterchainAccountsTestSuite) TestMsgSubmitTx_FailedBankSend
 	})
 
 	t.Run("execute interchain account bank send through controller", func(t *testing.T) {
-
 		t.Run("register counterparty payee", func(t *testing.T) {
 			resp, err := s.RegisterCounterPartyPayee(ctx, chainB, chainBRelayerUser, channelOutput.Counterparty.PortID, channelOutput.Counterparty.ChannelID, chainBRelayerWallet.Address, chainARelayerWallet.Address)
 			s.Require().NoError(err)

diff --git a/e2e/testsuite/testsuite.go b/e2e/testsuite/testsuite.go
@@ -236,7 +236,8 @@ func (s *E2ETestSuite) BroadcastMessages(ctx context.Context, chain *cosmos.Cosm
 
 // RegisterCounterPartyPayee broadcasts a MsgRegisterCounterpartyPayee message.
 func (s *E2ETestSuite) RegisterCounterPartyPayee(ctx context.Context, chain *cosmos.CosmosChain,
-	user *ibc.Wallet, portID, channelID, relayerAddr, counterpartyPayeeAddr string) (sdk.TxResponse, error) {
+	user *ibc.Wallet, portID, channelID, relayerAddr, counterpartyPayeeAddr string,
+) (sdk.TxResponse, error) {
 	msg := feetypes.NewMsgRegisterCounterpartyPayee(portID, channelID, relayerAddr, counterpartyPayeeAddr)
 	return s.BroadcastMessages(ctx, chain, user, msg)
 }

diff --git a/modules/core/04-channel/keeper/keeper_test.go b/modules/core/04-channel/keeper/keeper_test.go
@@ -85,7 +85,6 @@ func (suite *KeeperTestSuite) TestGetAppVersion() {
 
 // TestGetAllChannelsWithPortPrefix verifies ports are filtered correctly using a port prefix.
 func (suite *KeeperTestSuite) TestGetAllChannelsWithPortPrefix() {
-
 	const (
 		secondChannelID        = "channel-1"
 		differentChannelPortID = "different-portid"