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

pallet-vesting: Move tests module and create mock module #9234

Merged
4 commits merged into from
Jun 30, 2021

Conversation

emostov
Copy link
Contributor

@emostov emostov commented Jun 29, 2021

This PR moves the tests module to its own file and factors the mock runtime out of tests into its own module. This is in preparation for adding more tests to the pallet-vesting crate in #9202.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jun 29, 2021
@emostov emostov added 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 Jun 29, 2021
@emostov emostov changed the title pallet-vesting: Move tests module to seperate file pallet-vesting: Move tests module and create mock module Jun 29, 2021
Copy link
Member

@shawntabrizi shawntabrizi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs file header

@emostov emostov requested a review from coriolinus June 29, 2021 22:56
@shawntabrizi
Copy link
Member

bot merge

@ghost
Copy link

ghost commented Jun 30, 2021

Trying merge.

@ghost ghost merged commit a4c8ab9 into master Jun 30, 2021
@ghost ghost deleted the zeke-vest-refactor-tests-and-mock branch June 30, 2021 13:02
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.

4 participants