Both types of tests are now working

server.js

@@ -25,6 +25,24 @@ app.use(mw.admin);
 app.use(require('./server/brew.api.js'));
 app.use(require('./server/interface.routes.js'));
 
+setTimeout(()=>{
+	var test = require('./server/brew.data.js');
+
+	test.create({text : 'test'})
+		.then((brew) => {
+			console.log(brew);
+		})
+		.catch(console.log);
+
+	test.get({})
+	.then((brew) => {
+		console.log(brew);
+	})
+	.catch(console.log);
+
+
+}, 1000);
+
 
 //app.use(require('./server/admin.api.js'));
 

server/brew.data.js

@@ -84,15 +84,6 @@ const BrewData = {
 		const newBrew = new Brew(brew);
 
 		return newBrew.save();
-
-		//TODO: add error decorators to the catches
-		const temp = newBrew.save().then(()=>{
-			console.log('SAVED');
-		});
-
-		console.log(typeof temp, _.keys(temp), temp);
-
-		return temp;
 	},
 	update : (newBrew) => {
 		return Brew.findOneAndUpdate({ editId : newBrew.editId },

server/db.js

@@ -6,13 +6,18 @@ const dbPath = process.env.MONGODB_URI || process.env.MONGOLAB_URI || 'mongodb:/
 module.exports = {
 	connect : ()=>{
 		return new Promise((resolve, reject)=>{
-			mongoose.createConnection(dbPath,
+			if(mongoose.connection.readyState == 1){
+				console.log('already connected');
+				return resolve();
+			}
+			mongoose.connect(dbPath,
 				(err) => {
 					if(err){
 						console.log('Error : Could not connect to a Mongo Database.');
 						console.log('        If you are running locally, make sure mongodb.exe is running.');
 						return reject(err);
 					}
+					console.log('mongo connected.');
 					return resolve();
 				}
 			);

test/brew.apitest.js

@@ -1,15 +1,14 @@
-
+const should = require('chai').use(require('chai-as-promised')).should();
 const request = require('supertest-as-promised');
 const app = require('../server.js');
 
 
 const apiPath = '/api/brew';
 
-describe('/api/brew', () => {
 
+describe(apiPath, () => {
 
 	describe('POST', () => {
-
 		it('creates a new brew', () => {
 			return request(app)
 				.post(apiPath)
@@ -18,8 +17,12 @@ describe('/api/brew', () => {
 				})
 				.expect(200)
 				.then((res) => {
-					console.log(res.body);
+					const brew = res.body;
-				});
+					should.exist(brew);
+					brew.should.have.property('editId').that.is.a('string');
+					brew.should.have.property('shareId').that.is.a('string');
+					brew.should.have.property('text').that.is.a('string');
+				})
 		});
 
 	});

test/brew.datatest.js

@@ -1,38 +1,24 @@
-
 const should = require('chai').use(require('chai-as-promised')).should();
 
 const BrewDB = require('../server/db.js');
 const BrewData = require('../server/brew.data.js');
 
 
-describe.skip('BrewDB', () => {
+describe('BrewDB', () => {
-
 	before('Await DB', ()=>{
-		return BrewDB.connect()
+		return BrewDB.connect().catch()
-			.then(()=>{
-				console.log('connected');
-			})
-			.catch(()=>{
-				console.log('sdfdsfdsfdsf');
-			})
 	});
 
-	it('generates ID on save', (done) => {
+	it('generates ID on save', () => {
-		console.log('getting here');
 		return BrewData.create({
 			text : "Brew Text"
 		}).then((brew) => {
-			console.log('here2');
 			should.exist(brew);
 			brew.should.have.property('editId').that.is.a('string');
 			brew.should.have.property('shareId').that.is.a('string');
 			brew.should.have.property('text').that.is.a('string');
-			console.log('and I am done');
-			done();
-		})
-		.catch((e) => {
-			console.log('an error', e);
 		})
+
 	});
 
 });