fix: tests coverage restored to 100%

morfeojs · Oct 28, 2021 · 9abb112 · 9abb112
1 parent 46b95ef
commit 9abb112
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/packages/fonts/src/mountFont.ts b/packages/fonts/src/mountFont.ts
@@ -65,15 +65,14 @@ function getFontCss(font: MountFontParams) {
  * ```
  */
 export function mountFont(font: MountFontParams) {
-  const css = getFontCss(font);
-
-  const newFontStyle = `
-    <style class="morfeo-font" id="font-${font.name}">
-      ${css}
-    </style>
-  `;
-
   if (globalThis && globalThis.document) {
+    const css = getFontCss(font);
+
+    const newFontStyle = `
+      <style class="morfeo-font" id="font-${font.name}">
+        ${css}
+      </style>
+    `;
     unmountFont(font.name);
     document.head.innerHTML += newFontStyle;
   }

diff --git a/packages/fonts/tests/mountFont.test.ts b/packages/fonts/tests/mountFont.test.ts
@@ -63,4 +63,13 @@ describe('mountFont', () => {
       expect(boldStyle?.textContent).toContain('@import');
     });
   });
+
+  describe('when document is not defined', () => {
+    test('should not fail', () => {
+      Object.defineProperty(global, 'globalThis', { get: () => {} });
+      mountFont({ name: 'should-not-be-added' } as any);
+      const element = document.getElementById('font-should-not-be-added');
+      expect(element).toBeNull();
+    });
+  });
 });