diff --git a/package-lock.json b/package-lock.json index 8a7aac90fd2..347709d0e65 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1779,16 +1779,16 @@ } }, "@openzeppelin/test-helpers": { - "version": "0.5.4", - "resolved": "https://registry.npmjs.org/@openzeppelin/test-helpers/-/test-helpers-0.5.4.tgz", - "integrity": "sha512-sSjft0CcmC6Pwsd/j1uakk2MLamEH4mmphVlF5hzUUVyoxL0KGCbxQgBoLoEpUbw2M9HtFAEMJJPGccEcb9Cog==", + "version": "0.5.5", + "resolved": "https://registry.npmjs.org/@openzeppelin/test-helpers/-/test-helpers-0.5.5.tgz", + "integrity": "sha512-jTSCQojQ0Q7FBMN3Me7o0OIVuRnfHRR9TcE+ZlfbSfdqrHkFLwSfeDHSNWtQGlF1xPQR5r3iRI0ccsCrN+JblA==", "dev": true, "requires": { "@openzeppelin/contract-loader": "^0.4.0", "@truffle/contract": "^4.0.35", "ansi-colors": "^3.2.3", "chai": "^4.2.0", - "chai-bn": "^0.2.0", + "chai-bn": "^0.2.1", "ethjs-abi": "^0.2.1", "lodash.flatten": "^4.4.0", "semver": "^5.6.0", @@ -5017,9 +5017,9 @@ } }, "chai-bn": { - "version": "0.2.0", - "resolved": "https://registry.npmjs.org/chai-bn/-/chai-bn-0.2.0.tgz", - "integrity": "sha512-h+XqIFikre13N3uiZSc50PZ0VztVjuD/Gytle07EUFkbd8z31tWp37DLAsR1dPozmOLA3yu4hi3IFjJDQ5CKBg==", + "version": "0.2.1", + "resolved": "https://registry.npmjs.org/chai-bn/-/chai-bn-0.2.1.tgz", + "integrity": "sha512-01jt2gSXAw7UYFPT5K8d7HYjdXj2vyeIuE+0T/34FWzlNcVbs1JkPxRu7rYMfQnJhrHT8Nr6qjSf5ZwwLU2EYg==", "dev": true }, "chalk": { @@ -32265,8 +32265,8 @@ } }, "openzeppelin-docs-utils": { - "version": "github:OpenZeppelin/docs-utils#459f1710a07ec02cbd683cab4800c32a07aed274", - "from": "github:OpenZeppelin/docs-utils#459f1710a07ec02cbd683cab4800c32a07aed274", + "version": "github:OpenZeppelin/docs-utils#dbdcce52e45c89f67f07683c536baf4b541ba68a", + "from": "github:OpenZeppelin/docs-utils", "dev": true, "requires": { "chalk": "^3.0.0", @@ -32322,9 +32322,9 @@ "dev": true }, "minimist": { - "version": "1.2.0", - "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.0.tgz", - "integrity": "sha1-o1AIsg9BOD7sH7kU9M1d95omQoQ=", + "version": "1.2.4", + "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.4.tgz", + "integrity": "sha512-wTiNDqe4D2rbTJGZk1qcdZgFtY0/r+iuE6GDT7V0/+Gu5MLpIDm4+CssDECR79OJs/OxLPXMzdxy153b5Qy3hg==", "dev": true }, "supports-color": { diff --git a/package.json b/package.json index e8c1361f145..f70a1d3595a 100644 --- a/package.json +++ b/package.json @@ -48,7 +48,7 @@ "@openzeppelin/gsn-helpers": "^0.2.3", "@openzeppelin/gsn-provider": "^0.1.9", "@openzeppelin/test-environment": "^0.1.3", - "@openzeppelin/test-helpers": "^0.5.4", + "@openzeppelin/test-helpers": "^0.5.5", "chai": "^4.2.0", "eslint": "^6.5.1", "eslint-config-standard": "^14.1.0", diff --git a/test/access/AccessControl.test.js b/test/access/AccessControl.test.js index 27b4667ae35..193892c4015 100644 --- a/test/access/AccessControl.test.js +++ b/test/access/AccessControl.test.js @@ -49,7 +49,7 @@ describe('AccessControl', function () { it('accounts can be granted a role multiple times', async function () { await this.accessControl.grantRole(ROLE, authorized, { from: admin }); const receipt = await this.accessControl.grantRole(ROLE, authorized, { from: admin }); - //await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleGranted'); + await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleGranted'); }); }); @@ -58,7 +58,7 @@ describe('AccessControl', function () { expect(await this.accessControl.hasRole(ROLE, authorized)).to.equal(false); const receipt = await this.accessControl.revokeRole(ROLE, authorized, { from: admin }); - //await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); + await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); }); context('with granted role', function () { @@ -84,7 +84,7 @@ describe('AccessControl', function () { await this.accessControl.revokeRole(ROLE, authorized, { from: admin }); const receipt = await this.accessControl.revokeRole(ROLE, authorized, { from: admin }); - //await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); + await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); }); }); }); @@ -92,7 +92,7 @@ describe('AccessControl', function () { describe('renouncing', function () { it('roles that are not had can be renounced', async function () { const receipt = await this.accessControl.renounceRole(ROLE, authorized, { from: authorized }); - //await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); + await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); }); context('with granted role', function () { @@ -118,7 +118,7 @@ describe('AccessControl', function () { await this.accessControl.renounceRole(ROLE, authorized, { from: authorized }); const receipt = await this.accessControl.renounceRole(ROLE, authorized, { from: authorized }); - //await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); + await expectEvent.not.inTransaction(receipt.tx, AccessControlMock, 'RoleRevoked'); }); }); });