Second attempt at fixing flaky cookie test on Edge

Author: fathyb

test/cookie.test.js

@@ -8,10 +8,6 @@ describe('cookie', function() {
     // Just to make sure that
     // URIError is never thrown here.
     document.cookie = 'bad=%';
-
-    // enable debugging for cookie package so we can better
-    // observe flaky tests on IE/Edge.
-    window.localStorage.setItem('debug', 'cookie');
   });
 
   afterEach(function() {
@@ -38,11 +34,18 @@ describe('cookie', function() {
 
   describe('#set', function() {
     it('should set a cookie', function() {
-      // Logging so we can better observe flaky tests on IE/Edge.
-      console.log('setting cookie. document.cookie is', document.cookie);
-      cookie.set('x', { a: 'b' });
-      console.log('cookie set. document.cookie is', document.cookie);
-      assert.deepEqual(cookie.get('x'), { a: 'b' });
+      // Use a random key for the cookie
+      // Workaround for flaky concurrent tests on Edge
+      var key = Math.random()
+        .toString(36)
+        .slice(2);
+
+      try {
+        cookie.set(key, { a: 'b' });
+        assert.deepEqual(cookie.get(key), { a: 'b' });
+      } finally {
+        cookie.remove(key);
+      }
     });
   });