Break test order dependencies in security-rules integration tests.

Previously, this would work:
yarn test:integration --updateRules --grep 'admin.securityRules'

but this would not:
yarn test:integration --updateRules --grep 'admin.securityRules.getRuleset\(\).rejects with not-found'

because that test relied on side effects from earier tests, making this
test suite difficult to debug.

Author: rsgowman

test/integration/security-rules.spec.ts

@@ -45,7 +45,6 @@ const RULESET_NAME_PATTERN = /[0-9a-zA-Z-]+/;
 
 describe('admin.securityRules', () => {
 
-  let testRuleset: admin.securityRules.Ruleset = null;
   const rulesetsToDelete: string[] = [];
 
   function scheduleForDelete(ruleset: admin.securityRules.Ruleset) {
@@ -65,7 +64,7 @@ describe('admin.securityRules', () => {
     return Promise.all(promises);
   }
 
-  after(() => {
+  afterEach(() => {
     return deleteTempRulesets();
   });
 
@@ -91,7 +90,6 @@ describe('admin.securityRules', () => {
         RULES_FILE_NAME, SAMPLE_FIRESTORE_RULES);
       return admin.securityRules().createRuleset(rulesFile)
         .then((ruleset) => {
-          testRuleset = ruleset;
           scheduleForDelete(ruleset);
           verifyFirestoreRuleset(ruleset);
         });
@@ -105,23 +103,40 @@ describe('admin.securityRules', () => {
     });
   });
 
+  function getRandomInt(max: number) {
+    return Math.floor(Math.random() * Math.floor(max));
+  }
+
+  function createTemporaryRuleset(): Promise<admin.securityRules.Ruleset> {
+    const name = "random_name_" + getRandomInt(100000) + ".rules";
+    const rulesFile = admin.securityRules().createRulesFileFromSource(name, SAMPLE_FIRESTORE_RULES);
+    return admin.securityRules().createRuleset(rulesFile)
+      .then((ruleset) => {
+        scheduleForDelete(ruleset);
+        return ruleset;
+      });
+  }
+
   describe('getRuleset()', () => {
     it('rejects with not-found when the Ruleset does not exist', () => {
-      const name = 'e1212' + testRuleset.name.substring(5);
-      return admin.securityRules().getRuleset(name)
+      const uuid = '00000000-1111-2222-3333-444444444444';
+      return admin.securityRules().getRuleset(uuid)
         .should.eventually.be.rejected.and.have.property('code', 'security-rules/not-found');
     });
 
     it('rejects with invalid-argument when the Ruleset name is invalid', () => {
-      return admin.securityRules().getRuleset('invalid')
+      return admin.securityRules().getRuleset('invalid uuid')
         .should.eventually.be.rejected.and.have.property('code', 'security-rules/invalid-argument');
     });
 
     it('resolves with existing Ruleset', () => {
-      return admin.securityRules().getRuleset(testRuleset.name)
-        .then((ruleset) => {
-          verifyFirestoreRuleset(ruleset);
-        });
+      return createTemporaryRuleset()
+        .then((expectedRuleset) =>
+          admin.securityRules().getRuleset(expectedRuleset.name)
+            .then((actualRuleset) => {
+              expect(actualRuleset).to.deep.equal(expectedRuleset);
+            })
+        );
     });
   });
 
@@ -137,7 +152,7 @@ describe('admin.securityRules', () => {
       return admin.securityRules().releaseFirestoreRuleset(oldRuleset);
     }
 
-    after(() => {
+    afterEach(() => {
       return revertFirestoreRuleset();
     });
 
@@ -185,7 +200,7 @@ describe('admin.securityRules', () => {
       return admin.securityRules().releaseStorageRuleset(oldRuleset);
     }
 
-    after(() => {
+    afterEach(() => {
       return revertStorageRuleset();
     });
 
@@ -240,9 +255,13 @@ describe('admin.securityRules', () => {
           });
       }
 
-      return listAllRulesets()
-        .then((rulesets) => {
-          expect(rulesets.some((rs) => rs.name === testRuleset.name)).to.be.true;
+      return Promise.all([createTemporaryRuleset(), createTemporaryRuleset()])
+        .then((expectedRulesets) => {
+          return listAllRulesets().then((actualRulesets) => {
+            expectedRulesets.forEach((expectedRuleset) => {
+              expect(actualRulesets.map(r => r.name)).to.deep.include(expectedRuleset.name);
+            });
+          });
         });
     });
 
@@ -257,26 +276,27 @@ describe('admin.securityRules', () => {
 
   describe('deleteRuleset()', () => {
     it('rejects with not-found when the Ruleset does not exist', () => {
-      const name = 'e1212' + testRuleset.name.substring(5);
-      return admin.securityRules().deleteRuleset(name)
+      const uuid = '00000000-1111-2222-3333-444444444444'
+      return admin.securityRules().deleteRuleset(uuid)
         .should.eventually.be.rejected.and.have.property('code', 'security-rules/not-found');
     });
 
     it('rejects with invalid-argument when the Ruleset name is invalid', () => {
-      return admin.securityRules().deleteRuleset('invalid')
+      return admin.securityRules().deleteRuleset('invalid uuid')
         .should.eventually.be.rejected.and.have.property('code', 'security-rules/invalid-argument');
     });
 
     it('deletes existing Ruleset', () => {
-      return admin.securityRules().deleteRuleset(testRuleset.name)
-        .then(() => {
-          return admin.securityRules().getRuleset(testRuleset.name)
-            .should.eventually.be.rejected.and.have.property('code', 'security-rules/not-found');
-        })
-        .then(() => {
-          unscheduleForDelete(testRuleset); // Already deleted.
-          testRuleset = null;
-        });
+      return createTemporaryRuleset().then((ruleset) => {
+        return admin.securityRules().deleteRuleset(ruleset.name)
+          .then(() => {
+            return admin.securityRules().getRuleset(ruleset.name)
+              .should.eventually.be.rejected.and.have.property('code', 'security-rules/not-found');
+          })
+          .then(() => {
+            unscheduleForDelete(ruleset); // Already deleted.
+          });
+      });
     });
   });