fix: unique constraint on user email

Author: jeremyben

src/__tests__/users.ts

@@ -32,6 +32,13 @@ describe('Register user', () => {
 			.expect(400, /required/i)
 	})
 
+	test('[SAD] Email already exists', () => {
+		return rq
+			.post('/register')
+			.send({ email: '[email protected]', password: 'azerty123' })
+			.expect(400, /already/i)
+	})
+
 	test('Alternative routes', async () => {
 		const signupRes = await rq.post('/signup')
 		expect(signupRes.notFound).toBe(false)

src/users.ts

@@ -57,6 +57,13 @@ const create: RequestHandler = (req, res, next) => {
 		throw Error('You must bind the router db to the app')
 	}
 
+	// prettier-ignore
+	const existingUser = db.get('users').find({ email }).value()
+	if (existingUser) {
+		res.status(400).jsonp('Email already exists')
+		return
+	}
+
 	bcrypt
 		.hash(password, SALT_LENGTH)
 		.then((hash) => {