Add early return if JSON RPC request is not found (#35)

Author: jpuri

jest.config.js

@@ -21,10 +21,10 @@ module.exports = {
   // An object that configures minimum threshold enforcement for coverage results
   coverageThreshold: {
     global: {
-      branches: 86.95,
+      branches: 91.3,
       functions: 100,
-      lines: 97.51,
-      statements: 97.51,
+      lines: 98.51,
+      statements: 98.51,
     },
   },
 

src/createStreamMiddleware.ts

@@ -104,6 +104,7 @@ export default function createStreamMiddleware(options: Options = {}) {
     const context = idMap[res.id as unknown as string];
     if (!context) {
       console.warn(`StreamMiddleware - Unknown response id "${res.id}"`);
+      return;
     }
 
     delete idMap[res.id as unknown as string];

src/index.test.ts

@@ -223,6 +223,17 @@ describe('retry logic in middleware connected to a port', () => {
     }).toThrow('StreamMiddleware - Retry limit exceeded for request id');
   });
 
+  it('does not throw error when response is received for request not in map', async () => {
+    const res = { id: 1, jsonrpc, result: 'test' };
+
+    messageConsumer(res);
+
+    expect(() => {
+      messageConsumer(res);
+      messageConsumer(res);
+    }).not.toThrow();
+  });
+
   it('does not retry if the request has no id', async () => {
     // request and expected result
     const req = { id: undefined, jsonrpc, method: 'test' };