Skip to content

Commit cb20020

Browse files
committed
Added coverage
1 parent c848e48 commit cb20020

File tree

5 files changed

+31
-5
lines changed

5 files changed

+31
-5
lines changed

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,5 @@ node_modules/
33
dist/
44
package-lock.json
55
yarn.lock
6+
coverage/
7+
.nyc_output/

package.json

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,18 @@
3030
"eslint-plugin-standard": "^4.0.0",
3131
"husky": "^1.3.1",
3232
"mocha": "^5.2.0",
33-
"nodemon": "^1.18.9"
33+
"nodemon": "^1.18.9",
34+
"nyc": "^13.3.0"
3435
},
3536
"scripts": {
36-
"test": "./node_modules/.bin/mocha",
37+
"test": "APP_ENVIROMENT=test ./node_modules/.bin/nyc ./node_modules/.bin/mocha --exit",
3738
"dev": "npm run lint && DEBUG=app nodemon --exec babel-node index.js",
3839
"build": "rm -rf dist && npm run lint && ./node_modules/.bin/babel . --out-dir dist --ignore node_modules && cp .env dist",
3940
"serve": "node dist/index.js",
4041
"lint": "./node_modules/.bin/eslint 'index.js' 'server/**/*.js' 'server/app.js' ",
4142
"precommit": "npm run lint",
42-
"prepush": "npm run lint"
43+
"prepush": "npm run lint",
44+
"test:coverage":"nyc --reporter=lcov yarn test"
4345
},
4446
"repository": {
4547
"type": "git",

server/config/mongoconnect.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ mongoose.connection.on('error', (error) => {
2626
process.exit(1)
2727
})
2828

29-
mongoose.set('debug', process.env.MONGO_DEBUG)
29+
if (process.env.APP_ENVIROMENT !== 'test') mongoose.set('debug', process.env.MONGO_DEBUG)
3030

3131
const connectMongo = async () => {
3232
let connectionuri =

server/config/routes.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { userRoutes } from '../modules/users/user.routes'
33
import { authRoutes } from '../modules/auth/auth.routes'
44
const Router = express.Router()
55

6-
Router.get('/health-check', (req, res) => res.json({ message: 'OK' }))
6+
Router.all('/health-check', (req, res) => res.json({ message: 'OK' }))
77
Router.use('/users', userRoutes)
88
Router.use('/auth', authRoutes)
99

test/integration/app.test.js

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,14 @@ describe('POST /auth/login', () => {
4040
})
4141
})
4242

43+
describe('GET /users', () => {
44+
it('should return 200', async () => {
45+
let response = await chai.request(server).get(`/api/users`).send()
46+
expect(response).to.have.status(200)
47+
expect(response.body.data.users).to.be.a('array').of.length(1)
48+
})
49+
})
50+
4351
describe('GET /auth/profile', () => {
4452
it('should return 200', async () => {
4553
let response = await chai.request(server).get(`/api/auth/profile`).set('Authorization', `Bearer ${token}`).send()
@@ -56,6 +64,20 @@ describe('PUT /users/:id', () => {
5664
expect(response).to.have.status(200)
5765
expect(response.body.message).to.be.equal('Record updated')
5866
})
67+
})
68+
69+
describe('GET /auth/login', () => {
70+
it('should return 404', async () => {
71+
let response = await chai.request(server).get('/api/auth/login').send({})
72+
expect(response).to.have.status(404)
73+
})
74+
})
75+
76+
describe('GET /auth/profile', () => {
77+
it('should return 401', async () => {
78+
let response = await chai.request(server).get(`/api/auth/profile`).send()
79+
expect(response).to.have.status(401)
80+
})
5981

6082
after('Cleaning users collection', async function () {
6183
await userModel.findOneAndDelete({ _id: sampleuserid })

0 commit comments

Comments
 (0)