Update Test Helpers to 0.5.6 (#2261)

* update test-helpers to 0.5.6-rc.0

* remove comment that no longer applies

* fix uses of expectEvent.notEmitted

* remove mocha only

* update to final version

* Update test/token/ERC777/ERC777.test.js

Co-authored-by: Nicolás Venturo <[email protected]>

Co-authored-by: Nicolás Venturo <[email protected]>

Author: frangio

package-lock.json

@@ -50,7 +50,7 @@
     "@openzeppelin/gsn-helpers": "^0.2.3",
     "@openzeppelin/gsn-provider": "^0.1.10",
     "@openzeppelin/test-environment": "^0.1.4",
-    "@openzeppelin/test-helpers": "^0.5.5",
+    "@openzeppelin/test-helpers": "^0.5.6",
     "chai": "^4.2.0",
     "eslint": "^6.5.1",
     "eslint-config-standard": "^14.1.1",

package.json

@@ -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');
+      expectEvent.notEmitted(receipt, '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');
+      expectEvent.notEmitted(receipt, 'RoleRevoked');
     });
 
     context('with granted role', function () {
@@ -84,15 +84,15 @@ 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');
+        expectEvent.notEmitted(receipt, 'RoleRevoked');
       });
     });
   });
 
   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');
+      expectEvent.notEmitted(receipt, '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');
+        expectEvent.notEmitted(receipt, 'RoleRevoked');
       });
     });
   });

test/access/AccessControl.test.js

@@ -60,8 +60,8 @@ describe('ERC777', function () {
       });
     });
 
-    it.skip('does not emit AuthorizedOperator events for default operators', async function () {
-      expectEvent.not.inConstructor(this.token, 'AuthorizedOperator'); // This helper needs to be implemented
+    it('does not emit AuthorizedOperator events for default operators', async function () {
+      await expectEvent.notEmitted.inConstruction(this.token, 'AuthorizedOperator');
     });
 
     describe('basic information', function () {