Remove unnecessary code

Author: Mrtenz

packages/snaps-rpc-methods/jest.config.js

@@ -10,10 +10,10 @@ module.exports = deepmerge(baseConfig, {
   ],
   coverageThreshold: {
     global: {
-      branches: 93.33,
+      branches: 93.3,
       functions: 97.46,
       lines: 98.03,
-      statements: 97.62,
+      statements: 97.61,
     },
   },
 });

packages/snaps-rpc-methods/src/permitted/setState.test.ts

@@ -461,18 +461,6 @@ describe('set', () => {
     });
   });
 
-  it('overwrites the state in an existing object', () => {
-    const object = {
-      nested: {
-        key: 'oldValue',
-      },
-    };
-
-    expect(set(object, undefined, { foo: 'bar' })).toStrictEqual({
-      foo: 'bar',
-    });
-  });
-
   it('overwrites the nested state in an existing object', () => {
     const object = {
       nested: {

packages/snaps-rpc-methods/src/permitted/setState.ts

@@ -220,14 +220,9 @@ async function getNewState(
 export function set(
   // eslint-disable-next-line @typescript-eslint/default-param-last
   object: Record<string, Json> | null,
-  key: string | undefined,
+  key: string,
   value: Json,
 ): JsonObject {
-  if (key === undefined) {
-    assert(isObject(value));
-    return value;
-  }
-
   const keys = key.split('.');
   const requiredObject = object ?? {};
   let currentObject: Record<string, Json> = requiredObject;