feat: accounts.add({}) (#10)

hoodiehq · Oct 16, 2016 · 1597def · 1597def
1 parent bd11905
commit 1597def
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/lib/accounts/add.js b/lib/accounts/add.js
@@ -9,6 +9,11 @@ function addAccount (state, properties, options) {
   if (!options) {
     options = {}
   }
+
+  if (properties.username === '') {
+    return Promise.reject(errors.USERNAME_EMPTY)
+  }
+
   var accountKey = 'org.couchdb.user:' + properties.username
   var accountId = properties.id || uuid.v4()
 

diff --git a/lib/utils/errors.js b/lib/utils/errors.js
@@ -31,3 +31,9 @@ module.exports.USERNAME_EXISTS = hoodieError({
   message: 'An account with that username already exists',
   status: 409
 })
+
+module.exports.USERNAME_EMPTY = hoodieError({
+  name: 'Bad Request',
+  message: 'username must be set',
+  status: 400
+})
diff --git a/test/unit/accounts/add-test.js b/test/unit/accounts/add-test.js
@@ -3,7 +3,7 @@ var test = require('tap').test
 var addAccount = require('../../../lib/accounts/add')
 
 test('addAccount', function (group) {
-  group.test('with empty username', {skip: true}, function (t) {
+  group.test('with empty username', function (t) {
     t.plan(3)
 
     var state = {