Continued fixing bugs/typos

test/thread_post.js

@@ -12,7 +12,7 @@ chai.use(require('chai-http'))
 chai.use(require('chai-things'))
 
 describe('Thread and post', () => {
-	let userAgent
+	let userAgent, replyAgent
 
 	//Wait for app to start before commencing
 	before((done) => {
@@ -24,6 +24,7 @@ describe('Thread and post', () => {
 
 		function mockData() {
 			userAgent = chai.request.agent(server)
+			replyAgent = chai.request.agent(server)
 
 			userAgent
 				.post('/api/v1/user')
@@ -143,22 +144,181 @@ describe('Thread and post', () => {
 		})
 	})
 
+	describe('POST /post', () => {
+		it('should create a post if logged in', async () => {
+			let res = await userAgent
+				.post('/api/v1/post')
+				.set('content-type', 'application/json')
+				.send({
+					content: 'content',
+					threadId: 1
+				})
+
+			res.should.be.json
+			res.should.have.status(200)
+			res.body.should.have.property('content', '<p>content</p>\n')
+			res.body.should.have.deep.property('User.username', 'username')
+			res.body.should.have.deep.property('Thread.name', 'thread')
+
+		})
+		it('should return an error if not logged in', async () => {
+			try {
+				let res = await chai.request(server)
+					.post('/api/v1/post')
+					.set('content-type', 'application/json')
+					.send({
+						content: 'content',
+						threadId: 1
+					})
+
+				res.should.be.json
+				res.should.have.status(401)
+				res.body.errors.should.contain.something.that.deep.equals(Errors.requestNotAuthorized)
+			} catch (res) {
+				res.should.have.status(401)
+				JSON.parse(res.response.text).errors.should.contain.something.that.deep.equals(Errors.requestNotAuthorized)
+			}
+		})
+		it('should return an error if missing parameters', async () => {
+			try {
+				let res = await userAgent
+					.post('/api/v1/post')
+
+				res.should.be.json
+				res.should.have.status(400)
+				res.body.errors.should.contain.something.that.deep.equals(Errors.missingParameter('content'))
+				res.body.errors.should.contain.something.that.deep.equals(Errors.missingParameter('thread'))
+			} catch (res) {
+				let body = JSON.parse(res.response.text)
+				res.should.have.status(400)
+				body.errors.should.contain.something.that.deep.equals(Errors.missingParameter('content'))
+				body.errors.should.contain.something.that.deep.equals(Errors.missingParameter('thread'))
+			}
+		})
+		it('should return an error if invalid types', async () => {
+			try {
+				let res = await userAgent
+					.post('/api/v1/thread')
+					.set('content-type', 'application/json')
+					.send({
+						content: 123,
+						threadId: 'string',
+						replyingToId: 'string'
+					})
+
+				res.should.be.json
+				res.should.have.status(400)
+				res.body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('content', 'string'))
+				res.body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('thread', 'integer'))
+				res.body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('replyingToId', 'integer'))
+			} catch (res) {
+				let body = JSON.parse(res.response.text)
+				res.should.have.status(400)
+				body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('content', 'string'))
+				body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('thread', 'integer'))
+				body.errors.should.contain.something.that.deep.equals(Errors.invalidParameterType('replyingToId', 'integer'))
+			}
+		})
+		it('should return an error if thread id does not exist', async () => {
+			try {
+				let res = await userAgent
+					.post('/api/v1/post')
+					.set('content-type', 'application/json')
+					.send({
+						content: 'content',
+						threadId: 10
+					})
+
+				res.should.be.json
+				res.should.have.status(400)
+				res.body.should.include.something.that.deep.equals(Errors.invalidParameter('thread', 'thread does not exist'))
+			} catch (e) {
+				let body = JSON.parse(res.response.text)
+				res.should.have.status(400)
+				body.should.include.something.that.deep.equals(Errors.invalidParameter('thread', 'thread does not exist'))
+			}
+		})
+		it('should be able to reply to a post', async () => {
+			await replyAgent
+				.post('/api/v1/user')
+				.set('content-type', 'application/json')
+				.send({
+					username: 'username1',
+					password: 'password'
+				})
+
+			let res = replyAgent
+				.post('/api/v1/post')
+				.set('content-type', 'application/json')
+				.send({
+					content: 'another post',
+					threadId: 1,
+					replyingToId: 1
+				})
+
+			res.should.be.json
+			res.should.have.status(200)
+			res.body.should.have.property('content', '<p>another post</p>\n')
+			res.body.should.have.deep.property('User.username', 'username1')
+			res.body.should.have.deep.property('Thread.name', 'thread')
+			res.body.should.have.deep.property('replyingToId.username', 'username')
+		})
+		it('should return an error if reply id does not exist', async () => {
+			try {
+				let res = replyAgent
+					.post('/api/v1/post')
+					.set('content-type', 'application/json')
+					.send({
+						content: 'another post',
+						threadId: 1,
+						replyingToId: 10
+					})
+
+				res.should.have.status(400)
+				res.body.should.contain.something.that.deep.equals(Errors.invalidParameter('replyingToId', 'post does not exist'))
+			} catch (e) {
+				let body = JSON.parse(res.response.text)
+
+				res.should.have.status(400)
+				body.should.contain.something.that.deep.equals(Errors.invalidParameter('replyingToId', 'post does not exist'))
+			}
+		})
+		it('should return an error if post reply not in same thread', async () => {
+			try {
+				let threadId = (await replyAgent
+					.post('/api/v1/thread')
+					.set('content-type', 'application/json')
+					.send({
+						name: 'another thread',
+						category: 'category_name'
+					})).body.id
+
+				let res = await replyAgent
+					.post('/api/v1/post')
+					.set('content-type', 'application/json')
+					.send({
+						content: 'another post',
+						threadId: threadId,
+						replyingToId: 1
+					})
+
+				res.should.have.status(400)
+				res.body.should.contain.something.that.deep.equals(Errors.invalidParameter('replyingToId', 'replies must be in same thread'))
+			} catch (res) {
+				let body = JSON.parse(res.response.text)
+
+				res.should.have.status(400)
+				body.should.contain.something.that.deep.equals(Errors.invalidParameter('replyingToId', 'replies must be in same thread'))
+			}
+		})
+	})
+
 	describe('GET /thread/:id', () => {
 		it('should return the thread and corresponding posts', async () => {})
 		it('should return an error if :id is invalid', async () => {})
 	})
 
-	describe('POST /post', () => {
-		it('should create a post if logged in', async () => {})
-		it('should return an error if not logged in', async () => {})
-		it('should return an error if missing parameters', async () => {})
-		it('should return an error if invalid types', async () => {})
-		it('should return an error if thread id does not exist', async () => {})
-
-		it('should be able to reply to a post', async () => {})
-		it('should return an error if reply id does not exist', async () => {})
-		it('should return an error if post reply not in same thread', async () => {})
-	})
+	
 
 	describe('GET /post/:id', () => {
 		it('should return the post', async () => {})