Merge pull request strongloop#3161 from strongloop/fix/role-properties

Add missing type to Role properties definition
mrjrdnthms · Jan 30, 2017 · bb939be · bb939be
2 parents 5fd79b1 + c072d06
commit bb939be
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/common/models/role.json b/common/models/role.json
@@ -12,11 +12,12 @@
       "required": true
     },
     "description": "string",
-
     "created": {
+      "type":"date",
       "defaultFn": "now"
     },
     "modified": {
+      "type":"date",
       "defaultFn": "now"
     }
   },

diff --git a/test/role.test.js b/test/role.test.js
@@ -96,6 +96,14 @@ describe('role model', function() {
     });
   });
 
+  it('should generate created/modified properties', () => {
+    return Role.create({name: 'ADMIN'})
+      .then(role => {
+        expect(role.toJSON().created).to.be.instanceOf(Date);
+        expect(role.toJSON().modified).to.be.instanceOf(Date);
+      });
+  });
+
   it('should define role/user relations', function(done) {
     User.create({name: 'Raymond', email: '[email protected]', password: 'foobar'}, function(err, user) {
       if (err) return done(err);