fix: throw error if URL is provided but fetch is not available

nbesli · Jan 27, 2024 · af96727 · af96727
1 parent 8bca3b7
commit af96727
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/PDFMergerBase.js b/PDFMergerBase.js
@@ -162,6 +162,9 @@ export default class PDFMergerBase {
 
     // see https://developer.mozilla.org/en-US/docs/Web/API/URL
     if (input instanceof URL) {
+      if (typeof fetch === 'undefined') {
+        throw new Error('fetch is not defined. You need to use a polyfill for this to work in Node.js.')
+      }
       const res = await fetch(input)
       const aBuffer = await res.arrayBuffer()
       return new Uint8Array(aBuffer)

diff --git a/test/PDFMerger.test.js b/test/PDFMerger.test.js
@@ -121,7 +121,8 @@ describe('PDFMerger', () => {
       expect(diff).toBeFalsy()
     })
 
-    test('provide input as URL', async () => {
+    const testIfFetch = typeof fetch !== 'undefined' ? test : test.skip
+    testIfFetch('provide input as URL', async () => {
       const merger = new PDFMerger()
       const TestfileAURL = new URL('https://raw.githubusercontent.com/nbesli/pdf-merger-js/master/test/fixtures/Testfile_A.pdf')
       expect(TestfileAURL).toBeInstanceOf(URL)