Add BaseStore object

Author: pseudomuto

lib/store.js

@@ -1,25 +1,72 @@
 var trie = require("./trie.js");
 
+var NotImplemented = function (name) {
+  return {
+    name: "NotImplementedException",
+    message: "method '" + name + "' not implemented"
+  };
+};
+
+var BaseStore = Object.create(Object.prototype, {
+  // "abstract" methods
+  get:       { value: function (path) { throw NotImplemented("get"); } },
+  getTarget: { value: function (path) { throw NotImplemented("getTarget"); } },
+  getAll:    { value: function (path) { throw NotImplemented("getAll"); } },
+  add:       { value: function (path) { throw NotImplemented("add"); } },
+  update:    { value: function (path) { throw NotImplemented("update"); } },
+  remove:    { value: function (path) { throw NotImplemented("remove"); } },
+  hasRoute:  { value: function (path) { throw NotImplemented("hasRoute"); } },
+
+  cleanPath: {
+    value: function (path) {
+      return trie.trim_prefix(path);
+    }
+  }
+});
+
 function MemoryStore () {
   var routes = {};
   var urls   = new trie.URLTrie()
 
-  return {
-    get: function (path) { return routes[path]; },
-    getTarget: function(path) { return urls.get(path); },
-    getAll: function () { return routes; },
-    cleanPath: function(path) { return trie.trim_prefix(path); },
-    add: function (path, data) {
-      routes[path] = data;
-      urls.add(path, data);
+  return Object.create(BaseStore, {
+    get: {
+      value: function (path) {
+        return routes[path];
+      }
     },
-    update: function(path, data) { Object.assign(routes[path], data) },
-    remove: function (path) {
-      delete routes[path];
-      urls.remove(path);
+    getTarget: {
+      value: function (path) {
+        return urls.get(path);
+      }
     },
-    hasRoute: function (path) { return routes.hasOwnProperty(path); }
-  };
+    getAll: {
+      value: function () {
+        return routes;
+      }
+    },
+    add: {
+      value: function (path, data) {
+        routes[path] = data;
+        urls.add(path, data);
+      }
+    },
+    update: {
+      value: function (path, data) {
+        Object.assign(routes[path], data);
+      }
+    },
+    remove: {
+      value: function (path) {
+        delete routes[path];
+        urls.remove(path);
+      }
+    },
+    hasRoute: {
+      value: function (path) {
+        return routes.hasOwnProperty(path);
+      }
+    }
+  });
 }
 
 exports.MemoryStore = MemoryStore;

test/store_spec.js

@@ -19,6 +19,16 @@ describe("MemoryStore", function () {
     });
   });
 
+  describe("getTarget", function () {
+    it("returns the target object for the path", function () {
+      this.subject.add("/my_route", { "target": "http://localhost:8213" });
+
+      var target = this.subject.getTarget("/my_route");
+      expect(target.prefix).toEqual("/my_route");
+      expect(target.data.target).toEqual("http://localhost:8213");
+    });
+  });
+
   describe("getAll", function () {
     it("returns all routes", function () {
       this.subject.add("/my_route", { "test": "value1" });