150d17c3fc
Co-authored-by: cool-developer <51834436+cool-develope@users.noreply.github.com> Co-authored-by: Julien Robert <julien@rbrt.fr>
42 lines
1.3 KiB
Go
42 lines
1.3 KiB
Go
package baseapp
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
coretesting "cosmossdk.io/core/testing"
|
|
"cosmossdk.io/log"
|
|
"cosmossdk.io/store"
|
|
storemetrics "cosmossdk.io/store/metrics"
|
|
)
|
|
|
|
// Ensures that error checks are performed before sealing the app.
|
|
// Please see https://github.com/cosmos/cosmos-sdk/issues/18726
|
|
func TestNilCmsCheckBeforeSeal(t *testing.T) {
|
|
app := new(BaseApp)
|
|
|
|
// 1. Invoking app.Init with a nil cms MUST not seal the app
|
|
// and should return an error firstly, which can later be reversed.
|
|
for i := 0; i < 10; i++ { // N times, the app shouldn't be sealed.
|
|
err := app.Init()
|
|
require.Error(t, err)
|
|
require.Contains(t, err.Error(), "commit multi-store must not be nil")
|
|
require.False(t, app.IsSealed(), "the app MUST not be sealed")
|
|
}
|
|
|
|
// 2. Now that we've figured out and gotten back an error, let's rectify the problem.
|
|
// and we should be able to set the commit multistore then reinvoke app.Init successfully!
|
|
db := coretesting.NewMemDB()
|
|
logger := log.NewTestLogger(t)
|
|
app.cms = store.NewCommitMultiStore(db, logger, storemetrics.NewNoOpMetrics())
|
|
err := app.Init()
|
|
require.Nil(t, err, "app.Init MUST now succeed")
|
|
require.True(t, app.IsSealed(), "the app must now be sealed")
|
|
|
|
// 3. Now we should expect that panic.
|
|
require.Panics(t, func() {
|
|
_ = app.Init()
|
|
})
|
|
}
|