add support for uuidv4 data type (#691)

mickhansen · Dec 20, 2019 · 4538569 · 4538569
1 parent 5774259
commit 4538569
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 0 deletions.
diff --git a/src/typeMapper.js b/src/typeMapper.js
@@ -50,6 +50,7 @@ export function toGraphQL(sequelizeType, sequelizeTypes) {
     STRING,
     TEXT,
     UUID,
+    UUIDV4,
     DATE,
     DATEONLY,
     TIME,
@@ -80,6 +81,7 @@ export function toGraphQL(sequelizeType, sequelizeTypes) {
       sequelizeType instanceof STRING ||
       sequelizeType instanceof TEXT ||
       sequelizeType instanceof UUID ||
+      sequelizeType instanceof UUIDV4 ||
       sequelizeType instanceof DATEONLY ||
       sequelizeType instanceof TIME ||
       sequelizeType instanceof BIGINT ||

diff --git a/test/unit/defaultArgs.test.js b/test/unit/defaultArgs.test.js
@@ -59,6 +59,22 @@ describe('defaultArgs', function () {
     expect(args.uuid.type).to.equal(GraphQLString);
   });
 
+  it('should return a key for a UUIDV4 primary key', function () {
+    var Model
+      , args;
+
+    Model = sequelize.define('DefaultArgModel', {
+      uuidv4: {
+        type: Sequelize.UUIDV4,
+        primaryKey: true
+      }
+    });
+
+    args = defaultArgs(Model);
+
+    expect(args.uuidv4.type).to.equal(GraphQLString);
+  });
+
   it('should return multiple keys for a compound primary key', function () {
     var Model
       , args;

diff --git a/test/unit/typeMapper.test.js b/test/unit/typeMapper.test.js
@@ -18,6 +18,7 @@ const {
   STRING,
   TEXT,
   UUID,
+  UUIDV4,
   DATE,
   DATEONLY,
   TIME,
@@ -175,6 +176,12 @@ describe('typeMapper', () => {
     });
   });
 
+  describe('UUIDV4', function () {
+    it('should map to GraphQLString', function () {
+      expect(toGraphQL(new UUIDV4(), Sequelize)).to.equal(GraphQLString);
+    });
+  });
+
   describe('VIRTUAL', function () {
 
     it('should map to the sequelize return type', function () {