Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

mega cleanup of staking tests #9516

Merged
5 commits merged into from
Aug 11, 2021
Merged

mega cleanup of staking tests #9516

5 commits merged into from
Aug 11, 2021

Conversation

kianenigma
Copy link
Contributor

part of #6242

Initially, I wanted to just fix the fact that genesis stakers were not being checked, but this cascaded into a spiral of fixes.

No logic has changed.

@kianenigma kianenigma added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Aug 7, 2021
@kianenigma kianenigma requested review from gui1117 and emostov August 7, 2021 14:46
@kianenigma kianenigma requested a review from shawntabrizi August 7, 2021 20:04
frame/staking/src/mock.rs Outdated Show resolved Hide resolved
frame/staking/src/mock.rs Outdated Show resolved Hide resolved
frame/staking/src/tests.rs Outdated Show resolved Hide resolved
frame/staking/src/tests.rs Outdated Show resolved Hide resolved
frame/staking/src/tests.rs Outdated Show resolved Hide resolved
@kianenigma
Copy link
Contributor Author

bot merge

1 similar comment
@kianenigma
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Aug 11, 2021

Trying merge.

@ghost ghost merged commit c34fc32 into master Aug 11, 2021
@ghost ghost deleted the kiz-fix-genesis-stakers branch August 11, 2021 13:34
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants