nit(callbacks): removed unused testing func params (#4462)

Co-authored-by: Carlos Rodriguez <[email protected]>
cosmos · Aug 28, 2023 · d5e55b7 · d5e55b7
1 parent 01995a5
commit d5e55b7
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/modules/apps/callbacks/fee_transfer_test.go b/modules/apps/callbacks/fee_transfer_test.go
@@ -184,7 +184,7 @@ func (s *CallbacksTestSuite) TestIncentivizedTransferTimeoutCallbacks() {
 
 			s.ExecutePayPacketFeeMsg(fee)
 			preRelaySenderBalance := sdk.NewCoins(GetSimApp(s.chainA).BankKeeper.GetBalance(s.chainA.GetContext(), s.chainA.SenderAccount.GetAddress(), ibctesting.TestCoin.Denom))
-			s.ExecuteTransferTimeout(tc.transferMemo, 1)
+			s.ExecuteTransferTimeout(tc.transferMemo)
 
 			// after incentivizing the packets
 			s.AssertHasExecutedExpectedCallbackWithFee(tc.expCallback, tc.expSuccess, true, preRelaySenderBalance, fee)

diff --git a/modules/apps/callbacks/ica_test.go b/modules/apps/callbacks/ica_test.go
@@ -81,7 +81,7 @@ func (s *CallbacksTestSuite) TestICACallbacks() {
 		s.Run(tc.name, func() {
 			icaAddr := s.SetupICATest()
 
-			s.ExecuteICATx(icaAddr, tc.icaMemo, 1)
+			s.ExecuteICATx(icaAddr, tc.icaMemo)
 			s.AssertHasExecutedExpectedCallback(tc.expCallback, tc.expSuccess)
 		})
 	}
@@ -131,14 +131,14 @@ func (s *CallbacksTestSuite) TestICATimeoutCallbacks() {
 		s.Run(tc.name, func() {
 			icaAddr := s.SetupICATest()
 
-			s.ExecuteICATimeout(icaAddr, tc.icaMemo, 1)
+			s.ExecuteICATimeout(icaAddr, tc.icaMemo)
 			s.AssertHasExecutedExpectedCallback(tc.expCallback, tc.expSuccess)
 		})
 	}
 }
 
 // ExecuteICATx executes a stakingtypes.MsgDelegate on chainB by sending a packet containing the msg to chainB
-func (s *CallbacksTestSuite) ExecuteICATx(icaAddress, memo string, seq uint64) {
+func (s *CallbacksTestSuite) ExecuteICATx(icaAddress, memo string) {
 	timeoutTimestamp := uint64(s.chainA.GetContext().BlockTime().Add(time.Minute).UnixNano())
 	icaOwner := s.chainA.SenderAccount.GetAddress().String()
 	connectionID := s.path.EndpointA.ConnectionID
@@ -159,7 +159,7 @@ func (s *CallbacksTestSuite) ExecuteICATx(icaAddress, memo string, seq uint64) {
 }
 
 // ExecuteICATx sends and times out an ICA tx
-func (s *CallbacksTestSuite) ExecuteICATimeout(icaAddress, memo string, seq uint64) {
+func (s *CallbacksTestSuite) ExecuteICATimeout(icaAddress, memo string) {
 	relativeTimeout := uint64(1)
 	icaOwner := s.chainA.SenderAccount.GetAddress().String()
 	connectionID := s.path.EndpointA.ConnectionID

diff --git a/modules/apps/callbacks/transfer_test.go b/modules/apps/callbacks/transfer_test.go
@@ -161,7 +161,7 @@ func (s *CallbacksTestSuite) TestTransferTimeoutCallbacks() {
 	for _, tc := range testCases {
 		s.SetupTransferTest()
 
-		s.ExecuteTransferTimeout(tc.transferMemo, 1)
+		s.ExecuteTransferTimeout(tc.transferMemo)
 		s.AssertHasExecutedExpectedCallback(tc.expCallback, tc.expSuccess)
 	}
 }
@@ -206,7 +206,7 @@ func (s *CallbacksTestSuite) ExecuteTransfer(memo string) {
 
 // ExecuteTransferTimeout executes a transfer message on chainA for 100 denom.
 // This message is not relayed to chainB, and it times out on chainA.
-func (s *CallbacksTestSuite) ExecuteTransferTimeout(memo string, nextSeqRecv uint64) {
+func (s *CallbacksTestSuite) ExecuteTransferTimeout(memo string) {
 	timeoutHeight := clienttypes.GetSelfHeight(s.chainB.GetContext())
 	timeoutTimestamp := uint64(s.chainB.GetContext().BlockTime().UnixNano())