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.

Sign up for GitHub

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

Jump to bottom

[CENT-367] Rewrite unit tests to use MasterMinter #226

Merged

o-a-hudson merged 2 commits into circlefin:multi-issuer from mirathewhite:testMasterMinter

Oct 12, 2018

Contributor

mirathewhite commented Oct 11, 2018

Fixed bug in MasterMinter.sol (missing constructor).
Modified MINT p0 unit tests to run using a MintController and a MasterMinter. This will be necessary because future contracts may inherit from MintController, so we want to verify that both MintController and MasterMinter exhibit the desired behavior.

mirathewhite added 2 commits

October 11, 2018 09:47


          Fixed bug in MasterMinter.sol constructor and added MasterMinter unit…

4b9ebf3

… tests


          beforeEach sends MintController as argument to initialization function

40fdc07

o-a-hudson approved these changes

View reviewed changes

o-a-hudson merged commit 4d45c5a into circlefin:multi-issuer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment