Don't send undefined params (#290)

MetaMask · Nov 15, 2023 · d5d5de4 · d5d5de4
1 parent ae74a05
commit d5d5de4
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/jest.config.js b/jest.config.js
@@ -45,10 +45,10 @@ const baseConfig = {
   // An object that configures minimum threshold enforcement for coverage results
   coverageThreshold: {
     global: {
-      branches: 64.78,
+      branches: 65.11,
       functions: 65.65,
-      lines: 66.66,
-      statements: 66.74,
+      lines: 66.74,
+      statements: 66.81,
     },
   },
 

diff --git a/src/BaseProvider.ts b/src/BaseProvider.ts
@@ -208,11 +208,18 @@ export abstract class BaseProvider extends SafeEventEmitter {
       });
     }
 
+    const payload =
+      params === undefined
+        ? {
+            method,
+          }
+        : {
+            method,
+            params,
+          };
+
     return new Promise<T>((resolve, reject) => {
-      this._rpcRequest(
-        { method, params },
-        getRpcPromiseCallback(resolve, reject),
-      );
+      this._rpcRequest(payload, getRpcPromiseCallback(resolve, reject));
     });
   }