fix JTD serializer for Date objects, fixes #1691

ajv-validator · Jul 15, 2021 · 71a29b8 · 71a29b8
1 parent 001f829
commit 71a29b8
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/compile/jtd/serialize.ts b/lib/compile/jtd/serialize.ts
@@ -213,7 +213,7 @@ function serializeType(cxt: SerializeCxt): void {
     case "timestamp":
       gen.if(
         _`${data} instanceof Date`,
-        () => gen.add(N.json, _`${data}.toISOString()`),
+        () => gen.add(N.json, _`'"' + ${data}.toISOString() + '"'`),
         () => serializeString(cxt)
       )
       break

diff --git a/spec/jtd-timestamps.spec.ts b/spec/jtd-timestamps.spec.ts
@@ -67,4 +67,16 @@ describe("JTD timestamps", function () {
       assert.strictEqual(parseTS('"2021-05-14"')?.toISOString(), "2021-05-14T00:00:00.000Z")
     })
   })
+
+  describe("serializing Date objects", () => {
+    it("should serialize Date as JSON string", () => {
+      const schema: JTDSchemaType<Date> = {type: "timestamp"}
+      const ajv = new _AjvJTD()
+      const serializeTS = ajv.compileSerializer(schema)
+      assert.strictEqual(
+        serializeTS(new Date("2021-05-14T17:59:03.851Z")),
+        '"2021-05-14T17:59:03.851Z"'
+      )
+    })
+  })
 })