Skip to content

Commit

Permalink
fix concurrency issue in go test(x/lockup) (#8768)
Browse files Browse the repository at this point in the history
* additional lockup change

* Add changelog
  • Loading branch information
mattverse authored Oct 18, 2024
1 parent b44289b commit 56ddf9d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
* [#8731](https://github.com/osmosis-labs/osmosis/pull/8731) fix: in place testnet logs
* [#8728](https://github.com/osmosis-labs/osmosis/pull/8728) fix unsupported sign-mode issue
* [#8743](https://github.com/osmosis-labs/osmosis/pull/8743) chore: bump sdk and cometbft

* [#8765, 8768](https://github.com/osmosis-labs/osmosis/pull/8765) fix concurrency issue in go test(x/lockup)
* [#8563](https://github.com/osmosis-labs/osmosis/pull/8755) [x/concentratedliquidity]: Fix Incorrect Event Emission
* [#8765](https://github.com/osmosis-labs/osmosis/pull/8765) fix concurrency issue in go test(x/lockup)

Expand Down
7 changes: 0 additions & 7 deletions x/lockup/keeper/genesis_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -176,13 +176,6 @@ func TestMarshalUnmarshalGenesis(t *testing.T) {

// Ensure no panic occurs when initializing genesis in a fresh app
assert.NotPanics(t, func() {
// Setup a new app instance
app := osmoapp.Setup(false)
ctx := app.BaseApp.NewContextLegacy(false, tmproto.Header{})
ctx = ctx.WithBlockTime(now.Add(time.Second))

// Initialize genesis with the exported state
am := lockup.NewAppModule(*app.LockupKeeper, app.AccountKeeper, app.BankKeeper)
am.InitGenesis(ctx, appCodec, genesisExported)
})
}

0 comments on commit 56ddf9d

Please sign in to comment.