test and lint

src/controllers/testPageController.ts

@@ -1,9 +1,9 @@
 import { Request, Response } from "express";
-import {AppLocals} from "../types/app.js";
+import { AppLocals } from "../types/app.js";
 
 export class TestPageController {
     static getTestPage = (req: Request, res: Response) => {
         const { rootDir } = req.app.locals as AppLocals;
-        res.sendFile(`${rootDir}/static/test.html`);
-    }
-}
+        res.sendFile(`${rootDir}/static/test.html`); // TODO: use path
+    };
+}

src/routes.ts

@@ -3,7 +3,7 @@ import { IndexController } from "./controllers/indexController";
 import { TileController } from "./controllers/tileController";
 import notFound from "./errors/notFound";
 import { MetadataController } from "./controllers/metadataController";
-import {TestPageController} from "./controllers/testPageController.js";
+import { TestPageController } from "./controllers/testPageController.js";
 
 export const registerRoutes = () => {
     const router = Router();

static/test.html

@@ -74,8 +74,6 @@ <h1>GROUT Test Page</h1>
             map.options.minZoom = 3;
             map.setMaxBounds(map.getBounds());
 
-
-            //const nameProp = "NAME_1";
             const groutServers = {
                 "local": "http://localhost:5000",
                 "mrcdata": "https://mrcdata.dide.ic.ac.uk/grout"

tests/unit/controllers/testPageController.spec.ts

@@ -0,0 +1,19 @@
+import { describe, expect, test, vi } from "vitest";
+import { TestPageController } from "../../../src/controllers/testPageController";
+
+describe("TestPageController", () => {
+    test("sends test page file", () => {
+        const req = {
+            app: {
+                locals: {
+                    rootDir: "testRoot"
+                }
+            }
+        } as any;
+        const res = {
+            sendFile: vi.fn()
+        } as any;
+        TestPageController.getTestPage(req, res);
+        expect(res.sendFile).toHaveBeenCalledWith("testRoot/static/test.html");
+    });
+});

tests/unit/routes.spec.ts

@@ -4,6 +4,7 @@ import { IndexController } from "../../src/controllers/indexController";
 import { TileController } from "../../src/controllers/tileController";
 import notFound from "../../src/errors/notFound";
 import { MetadataController } from "../../src/controllers/metadataController";
+import { TestPageController } from "../../src/controllers/testPageController";
 
 const { mockRouterConstructor, mockRouter } = vi.hoisted(() => {
     const mockRouter = {
@@ -38,6 +39,11 @@ describe("registerRoutes", () => {
             "/tile/:dataset/:level/:z/:x/:y",
             TileController.getTile
         );
+        expect(mockRouter.get).toHaveBeenNthCalledWith(
+            4,
+            "/test",
+            TestPageController.getTestPage
+        );
         expect(mockRouter.use).toHaveBeenCalledWith(notFound);
     });
 });