Move mock feature data behind a naive db interface.

unleash-server/lib/api.js

@@ -1,41 +1,37 @@
-var featuresMock = require('./featuresMock');
+var db = require('./db');
 
-function getFeature(name) {
-    var featureFound;
-    featuresMock.forEach(function (feature) {
-        if (feature.name === name) {
-            featureFound = feature;
-        }
-    });
-    return featureFound;
-}
 
 module.exports = function (app) {
 
     app.get('/features', function (req, res) {
-        res.json(featuresMock);
+        db.getFeatures().then(function (features) {
+            res.json(features);
+        });
     });
 
     app.get('/features/:id', function (req, res) {
-        var feature = getFeature(req.params.id);
-
+        db.getFeature.then(function (feature) {
             if (feature) {
                 res.json(feature);
             } else {
                 res.json(404, {error: 'Could not find feature'});
             }
         });
+    });
 
     app.post('/features', function (req, res) {
         var newFeature = req.body;
 
-        if (!getFeature(newFeature.name)) {
-            featuresMock.push(newFeature);
-            res.status(201).end();
-        } else {
+        db.getFeature(newFeature.name).then(function (feature) {
+            if (feature) {
                 res.status(500).end();
+            } else {
+                db.addFeature(newFeature).then(function () {
+                    res.status(201).end();
+                });
             }
         });
+    });
 
 };
 

unleash-server/lib/db.js

@@ -0,0 +1,28 @@
+var Promise = require('bluebird');
+var featuresMock = require('./featuresMock');
+
+function getFeature(name) {
+    var featureFound;
+    featuresMock.forEach(function (feature) {
+        if (feature.name === name) {
+            featureFound = feature;
+        }
+    });
+
+    return Promise.resolve(featureFound);
+}
+
+function getFeatures() {
+    return Promise.resolve(featuresMock);
+}
+
+function addFeature(feature) {
+    featuresMock.push(feature);
+    return Promise.resolve();
+}
+
+module.exports = {
+    getFeature: getFeature,
+    getFeatures: getFeatures,
+    addFeature: addFeature
+};