fix: default createdAt to now on account creation (#88)

thanks @devguykyle
hoodiehq · Feb 8, 2018 · 76f8b26 · 76f8b26
1 parent 3ddd933
commit 76f8b26
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/lib/accounts/add.js b/lib/accounts/add.js
@@ -39,7 +39,7 @@ function addAccount (state, properties, options) {
     type: 'user',
     name: lowercaseUsername,
     password: properties.password,
-    createdAt: properties.createdAt,
+    createdAt: properties.createdAt || new Date().toISOString(),
     signedUpAt: properties.signedUpAt,
     roles: [
       'id:' + accountId

diff --git a/test/unit/accounts/add-test.js b/test/unit/accounts/add-test.js
@@ -81,6 +81,28 @@ test('addAccount', function (group) {
     .catch(t.catch)
   })
 
+  group.test('sets default createdAt', function (t) {
+    t.plan(1)
+
+    var state = {
+      cache: {
+        set: simple.stub().resolveWith({})
+      },
+      setupPromise: Promise.resolve()
+    }
+
+    addAccount(state, {
+      username: 'foo'
+    })
+
+    .then(function () {
+      var doc = state.cache.set.lastCall.arg
+      t.notEqual(doc.createdAt, undefined)
+    })
+
+    .catch(t.catch)
+  })
+
   group.test('with existing username', function (t) {
     t.plan(3)