update tests to wait for Database to finish seeding using events

Author: Jrschellenberg

package.json

@@ -7,8 +7,8 @@
     "babel-node": "babel-node --presets=es2015",
     "nodemon": "nodemon -e pug,js,css --exec npm run babel-node -- src/bin/www",
     "start": "SET NODE_ENV=dev && npm run nodemon",
-    "test": "mocha --exit --timeout 5000 --compilers js:babel-core/register",
-    "coverage": "cross-env NODE_ENV=test nyc --reporter=lcov --reporter=text mocha --exit --timeout 5000 --compilers js:babel-core/register"
+    "test": "mocha --exit --timeout 10000 --compilers js:babel-core/register",
+    "coverage": "cross-env NODE_ENV=test nyc --reporter=lcov --reporter=text mocha --exit --timeout 10000 --compilers js:babel-core/register"
   },
   "author": "",
   "license": "ISC",

src/app.js

@@ -42,6 +42,7 @@ db.on('connected', function() {
 			// Callback to populate DB once collections have been cleared
 			seeder.populateModels(data, function() {
 				console.log("Finished seeding Database!");
+				app.emit("appStarted"); //Emits an event to tell our tests it is ok to now test.
 			});
 		});
 	});

src/bin/www

@@ -16,6 +16,7 @@ var port = normalizePort(process.env.PORT || '3000');
 app.set('port', port);
 console.log("Listening on port", port);
 
+
 /**
  * Create HTTP server.
  */
@@ -26,10 +27,11 @@ var server = http.createServer(app);
  * Listen on provided port, on all network interfaces.
  */
 
-server.listen(port);
 server.on('error', onError);
 server.on('listening', onListening);
 
+
+
 /**
  * Normalize a port into a number, string, or false.
  */

src/finishSeed.js

@@ -0,0 +1,9 @@
+const server = require("./app");
+
+let dataBaseFinishSeed = false;
+
+server.on('appStarted', () => {
+	dataBaseFinishSeed = true;
+});
+
+export {dataBaseFinishSeed}

test/course_test.js

@@ -1,6 +1,7 @@
 'use strict';
 //During the test the env variable is set to test
 process.env.NODE_ENV = 'test';
+import {dataBaseFinishSeed} from "../src/finishSeed";
 
 const Course = require('../src/models/course'),
 	chai = require('chai'),
@@ -12,17 +13,19 @@ const should = chai.should();
 const courseIndexLink = '/api/courses/';
 chai.use(chaiHttp);
 
+
 const validAuth = {
 	user: 'joe@smith.com',
 	pass: 'password'
 };
 
 describe('Courses', () => {
-	// beforeEach((done) => {
-	// 	Course.remove({}, (err) => {
-	// 		done();
-	// 	});
-	// });
+	beforeEach((done) => { //This hangs the tests until databaesFinish seed..
+		if(dataBaseFinishSeed){done();} //if it has finished seeding it will hit callback, else forever hangs
+		server.on('appStarted', () => { //Once finish seed, set to true and stop hang.
+			done();
+		});
+	});
 	/*
 	Our GET Tests
 	 */
@@ -68,7 +71,7 @@ describe('Courses', () => {
 				.auth(validAuth.user, validAuth.pass)
 				.send(course)
 				.end((err, res) => {
-					res.body.should.have.property('message').equal('test');
+					res.body.should.have.property('message').equal('Course Successfully added!');
 					res.should.have.status(201);
 					res.body.should.be.a('object');
 					res.body.course.should.have.property('title');

test/user_test.js

@@ -13,8 +13,16 @@ const expect = chai.expect;
 const postAPI = '/api/users';
 const getAPI = postAPI;
 
+import {dataBaseFinishSeed} from "../src/finishSeed";
+
 
 describe('Users', () => {
+	beforeEach((done) => { //This hangs the tests until databaesFinish seed..
+		if(dataBaseFinishSeed){done();} //if it has finished seeding it will hit callback, else forever hangs
+		server.on('appStarted', () => { //Once finish seed, set to true and stop hang.
+			done();
+		});
+	});
 
 	describe('/GET users', () => {
 		it('should not GET if auth headers are missing', (done) => {