test: add tests for valid input formats

nbesli · Jan 27, 2024 · c7dc5d0 · c7dc5d0
1 parent 00a3777
commit c7dc5d0
Show file tree

Hide file tree

Showing 7 changed files with 357 additions and 175 deletions.
diff --git a/PDFMergerBase.js b/PDFMergerBase.js
@@ -1,6 +1,6 @@
 import { PDFDocument } from 'pdf-lib'
 
-import { parsePagesString } from './parsePagesString'
+import { parsePagesString } from './parsePagesString.js'
 
 /**
  * @typedef {Object} Metadata
@@ -14,10 +14,6 @@ import { parsePagesString } from './parsePagesString'
  * @typedef {Uint8Array | ArrayBuffer | Blob | URL} PdfInput
  */
 
-/**
- * @class PDFMergerBase
- * @classdesc Base class for PDFMerger
- */
 export default class PDFMergerBase {
   /**
    * The internal pdf-lib document.

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -32,6 +32,7 @@
     "pdf-lib": "^1.17.1"
   },
   "devDependencies": {
+    "cross-blob": "^3.0.2",
     "fs-extra": "^11.2.0",
     "jest": "^29.7.0",
     "jest-environment-jsdom": "^29.7.0",

diff --git a/test/browser-fixtures.test.js → test/PDFMerger-browser.test.js b/test/browser-fixtures.test.js → test/PDFMerger-browser.test.js
@@ -34,7 +34,7 @@ async function readFixtureAsUint8Array (file) {
 // Note: The browser tests differ from standard as all files are expected
 // to be generated or fetched before being passed into the merger.
 // For testing, they are retrieved with fs.ReadFile() and then passed in.
-describe('PDFMerger', () => {
+describe('PDFMerger for browser', () => {
   beforeAll(async () => {
     await fs.ensureDir(TMP_DIR)
     fileA = await readFixtureAsUint8Array('Testfile_A.pdf')