test: add unit tests for getRequestURL function

Author: bjohansebas

test/utils/getRequestURL.test.js

@@ -0,0 +1,43 @@
+import { getRequestURL } from "../../src/utils.js";
+
+describe("getRequestURL", () => {
+  it("should use getURL() when available (pseudo API)", () => {
+    const req = { getURL: () => "/pseudo-url" };
+    expect(getRequestURL(req)).toBe("/pseudo-url");
+  });
+
+  it("should return req.url when no special API is present", () => {
+    const req = { url: "/basic" };
+    expect(getRequestURL(req)).toBe("/basic");
+  });
+
+  describe("Fastify (req.originalUrl present)", () => {
+    it("should return req.originalUrl when req.url was not modified", () => {
+      // Fastify decodes req.url, so originalUrl preserves encoding
+      const req = {
+        url: "/path with spaces",
+        originalUrl: "/path%20with%20spaces",
+      };
+      expect(getRequestURL(req)).toBe("/path%20with%20spaces");
+    });
+
+    it("should return encoded req.url when middleware (e.g. connect-history-api-fallback) modified it", () => {
+      // Middleware changed req.url to a different path
+      const req = { url: "/index.html", originalUrl: "/path%20with%20spaces" };
+      expect(getRequestURL(req)).toBe("/index.html");
+    });
+
+    it("should encode req.url when middleware sets a path with special chars", () => {
+      const req = { url: "/new path", originalUrl: "/old%20path" };
+      expect(getRequestURL(req)).toBe("/new%20path");
+    });
+
+    it("should preserve query string from modified req.url", () => {
+      const req = {
+        url: "/index.html?redirect=/other",
+        originalUrl: "/some%20page",
+      };
+      expect(getRequestURL(req)).toBe("/index.html?redirect=/other");
+    });
+  });
+});