From 71f8da2afab4f9ed852b0105a8a454ebd4a2d087 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicol=C3=A1s=20Venturo?= <nicolas.venturo@gmail.com>
Date: Thu, 12 Mar 2020 16:18:46 -0300
Subject: [PATCH] Uncomment expectEvent.not tests

---
 package-lock.json                 | 24 ++++++++++++------------
 package.json                      |  2 +-
 test/access/AccessControl.test.js | 10 +++++-----
 3 files changed, 18 insertions(+), 18 deletions(-)

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');
       });
     });
   });