Merge pull request #11 from smithki/development

Release v0.11.1

package.json

@@ -1,6 +1,6 @@
 {
   "name": "storage-proxy",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "description": "Use web storage (localStorage/sessionStorage) just like plain objects using ES6 Proxies.",
   "author": "Ian K Smith <[email protected]>",
   "license": "MIT",

src/lib.ts

@@ -87,7 +87,9 @@ function createProxy<TStorageDefinitions extends any>(
 
   if (defaults) {
     for (const [key, value] of Object.entries(defaults)) {
-      storageProxy[key] = value;
+      if (!data[key]) {
+        storageProxy[key] = value;
+      }
     }
   }
 

test/src/storage-proxy.spec.ts

@@ -18,6 +18,7 @@ interface TestStorage {
   defaults: {
     example: string;
   };
+  alreadySetDefault: number;
 }
 
 function getItem(storageTarget: StorageTarget, path: string) {
@@ -45,11 +46,15 @@ export class StorageProxyTestFixture {
 
   @SetupFixture
   public setupFixture() {
-    localStorage.setItem(testNamespace, JSON.stringify({ bar: testStr, test: 999, baz: testObj }));
+    localStorage.setItem(
+      testNamespace,
+      JSON.stringify({ bar: testStr, test: 999, baz: testObj, alreadySetDefault: 999 }),
+    );
     sessionStorage.setItem(testNamespace, JSON.stringify({ bar: testStr, test: 999, baz: testObj }));
 
     this.lStore = StorageProxy.createLocalStorage<TestStorage>(testNamespace, {
       defaults: { example: testStr },
+      alreadySetDefault: 123,
     });
     this.sStore = StorageProxy.createSessionStorage<TestStorage>(testNamespace);
   }
@@ -64,6 +69,11 @@ export class StorageProxyTestFixture {
     Expect(this.lStore.defaults!.example).toEqual(testStr);
   }
 
+  @Test('Default key that is already set in storage is not overwritten')
+  public defaultsNotAllPowerfulTest() {
+    Expect(this.lStore.alreadySetDefault).toEqual(999);
+  }
+
   @Test('Set a `localStorage` key')
   public setLocalStorageKeyTest() {
     this.lStore.fizz = 123;