test: add tests for server timings

packages/spec/express.js

@@ -0,0 +1,36 @@
+/* eslint-env node, mocha */
+
+var assert = require('assert');
+
+function clearRequireCache() {
+  Object.keys(require.cache).forEach(function(key) {
+    delete require.cache[key];
+  });
+}
+
+describe('express', function() {
+  beforeEach(clearRequireCache);
+  describe('timings', function() {
+    it('uses server-timings middleware when enableServerTimings is true', function() {
+      var hopsConfig = require('hops-config');
+      hopsConfig.enableServerTimings = true;
+      var hopsExpressTimings = require('hops-express').utils.timings;
+      var res = { locals: {} };
+      var req = {};
+
+      hopsExpressTimings(req, res, function() {});
+      assert(typeof res.locals.timings.start('foo') !== 'undefined');
+    });
+
+    it('uses noop timings middleware when enableServerTimings is false', function() {
+      var hopsConfig = require('hops-config');
+      hopsConfig.enableServerTimings = false;
+      var hopsExpressTimings = require('hops-express').utils.timings;
+      var res = { locals: {} };
+      var req = {};
+
+      hopsExpressTimings(req, res, function() {});
+      assert(typeof res.locals.timings.start('foo') === 'undefined');
+    });
+  });
+});

packages/spec/serve.js

@@ -123,4 +123,12 @@ describe('production server', function() {
       assert(response.status === 404);
     });
   });
+
+  it('adds server-timing header', function() {
+    return fetch('http://localhost:8080/').then(function(response) {
+      assert(
+        response.headers.get('server-timing').indexOf('desc="Request"') > -1
+      );
+    });
+  });
 });