fix: add tests for 404 response on non-existent files and update logic for URL handling

Author: bjohansebas

src/utils/getFilenameFromUrl.js

@@ -125,6 +125,7 @@ function getFilenameFromUrl(context, url, extra = {}) {
             filename = path.join(filename, "index.html");
           }
         }
+
         extra.stats = context.outputFileSystem.statSync(filename);
       } catch {
         continue;

test/middleware.test.js

@@ -1728,6 +1728,10 @@ describe.each([
                 value: "noextension",
                 code: 200,
               },
+              {
+                value: "noextension/",
+                code: 404,
+              },
             ],
           },
           {
@@ -1772,6 +1776,11 @@ describe.each([
                 contentType: "text/plain; charset=utf-8",
                 code: 200,
               },
+              {
+                value: "windows%202.txt/",
+                contentType: get404ContentTypeHeader(name),
+                code: 404,
+              },
             ],
           },
           {
@@ -1937,7 +1946,7 @@ describe.each([
 
                   expect(response.statusCode).toEqual(code);
 
-                  if (data) {
+                  if (data && code !== 404) {
                     expect(response.headers["content-length"]).toEqual(
                       String(data.length),
                     );
@@ -5179,6 +5188,15 @@ describe.each([
             "text/html; charset=utf-8",
           );
         });
+
+        it('should return the "404" code for the "GET" request with a non-existent file', async () => {
+          const response = await req.get("/default.html/");
+
+          expect(response.statusCode).toBe(404);
+          expect(response.headers["content-type"]).toBe(
+            get404ContentTypeHeader(name),
+          );
+        });
       });
 
       describe('should work with "string" value with a custom extension', () => {