Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix test-sim-import-export simulation bug #129

Closed
defihyung opened this issue Mar 3, 2021 · 1 comment
Closed

Fix test-sim-import-export simulation bug #129

defihyung opened this issue Mar 3, 2021 · 1 comment
Assignees
@defihyung
Labels
test and simulation test and simulation
Milestone
Long term conside...

Comments

@defihyung
Copy link
Contributor

defihyung commented Mar 3, 2021 •
edited
Loading

Summary of Bug

The following error occurs when running simulation test-sim-import-export test due to Cosmos SDK's capability module. This is the exact code line that fails with the panic.

Related issue: cosmos/cosmos-sdk#8503

--- FAIL: TestAppImportExport (38.00s)
panic: SetIndex requires index to not be set [recovered]
        panic: SetIndex requires index to not be set
@defihyung defihyung changed the title Fix test-sim-import-export bug Fix test-sim-import-export simulation bug Mar 3, 2021
@defihyung defihyung mentioned this issue Mar 3, 2021
Merged
9 tasks
@defihyung defihyung self-assigned this Mar 13, 2021
@defihyung defihyung added the test and simulation test and simulation label Mar 13, 2021
@dongsam dongsam added this to the Long term considerations milestone Jun 14, 2021
@jaybxyz
Copy link
Contributor

jaybxyz commented Sep 2, 2021

This is an issue related to using the old database. Make sure to use newDB instead of db that is declared above.

liquidity/app/sim_test.go

Line 153 in 02644bb

newApp := NewLiquidityApp(log.NewNopLogger(), newDB, nil, true, map[int64]bool{}, DefaultNodeHome, simapp.FlagPeriodValue, MakeEncodingConfig(), EmptyAppOptions{}, fauxMerkleModeOpt)

@jaybxyz jaybxyz closed this as completed Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@defihyung defihyung

Labels
test and simulation test and simulation
Projects
None yet
Milestone
Long term considerations
Development

No branches or pull requests
3 participants
@dongsam @jaybxyz @defihyung