From b9511b92050cf9bfd015209bb7b217734e98e20c Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sun, 11 Dec 2022 01:56:23 +0100 Subject: [PATCH] test: use temporary folder in tests (#14246) --- simapp/app_test.go | 10 +++++----- tests/integration/store/rootmulti/rollback_test.go | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/simapp/app_test.go b/simapp/app_test.go index 539db5dfa6..a071e0868f 100644 --- a/simapp/app_test.go +++ b/simapp/app_test.go @@ -44,7 +44,7 @@ func TestSimAppExportAndBlockedAddrs(t *testing.T) { app := NewSimappWithCustomOptions(t, false, SetupOptions{ Logger: logger, DB: db, - AppOpts: simtestutil.NewAppOptionsWithFlagHome(DefaultNodeHome), + AppOpts: simtestutil.NewAppOptionsWithFlagHome(t.TempDir()), }) // BlockedAddresses returns a map of addresses in app v1 and a map of modules name in app v2. @@ -67,7 +67,7 @@ func TestSimAppExportAndBlockedAddrs(t *testing.T) { logger2 := log.NewTMLogger(log.NewSyncWriter(os.Stdout)) // Making a new app object with the db, so that initchain hasn't been called - app2 := NewSimApp(logger2, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(DefaultNodeHome)) + app2 := NewSimApp(logger2, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(t.TempDir())) _, err := app2.ExportAppStateAndValidators(false, []string{}, []string{}) require.NoError(t, err, "ExportAppStateAndValidators should not have an error") } @@ -75,7 +75,7 @@ func TestSimAppExportAndBlockedAddrs(t *testing.T) { func TestRunMigrations(t *testing.T) { db := dbm.NewMemDB() logger := log.NewTMLogger(log.NewSyncWriter(os.Stdout)) - app := NewSimApp(logger, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(DefaultNodeHome)) + app := NewSimApp(logger, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(t.TempDir())) // Create a new baseapp and configurator for the purpose of this test. bApp := baseapp.NewBaseApp(app.Name(), logger, db, app.TxConfig().TxDecoder()) @@ -209,7 +209,7 @@ func TestRunMigrations(t *testing.T) { func TestInitGenesisOnMigration(t *testing.T) { db := dbm.NewMemDB() logger := log.NewTMLogger(log.NewSyncWriter(os.Stdout)) - app := NewSimApp(logger, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(DefaultNodeHome)) + app := NewSimApp(logger, db, nil, true, simtestutil.NewAppOptionsWithFlagHome(t.TempDir())) ctx := app.NewContext(true, tmproto.Header{Height: app.LastBlockHeight()}) // Create a mock module. This module will serve as the new module we're @@ -255,7 +255,7 @@ func TestUpgradeStateOnGenesis(t *testing.T) { app := NewSimappWithCustomOptions(t, false, SetupOptions{ Logger: logger, DB: db, - AppOpts: simtestutil.NewAppOptionsWithFlagHome(DefaultNodeHome), + AppOpts: simtestutil.NewAppOptionsWithFlagHome(t.TempDir()), }) // make sure the upgrade keeper has version map in state diff --git a/tests/integration/store/rootmulti/rollback_test.go b/tests/integration/store/rootmulti/rollback_test.go index 7139790e66..950f1f05cf 100644 --- a/tests/integration/store/rootmulti/rollback_test.go +++ b/tests/integration/store/rootmulti/rollback_test.go @@ -18,7 +18,7 @@ func TestRollback(t *testing.T) { options := simapp.SetupOptions{ Logger: log.NewNopLogger(), DB: db, - AppOpts: simtestutil.NewAppOptionsWithFlagHome(simapp.DefaultNodeHome), + AppOpts: simtestutil.NewAppOptionsWithFlagHome(t.TempDir()), } app := simapp.NewSimappWithCustomOptions(t, false, options) app.Commit() @@ -46,7 +46,7 @@ func TestRollback(t *testing.T) { require.Equal(t, target, app.LastBlockHeight()) // recreate app to have clean check state - app = simapp.NewSimApp(options.Logger, options.DB, nil, true, simtestutil.NewAppOptionsWithFlagHome(simapp.DefaultNodeHome)) + app = simapp.NewSimApp(options.Logger, options.DB, nil, true, simtestutil.NewAppOptionsWithFlagHome(t.TempDir())) store = app.NewContext(true, tmproto.Header{}).KVStore(app.GetKey("bank")) require.Equal(t, []byte("value5"), store.Get([]byte("key")))