Use '0', not 'null'

tancredi · Feb 8, 2021 · ea4ad73 · ea4ad73
1 parent ad45ea8
commit ea4ad73
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/generators/asset-types/__tests__/ttf.ts b/src/generators/asset-types/__tests__/ttf.ts
@@ -26,17 +26,17 @@ describe('`TTF` font generator', () => {
 
     expect(svg2ttf).toHaveBeenCalledTimes(1);
     expect(svg2ttf).toHaveBeenCalledWith(svg, {
-      ts: null,
+      ts: 0,
       __mock: 'options__'
     });
     expect(result).toEqual(Buffer.from(`::ttf(${svg})::`));
   });
 
-  test('passes correctly format options to `svg2ttf` and sets `ts` (timestamp) to `null` by default to avoid generating unnecessary diffs', async () => {
+  test('passes correctly format options to `svg2ttf` and sets `ts` (timestamp) to `0` by default to avoid generating unnecessary diffs', async () => {
     const formatOptions = { foo: 'bar' };
     await ttfGen.generate(mockOptions(formatOptions), svg);
 
     expect(svg2ttf).toHaveBeenCalledTimes(1);
-    expect(svg2ttf.mock.calls[0][1]).toEqual({ ts: null, ...formatOptions });
+    expect(svg2ttf.mock.calls[0][1]).toEqual({ ts: 0, ...formatOptions });
   });
 });
diff --git a/src/generators/asset-types/ttf.ts b/src/generators/asset-types/ttf.ts
@@ -6,7 +6,7 @@ const generator: FontGenerator<string> = {
   dependsOn: FontAssetType.SVG,
 
   async generate({ formatOptions }, svg) {
-    const font = svg2ttf(svg, { ts: null, ...(formatOptions?.ttf || {}) });
+    const font = svg2ttf(svg, { ts: 0, ...(formatOptions?.ttf || {}) });
     return Buffer.from(font.buffer);
   }
 };