test: ensure add intercept method is idempotent and does not create d…

…uplicates

src/bidiMapper/domains/network/NetworkStorage.spec.ts

@@ -117,6 +117,35 @@ describe('NetworkStorage', () => {
         2
       );
     });
+
+    it('is idempotent', () => {
+      const intercept1 = networkStorage.addIntercept({
+        urlPatterns: [
+          {
+            type: 'string',
+            pattern: 'http://example.com',
+          },
+        ],
+        phases: [Network.InterceptPhase.BeforeRequestSent],
+      });
+      const intercept2 = networkStorage.addIntercept({
+        urlPatterns: [
+          {
+            type: 'string',
+            pattern: 'http://example.com',
+          },
+        ],
+        phases: [Network.InterceptPhase.BeforeRequestSent],
+      });
+
+      expect(intercept1).to.match(UUID_REGEX);
+      expect(intercept2).to.match(UUID_REGEX);
+      expect(intercept1).to.be.equal(intercept2);
+
+      expect(networkStorage.getFetchEnableParams().patterns).to.have.lengthOf(
+        1
+      );
+    });
   });
 
   it('remove intercept', () => {

src/bidiMapper/domains/network/NetworkStorage.ts

@@ -82,8 +82,20 @@ export class NetworkStorage {
     urlPatterns: Network.UrlPattern[];
     phases: Network.InterceptPhase[];
   }): Network.Intercept {
-    const interceptId: Network.Intercept = uuidv4();
+    // Check if the given intercept entry already exists.
+    for (const [
+      interceptId,
+      {urlPatterns, phases},
+    ] of this.#interceptMap.entries()) {
+      if (
+        JSON.stringify(value.urlPatterns) === JSON.stringify(urlPatterns) &&
+        JSON.stringify(value.phases) === JSON.stringify(phases)
+      ) {
+        return interceptId;
+      }
+    }
 
+    const interceptId: Network.Intercept = uuidv4();
     this.#interceptMap.set(interceptId, value);
 
     return interceptId;