From 15921243d4821db6b4a52dd70165238fdb47bd79 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 8 May 2026 12:30:00 +0200 Subject: [PATCH] Make the `PDFDataRangeTransport` streaming unit-tests faster The delay between chunks when testing streaming is necessary to avoid the entire PDF document arriving all at once, since that would render those unit-tests somewhat pointless. However, the delay is unnecessarily large which causes these unit-tests to be slower than necessary. Also, update the range unit-tests to check the expected number of fetches *exactly* since those values are not supposed to vary. --- test/unit/api_spec.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/unit/api_spec.js b/test/unit/api_spec.js index 3f0c5fe35..84a54a799 100644 --- a/test/unit/api_spec.js +++ b/test/unit/api_spec.js @@ -5207,7 +5207,7 @@ have written that much by now. So, here’s to squashing bugs.`); describe("PDFDataRangeTransport", function () { async function streamDelay() { return new Promise(resolve => { - setTimeout(resolve, 250); + setTimeout(resolve, 100); }); } let dataPromise; @@ -5269,7 +5269,7 @@ have written that much by now. So, here’s to squashing bugs.`); expect(contentDispositionFilename).toEqual("aaa.pdf"); expect(contentLength).toEqual(dataLength); - expect(fetches).toBeGreaterThan(4); + expect(fetches).toEqual(5); expect(initialProgress).toEqual({ loaded: initialDataLength, @@ -5428,7 +5428,7 @@ have written that much by now. So, here’s to squashing bugs.`); expect(contentDispositionFilename).toEqual(null); expect(contentLength).toEqual(dataLength); - expect(fetches).toBeGreaterThan(2); + expect(fetches).toEqual(3); expect(initialProgress.loaded).toBeGreaterThan(initialDataLength); expect(initialProgress.total).toEqual(dataLength);