From 702117dbea73e68308cdb10eac224a0a5acc3fbe Mon Sep 17 00:00:00 2001 From: Supasate Choochaisri Date: Fri, 6 May 2016 21:48:38 +0700 Subject: [PATCH] Add blank lines to separate error-checking and done logic from other logic Signed-off-by: Supasate Choochaisri --- test/explorer.test.js | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/test/explorer.test.js b/test/explorer.test.js index 3527457..547ac28 100644 --- a/test/explorer.test.js +++ b/test/explorer.test.js @@ -38,6 +38,7 @@ describe('explorer', function() { assert(!!~res.text.indexOf('StrongLoop API Explorer'), 'text does not contain expected string'); + done(); }); }); @@ -49,8 +50,10 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + expect(res.body).to .have.property('url', '/explorer/swagger.json'); + done(); }); }); @@ -71,8 +74,10 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + expect(res.body).to .have.property('url', '/swagger/swagger.json'); + done(); }); }); @@ -89,8 +94,10 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + expect(res.body).to .have.property('url', '/explorer/swagger.json'); + done(); }); }); @@ -108,9 +115,11 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + var baseUrl = res.body.basePath; var apiPath = Object.keys(res.body.paths)[0]; expect(baseUrl + apiPath).to.equal('/apis/products'); + done(); }); }); @@ -167,8 +176,10 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + expect(res.body).to .have.property('url', '/explorer/swagger.json'); + done(); }); }); @@ -254,9 +265,11 @@ describe('explorer', function() { .options('/explorer/swagger.json') .end(function(err, res) { if (err) return done(err); + var allowOrigin = res.get('Access-Control-Allow-Origin'); expect(allowOrigin, 'Access-Control-Allow-Origin') .to.equal(undefined); + done(); }); }); @@ -284,10 +297,12 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + var modelNames = Object.keys(res.body.definitions); expect(modelNames).to.contain('Customer'); var paths = Object.keys(res.body.paths); expect(paths).to.have.contain('/Customers'); + done(); }); }); @@ -317,8 +332,10 @@ describe('explorer', function() { .expect(200) .end(function(err, res) { if (err) return done(err); + var paths = Object.keys(res.body.paths); expect(paths).to.not.contain('/products/findOne'); + done(); }); }); @@ -328,6 +345,7 @@ describe('explorer', function() { return function(done) { var app = this.app = loopback(); configureRestApiAndExplorer(app, explorerBase); + done(); }; }