fix: set undefined value should get "undefined" (#144)

clarkbw · Aug 11, 2022 · 383d571 · 383d571
1 parent 3ad36fc
commit 383d571
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/__tests__/index.js b/__tests__/index.js
@@ -45,7 +45,7 @@ describe('storage', () =>
       storage.setItem(KEY, null);
       expect(storage.__STORE__[KEY]).toBe('null');
       storage.setItem(KEY, undefined);
-      expect(storage.__STORE__[KEY]).toBe('');
+      expect(storage.__STORE__[KEY]).toBe('undefined');
       storage.setItem(KEY, {});
       expect(storage.__STORE__[KEY]).toBe('[object Object]');
     });

diff --git a/src/localstorage.js b/src/localstorage.js
@@ -7,7 +7,7 @@ export class LocalStorage {
     Object.defineProperty(this, 'setItem', {
       enumerable: false,
       // not mentioned in the spec, but we must always coerce to a string
-      value: jest.fn((key, val = '') => {
+      value: jest.fn((key, val) => {
         this[key] = val + '';
       }),
     });