Skip to content

Commit

Permalink
feat: remove underscore from test files #198
Browse files Browse the repository at this point in the history
  • Loading branch information
David Longworth committed Dec 1, 2017
1 parent c2095c0 commit 307943e
Show file tree
Hide file tree
Showing 13 changed files with 15 additions and 19 deletions.
16 changes: 12 additions & 4 deletions test/acceptance/app.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var nock = require("nock")
var path = require("path")
var request = require("supertest")
Expand Down Expand Up @@ -30,7 +29,9 @@ describe("Application", function() {
.times(5)
.reply(200, { accessToken: "xx" })

var scope1 = nock(apiHost).get("/").reply(200)
var scope1 = nock(apiHost)
.get("/")
.reply(200)

var configUpdate = {
server: {
Expand Down Expand Up @@ -135,7 +136,10 @@ describe("Application", function() {

TestHelper.enableApiConfig().then(() => {
TestHelper.startServer(pages).then(() => {
client.get("/api/status").expect(405).end(done)
client
.get("/api/status")
.expect(405)
.end(done)
})
})
})
Expand All @@ -158,7 +162,11 @@ describe("Application", function() {

TestHelper.enableApiConfig().then(() => {
TestHelper.startServer(pages).then(() => {
client.post("/api/status").send({}).expect(401).end(done)
client
.post("/api/status")
.send({})
.expect(401)
.end(done)
})
})
})
Expand Down
1 change: 0 additions & 1 deletion test/acceptance/public.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var nock = require("nock")
var path = require("path")
Expand Down
1 change: 0 additions & 1 deletion test/acceptance/routing.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var nock = require("nock")
var path = require("path")
var request = require("supertest")
Expand Down
7 changes: 3 additions & 4 deletions test/help.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var assert = require("assert")
var fs = require("fs")
var nock = require("nock")
Expand Down Expand Up @@ -85,14 +84,14 @@ TestHelper.prototype.enableApiConfig = function() {
TestHelper.prototype.getConfig = function() {
return new Promise((resolve, reject) => {
var originalConfig = JSON.parse(this.originalConfigString)
return resolve(_.extend({}, originalConfig))
return resolve(Object.assign({}, originalConfig))
})
}

TestHelper.prototype.updateConfig = function(configBlock) {
return new Promise((resolve, reject) => {
var originalConfig = JSON.parse(this.originalConfigString)
var newConfig = _.extend(originalConfig, configBlock)
var newConfig = Object.assign(originalConfig, configBlock)

fs.writeFileSync(config.configPath(), JSON.stringify(newConfig, null, 2))
config.loadFile(path.resolve(config.configPath()))
Expand Down Expand Up @@ -197,7 +196,7 @@ TestHelper.prototype.newPage = function(

TestHelper.prototype.startServer = function(pages) {
return new Promise((resolve, reject) => {
if (pages !== null && !_.isArray(pages)) {
if (pages !== null && !Array.isArray(pages)) {
pages = [pages]
}

Expand Down
1 change: 0 additions & 1 deletion test/unit/config.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var path = require("path")
var should = require("should")
Expand Down
1 change: 0 additions & 1 deletion test/unit/controller.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var nock = require("nock")
var request = require("supertest")
var should = require("should")
Expand Down
1 change: 0 additions & 1 deletion test/unit/helpTest.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var nock = require("nock")
var request = require("supertest")
Expand Down
1 change: 0 additions & 1 deletion test/unit/monitor.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var path = require("path")
var should = require("should")
Expand Down
1 change: 0 additions & 1 deletion test/unit/preloader.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var nock = require("nock")
var path = require("path")
Expand Down
1 change: 0 additions & 1 deletion test/unit/router.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var fs = require("fs")
var nock = require("nock")
var request = require("supertest")
Expand Down
1 change: 0 additions & 1 deletion test/unit/server.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var path = require("path")
var should = require("should")
var sinon = require("sinon")
Expand Down
1 change: 0 additions & 1 deletion test/unit/view.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
var _ = require("underscore")
var dust = require("dustjs-linkedin")
var dustHelpers = require("dustjs-helpers")
var fs = require("fs")
Expand Down
1 change: 0 additions & 1 deletion workspace/utils/helpers/paginate.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
var dust = require("dustjs-linkedin")
var _ = require("underscore")

/*
* Paginate pages
Expand Down

0 comments on commit 307943e

Please sign in to comment.