Fix stringifyVariables for Files

urql-graphql · Mar 20, 2020 · 686e933 · 686e933
1 parent 954a715
commit 686e933
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/packages/core/src/utils/stringifyVariables.test.ts b/packages/core/src/utils/stringifyVariables.test.ts
@@ -29,7 +29,19 @@ it('throws for circular structures', () => {
   }).toThrow();
 });
 
-it('stringifies date correctly', () => {
+it('stringifies dates correctly', () => {
   const date = new Date('2019-12-11T04:20:00');
   expect(stringifyVariables(date)).toBe(date.toJSON());
 });
+
+it('stringifies files correctly', () => {
+  const file = new File([0] as any, 'test.js');
+  Object.defineProperty(file, 'lastModified', { value: 123 });
+
+  expect(stringifyVariables(file)).toBe(
+    stringifyVariables({
+      name: 'test.js',
+      lastModified: 123,
+    })
+  );
+});
diff --git a/packages/core/src/utils/stringifyVariables.ts b/packages/core/src/utils/stringifyVariables.ts
@@ -26,6 +26,8 @@ const stringify = (x: any): string => {
     return out;
   } else if (seen.has(x)) {
     throw new TypeError('Converting circular structure to JSON');
+  } else if (typeof File === 'function' && x instanceof File) {
+    return stringify({ name: x.name, lastModified: x.lastModified });
   }
 
   const keys = Object.keys(x).sort();