* Error handling in ERC20 and ERC721 * Added message string for require. * Fixed solhint errors. * Updated PR as per issue #1709 * changes as per #1709 and openzeppelin forum. * Changes in require statement * Changes in require statement * build pipeline fix * Changes as per @nventuro's comment. * Update revert reason strings. * Fianal update of revert reason strings. * WIP: Updating reason strings in test cases * WIP: Added changes to ERC20 and ERC721 * Fixes linting errors in *.tes.js files * Achieved 100% code coverage * Updated the test cases with shouldFail.reverting.withMessage() * Fix package-lock. * address review comments * fix linter issues * fix remaining revert reasons
55 lines
1.6 KiB
JavaScript
55 lines
1.6 KiB
JavaScript
const { BN, constants, expectEvent, shouldFail } = require('openzeppelin-test-helpers');
|
|
const { ZERO_ADDRESS } = constants;
|
|
|
|
function shouldBehaveLikeERC20Mintable (minter, [other]) {
|
|
describe('as a mintable token', function () {
|
|
describe('mint', function () {
|
|
const amount = new BN(100);
|
|
|
|
context('when the sender has minting permission', function () {
|
|
const from = minter;
|
|
|
|
context('for a zero amount', function () {
|
|
shouldMint(new BN(0));
|
|
});
|
|
|
|
context('for a non-zero amount', function () {
|
|
shouldMint(amount);
|
|
});
|
|
|
|
function shouldMint (amount) {
|
|
beforeEach(async function () {
|
|
({ logs: this.logs } = await this.token.mint(other, amount, { from }));
|
|
});
|
|
|
|
it('mints the requested amount', async function () {
|
|
(await this.token.balanceOf(other)).should.be.bignumber.equal(amount);
|
|
});
|
|
|
|
it('emits a mint and a transfer event', async function () {
|
|
expectEvent.inLogs(this.logs, 'Transfer', {
|
|
from: ZERO_ADDRESS,
|
|
to: other,
|
|
value: amount,
|
|
});
|
|
});
|
|
}
|
|
});
|
|
|
|
context('when the sender doesn\'t have minting permission', function () {
|
|
const from = other;
|
|
|
|
it('reverts', async function () {
|
|
await shouldFail.reverting.withMessage(this.token.mint(other, amount, { from }),
|
|
'MinterRole: caller does not have the Minter role'
|
|
);
|
|
});
|
|
});
|
|
});
|
|
});
|
|
}
|
|
|
|
module.exports = {
|
|
shouldBehaveLikeERC20Mintable,
|
|
};
|