Add PoE staking query plugin #170
Conversation
| twasmKeeper := twasmkeeper.NewKeeper( | ||
|
|
||
| var twasmKeeper twasmkeeper.Keeper | ||
| handler := wasmkeeper.WithMessageHandlerDecorator(func(nested wasmkeeper.Messenger) wasmkeeper.Messenger { |
There was a problem hiding this comment.
If we want to ensure this is in sync with the version in app/wasm.go, then maybe we just pull this decoration into a function in app/wasm.go and call the same wrapping function from the test code.
There was a problem hiding this comment.
I see your point but we would have cyclic dependencies in app.
We don't need the full set of custom queries/messages here but only the twasm ones for genesis/ boostrap tests in this module.
There was a problem hiding this comment.
Okay. Cyclic dependencies are annoying and Go is not terribly good at handling them,
| @@ -257,7 +261,24 @@ func createTestInput( | |||
| stakingtypes.RegisterQueryServer(querier, stakingkeeper.Querier{Keeper: stakingKeeper}) | |||
| distributiontypes.RegisterQueryServer(querier, distKeeper) | |||
|
|
|||
| keeper := NewKeeper( | |||
| var keeper Keeper | |||
| handler := wasmkeeper.WithMessageHandlerDecorator(func(nested wasmkeeper.Messenger) wasmkeeper.Messenger { | |||
There was a problem hiding this comment.
Again, good to reuse a function from app/wasm.go
| example.BankKeeper.SetBalances(ctx, myOperatorAddr, sdk.NewCoins(sdk.NewCoin(types.DefaultBondDenom, sdk.NewInt(100)))) | ||
|
|
||
| var oneInt = sdk.OneInt() | ||
| specs := map[string]struct { |
There was a problem hiding this comment.
good set of queries and high-level integration test for the contracts
| reward, err := poeKeeper.DistributionContract(ctx).ValidatorOutstandingReward(ctx, delegator) | ||
| if err != nil { | ||
| return nil, err | ||
| CanRedelegate: wasmvmtypes.NewCoin(0, bondDenom), |
Resolves #96
Wasm query plugin to communicates with PoE contracts.
With this PR the wasm query setup moves into the app level so that different modules can provide their own extensions without creating dependency issues. Similar to the ante setup