Merge pull request #95 from christroutner/unstable

Moving unit tests to a 'unit' directory

package.json

@@ -5,11 +5,11 @@
   "main": "index.js",
   "scripts": {
     "start": "node index.js",
-    "test": "export KOA_ENV=test && npm run prep-test && nyc --reporter=text --timeout 15000 ./node_modules/.bin/mocha --exit",
+    "test": "export KOA_ENV=test && npm run prep-test && nyc --reporter=text mocha --exit --timeout 15000 test/unit/",
     "lint": "standard --env mocha --fix",
     "docs": "./node_modules/.bin/apidoc -i src/ -o docs",
     "coverage": "export KOA_ENV=test && npm run prep-test && nyc report --reporter=text-lcov | coveralls",
-    "coverage:report": "export KOA_ENV=test && npm run prep-test && nyc --reporter=html mocha --exit",
+    "coverage:report": "export KOA_ENV=test && npm run prep-test && nyc --reporter=html mocha --exit --timeout 15000 test/unit/",
     "prep-test": "node util/users/delete-all-test-users.js"
   },
   "keywords": [

src/modules/users/controller.js

@@ -89,6 +89,8 @@ class UserController {
         token
       }
     } catch (err) {
+      // console.log(`err.message: ${err.message}`)
+      // console.log('err: ', err)
       ctx.throw(422, err.message)
     }
   }

test/a01-auth.spec.js → test/unit/a01-auth.spec.js

@@ -1,6 +1,6 @@
-const app = require('../bin/server')
+const app = require('../../bin/server')
 const utils = require('./utils')
-const config = require('../config')
+const config = require('../../config')
 const assert = require('chai').assert
 
 const axios = require('axios').default

test/a02-users.spec.js → test/unit/a02-users.spec.js

@@ -1,6 +1,6 @@
 const testUtils = require('./utils')
 const assert = require('chai').assert
-const config = require('../config')
+const config = require('../../config')
 const axios = require('axios').default
 const sinon = require('sinon')
 
@@ -11,7 +11,7 @@ const LOCALHOST = `http://localhost:${config.port}`
 
 const context = {}
 
-const UserController = require('../src/modules/users/controller')
+const UserController = require('../../src/modules/users/controller')
 let uut
 let sandbox
 

test/a03-nodemailer.spec.js → test/unit/a03-nodemailer.spec.js

@@ -1,6 +1,6 @@
 const assert = require('chai').assert
 
-const NodeMailer = require('../src/lib/nodemailer')
+const NodeMailer = require('../../src/lib/nodemailer')
 
 const sinon = require('sinon')
 

test/a04-contact.spec.js → test/unit/a04-contact.spec.js

@@ -1,4 +1,4 @@
-const config = require('../config')
+const config = require('../../config')
 const axios = require('axios').default
 const assert = require('chai').assert
 const sinon = require('sinon')
@@ -9,7 +9,7 @@ const sinon = require('sinon')
 const LOCALHOST = `http://localhost:${config.port}`
 
 const mockContext = require('./mocks/ctx-mock').context
-const ContactController = require('../src/modules/contact/controller')
+const ContactController = require('../../src/modules/contact/controller')
 let uut
 let sandbox
 

test/a05-passport.spec.js → test/unit/a05-passport.spec.js

@@ -1,5 +1,5 @@
 const assert = require('chai').assert
-const PassportLib = require('../src/lib/passport')
+const PassportLib = require('../../src/lib/passport')
 
 const sinon = require('sinon')
 

test/a06-logapi.spec.js → test/unit/a06-logapi.spec.js

@@ -1,4 +1,4 @@
-const config = require('../config')
+const config = require('../../config')
 const assert = require('chai').assert
 
 const axios = require('axios').default
@@ -9,7 +9,7 @@ util.inspect.defaultOptions = { depth: 1 }
 
 const LOCALHOST = `http://localhost:${config.port}`
 
-const LogsController = require('../src/modules/logapi/controller')
+const LogsController = require('../../src/modules/logapi/controller')
 const mockContext = require('./mocks/ctx-mock').context
 const mockData = require('./mocks/log-api-mock')
 

test/a07-json-files.spec.js → test/unit/a07-json-files.spec.js

@@ -5,7 +5,7 @@ const sinon = require('sinon')
 const util = require('util')
 util.inspect.defaultOptions = { depth: 1 }
 
-const JsonFiles = require('../src/lib/utils/json-files')
+const JsonFiles = require('../../src/lib/utils/json-files')
 
 const JSON_FILE = 'test-json-file.json'
 const JSON_PATH = `${__dirname}/${JSON_FILE}`

test/a08-validators.spec.js → test/unit/a08-validators.spec.js

@@ -1,7 +1,7 @@
 const assert = require('chai').assert
 const testUtils = require('./utils')
 
-const Validators = require('../src/middleware/validators')
+const Validators = require('../../src/middleware/validators')
 
 const sinon = require('sinon')
 const mockContext = require('./mocks/ctx-mock').context

test/a09-admin.spec.js → test/unit/a09-admin.spec.js

@@ -1,6 +1,6 @@
 const assert = require('chai').assert
 
-const Admin = require('../src/lib/admin')
+const Admin = require('../../src/lib/admin')
 
 const sinon = require('sinon')
 

test/utils.js → test/unit/utils.js

@@ -1,5 +1,5 @@
 const mongoose = require('mongoose')
-const config = require('../config')
+const config = require('../../config')
 const axios = require('axios').default
 
 const LOCALHOST = `http://localhost:${config.port}`
@@ -79,7 +79,7 @@ async function loginTestUser () {
 
 async function loginAdminUser () {
   try {
-    const FILENAME = `../config/system-user-${config.env}.json`
+    const FILENAME = `../../config/system-user-${config.env}.json`
     const adminUserData = require(FILENAME)
     console.log(`adminUserData: ${JSON.stringify(adminUserData, null, 2)}`)
 
@@ -115,7 +115,7 @@ async function getAdminJWT () {
     // process.env.KOA_ENV = process.env.KOA_ENV || 'dev'
     // console.log(`env: ${process.env.KOA_ENV}`)
 
-    const FILENAME = `../config/system-user-${config.env}.json`
+    const FILENAME = `../../config/system-user-${config.env}.json`
     const adminUserData = require(FILENAME)
     // console.log(`adminUserData: ${JSON.stringify(adminUserData, null, 2)}`)
 

util/users/delete-all-test-users.js

@@ -29,4 +29,5 @@ async function deleteUsers () {
 
   mongoose.connection.close()
 }
+
 deleteUsers()