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.

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

Feat: Add try_state invariants to pallets #1846

Open
wischli opened this issue May 28, 2024 · 1 comment
Open

Feat: Add try_state invariants to pallets #1846

wischli opened this issue May 28, 2024 · 1 comment
Labels
I8-enhancement An additional feature. P2-nice-to-have Issue is worth doing.

Comments

@wischli
Copy link
Contributor

wischli commented May 28, 2024

Description

So far, we have not made use of the try_state hooks which are part of the try-runtime feature. It provides checks which should ensure that the state of the pallet is consistent and correct. Usually, these checks should check all of the invariants that are expected to be held on all of the storage items of some particular pallet. This hook should not alter any storage though.

This issue should serve as an entry point and only be closed once we have try_state implemented for all pallets or decided certain pallets don't need it.

Research/based on

How will this affect the code base

  • Improves stability

What are foreseen obstacles or hurdles to overcome?

@wischli wischli added P2-nice-to-have Issue is worth doing. I8-enhancement An additional feature. labels May 28, 2024
@lemunozm
Copy link
Contributor

Love it!

I think can be quite useful for pallet-loans, checking that i.e. the portfolio can be correctly computed after a migration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I8-enhancement An additional feature. P2-nice-to-have Issue is worth doing.
Projects
None yet
Development

No branches or pull requests

2 participants