search

tendermint / spn

A blockchain to launch blockchains.
Other
112 stars 41 forks source link

Refactor and improve `TestApplyRequest` #944

Open lumtis opened 2 years ago

lumtis commented 2 years ago

Part of https://github.com/tendermint/spn/issues/855

Creating a specific issue as this particular test file should be handled in a specific PR

We should follow guidelines of https://github.com/tendermint/spn/issues/556 for TestApplyRequest in x/launch/keeper/request_test.go

The current structure using sample.AllRequestContents is hard to maintain and not visual

Consider improving TestCheckRequest as well