fix: attempt at fixing flaky e2e tests (#13401)
* fix: attempt at fixing flaky e2e tests * `make format` * add links
This commit is contained in:
@@ -1,5 +1,6 @@
|
||||
name: Build SimApp
|
||||
# This workflow allows to skip the build step if the PR does not contain any changes to the code
|
||||
# See https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
|
||||
on:
|
||||
pull_request:
|
||||
paths-ignore:
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
name: Tests E2E
|
||||
# This workflow allows to skip the e2e step if the PR does not contain any changes to the code
|
||||
# See https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
|
||||
on:
|
||||
pull_request:
|
||||
paths-ignore:
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
name: Tests Integration
|
||||
# This workflow allows to skip the integration step if the PR does not contain any changes to the code
|
||||
# See https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
|
||||
on:
|
||||
pull_request:
|
||||
paths-ignore:
|
||||
|
||||
@@ -270,7 +270,6 @@ func (app *BaseApp) MountStore(key storetypes.StoreKey, typ storetypes.StoreType
|
||||
// LoadLatestVersion loads the latest application version. It will panic if
|
||||
// called more than once on a running BaseApp.
|
||||
func (app *BaseApp) LoadLatestVersion() error {
|
||||
|
||||
err := app.storeLoader(app.cms)
|
||||
if err != nil {
|
||||
return fmt.Errorf("failed to load latest version: %w", err)
|
||||
|
||||
@@ -2151,6 +2151,8 @@ func (s *IntegrationTestSuite) TestTxLeaveGroup() {
|
||||
),
|
||||
)
|
||||
s.Require().NoError(err, out.String())
|
||||
s.Require().NoError(s.network.WaitForNextBlock())
|
||||
|
||||
var txResp sdk.TxResponse
|
||||
s.Require().NoError(val.ClientCtx.Codec.UnmarshalJSON(out.Bytes(), &txResp), out.String())
|
||||
txResp, err = clitestutil.GetTxResponse(s.network, val.ClientCtx, txResp.TxHash)
|
||||
|
||||
@@ -1587,6 +1587,7 @@ func (s *IntegrationTestSuite) TestSignWithMultiSignersAminoJSON() {
|
||||
)
|
||||
require.NoError(err)
|
||||
require.NoError(s.network.WaitForNextBlock())
|
||||
require.NoError(s.network.WaitForNextBlock())
|
||||
|
||||
var txRes sdk.TxResponse
|
||||
require.NoError(val0.ClientCtx.Codec.UnmarshalJSON(res.Bytes(), &txRes))
|
||||
|
||||
Reference in New Issue
Block a user