Skip to content

Commit

Permalink
Add basic tests for express routes
Browse files Browse the repository at this point in the history
Relates #2
  • Loading branch information
abdullahchaudhry committed Aug 17, 2017
1 parent 52bb81b commit ee949d7
Show file tree
Hide file tree
Showing 3 changed files with 49 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"description": "",
"main": "index.js",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1",
"test": "node ./tests/express-test.js | tap-spec",
"start:watch": "nodemon ./src/index.js"
},
"repository": {
Expand Down
2 changes: 1 addition & 1 deletion src/controllers/browse.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ exports.get = (req, res, next) => {
},
data: stuff.rows
});
console.log('this is out stuff: ', stuff);
// console.log('this is out stuff: ', stuff);
// res.rows??
})
.catch((stuff, text)=>{
Expand Down
47 changes: 47 additions & 0 deletions tests/express-test.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
const test = require('tape');
const supertest = require('supertest');
const app = require('./../src/app');

// check if tape works
test('1 equals 1', (t) => {
t.equals(1, 1, 'one should equal one');
t.end();
});

// check if supetest works
test('check if supertest works', (t) => {
supertest(app)
.get('/')
.expect(200)
.expect('Content-Type', /json/)
.end((err, res) => {
t.same(res.statusCode, 200, 'Status code is 200');
t.end();
})
})

// check we get 404 on non-existing endpoint
test('404 when endpoint does not exist', (t) => {
supertest(app)
.get('/bsdjkhgablsg')
.expect(404)
.expect('Content-Type', /json/)
.end((err, res) => {
t.same(res.statusCode, 404, 'Status code is 404');
t.end();
})
})

test('check prepopulated browse database table', (t) => {
supertest(app)
.get('/browse')
.expect(200)
.expect('Content-Type', /json/)
.end((err, res) => {
let expected = "Movie recommendation app";
let actual = res;
console.log('res.rows ', res);
t.equals(actual, expected, 'Should be Movie recommendation app')
t.end();
})
})

0 comments on commit ee949d7

Please sign in to comment.