Fixed an issue where storage items would be saved twice and where an item wouldn't be returned by the full key name.

Author: niemyjski

Diff for: dist/exceptionless.js

@@ -18,6 +18,23 @@ describe('InMemoryStorage', () => {
     expect(storage.getList(key + '1').length).toBe(1);
   });
 
+  it('should save once', () => {
+    var storage = new InMemoryStorage<number>();
+    storage.save('one', 1);
+    expect(storage.getList().length).toBe(1);
+    storage.save('one', 1);
+    expect(storage.getList().length).toBe(1);
+  });
+
+  it('should get by key', () => {
+    var storage = new InMemoryStorage<any>();
+    storage.save('ex-server-settings.json-version', 1);
+    storage.save('ex-server-settings.json', { exist: true });
+    expect(storage.getList().length).toBe(2);
+    expect(storage.get('ex-server-settings.json-version')).toBe(1);
+    expect(storage.get('ex-server-settings.json')).toEqual({ exist: true });
+  });
+
   it('should get saved events', () => {
     var storage = new InMemoryStorage<IEvent>();
     var key = 'ex-q-';

Diff for: dist/exceptionless.js.map

@@ -15,6 +15,7 @@ export class InMemoryStorage<T> implements IStorage<T> {
       return false;
     }
 
+    this.remove(path);
     if (this._items.push({ created: new Date().getTime(), path: path, value: value }) > this._maxItems) {
       this._items.shift();
     }
@@ -23,7 +24,7 @@ export class InMemoryStorage<T> implements IStorage<T> {
   }
 
   public get(path:string):T {
-    var item:IStorageItem<T> = path ? this.getList(path, 1)[0] : null;
+    var item:IStorageItem<T> = path ? this.getList(`^${path}$`, 1)[0] : null;
     return item ? item.value : null;
   }
 
@@ -50,7 +51,7 @@ export class InMemoryStorage<T> implements IStorage<T> {
 
   public remove(path:string):void {
     if (path) {
-      var item = this.getList(path, 1)[0];
+      var item = this.getList(`^${path}$`, 1)[0];
       if (item) {
         this._items.splice(this._items.indexOf(item), 1);
       }