A bit of house-keeping

lib/routes/admin-api/metrics.js

@@ -4,7 +4,11 @@ const { Router } = require('express');
 
 const logger = require('../../logger')('/admin-api/metrics.js');
 const ClientMetrics = require('../../client-metrics');
-const { catchLogAndSendErrorResponse } = require('./route-utils');
+
+const catchLogAndSendErrorResponse = (err, res) => {
+    logger.error(err);
+    res.status(500).end();
+};
 
 exports.router = function(config) {
     const {
@@ -67,10 +71,7 @@ exports.router = function(config) {
         clientApplicationsStore
             .upsert(input)
             .then(() => res.status(202).end())
-            .catch(e => {
+            .catch(err => catchLogAndSendErrorResponse(err, res));
-                logger.error(e);
-                res.status(500).end();
-            });
     });
 
     function toLookup(metaData) {

lib/routes/admin-api/metrics.test.js

@@ -93,3 +93,31 @@ test('should return metrics for all toggles', t => {
             t.true(metrics.lastMinute !== undefined);
         });
 });
+
+test('should return applications', t => {
+    t.plan(2);
+    const { request, stores } = getSetup();
+    const appName = '123!23';
+
+    stores.clientApplicationsStore.upsert({ appName });
+
+    return request
+        .get(`/api/admin/metrics/applications/`)
+        .expect(200)
+        .expect(res => {
+            const metrics = res.body;
+            t.true(metrics.applications.length === 1);
+            t.true(metrics.applications[0].appName === appName);
+        });
+});
+
+test('should store application', t => {
+    t.plan(0);
+    const { request } = getSetup();
+    const appName = '123!23';
+
+    return request
+        .post(`/api/admin/metrics/applications/${appName}`)
+        .send({ appName })
+        .expect(202);
+});

lib/routes/admin-api/route-utils.js

@@ -1,10 +0,0 @@
-'use strict';
-
-const logger = require('../../logger')('route-utils.js');
-
-const catchLogAndSendErrorResponse = (err, res) => {
-    logger.error(err);
-    res.status(500).end();
-};
-
-module.exports = { catchLogAndSendErrorResponse };

test/fixtures/fake-client-applications-store.js

@@ -1,6 +1,12 @@
 'use strict';
 
+const _appliations = [];
+
 module.exports = () => ({
-    upsert: () => Promise.resolve(),
+    upsert: app => {
-    getApplications: () => Promise.resolve([]),
+        _appliations.push(app);
+        return Promise.resolve();
+    },
+    getApplications: () => Promise.resolve(_appliations),
+    getApplication: appName => _appliations.filter(a => a.name === appName)[0],
 });