From 74471651c79efceab7eb8b5aa0cfc2cc6085b531 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Thu, 21 May 2026 14:03:32 +0200 Subject: [PATCH] Extend unit-test coverage for the `getPdfFilenameFromUrl` helper function Currently there's a couple of branches, specifically for dealing with corrupt URLs, that are not covered by tests. --- test/unit/display_utils_spec.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/unit/display_utils_spec.js b/test/unit/display_utils_spec.js index 0859107ad..c1b591ec3 100644 --- a/test/unit/display_utils_spec.js +++ b/test/unit/display_utils_spec.js @@ -151,6 +151,9 @@ describe("display_utils", function () { expect(getPdfFilenameFromUrl("/pdfs/%AA.pdf")).toEqual("%AA.pdf"); expect(getPdfFilenameFromUrl("/pdfs/%2F.pdf")).toEqual("%2F.pdf"); + + // A corrupt relative URL. + expect(getPdfFilenameFromUrl("//%%file.pdf")).toEqual("document.pdf"); }); it("gets PDF filename from (some) standard protocols", function () {