linter fixes, unit tests

src/server/controllers/global-settings/get-global-settings-controller.spec.ts

@@ -0,0 +1,31 @@
+import { getGlobalSettingsController } from "./get-global-settings-controller"
+import { Response } from "express"
+import { AllowedRoles } from "../../middleware/authorization-middleware"
+import { IGetUserAuthInfoRequest } from "../../middleware/request.model"
+import { db } from "../../../db/db"
+
+jest.mock("../../../db/db")
+const mockResponse = () => {
+    const res: Partial<Response> = {}
+    res.json = jest.fn().mockReturnValue(res)
+    res.status = jest.fn().mockReturnValue(res)
+    return res
+}
+describe("getGlobalSettingsController", () => {
+    it("should return global settings", async () => {
+        const response = mockResponse()
+        const querySpy = jest.spyOn(require("../../queries/global-settings"), "getGlobalSettings")
+        const request = {
+            user: {
+                role: AllowedRoles.Admin,
+            },
+        }
+        db.one = jest.fn().mockReturnValueOnce({ project_auto_provisioning: true })
+        await getGlobalSettingsController(
+            request as unknown as IGetUserAuthInfoRequest,
+            response as unknown as Response)
+        expect(querySpy).toHaveBeenCalledTimes(1)
+        expect(response.json).toHaveBeenCalledTimes(1)
+        expect(response.json).toHaveBeenCalledWith({ projectAutoProvisioning: true })
+    })
+})

src/server/controllers/global-settings/update-global-settings-controller.spec.ts

@@ -0,0 +1,36 @@
+import { db } from "../../../db/db"
+import { AllowedRoles } from "../../middleware/authorization-middleware"
+import { IGetUserAuthInfoRequest } from "../../middleware/request.model"
+import { Response } from "express"
+import { updateGlobalSettingsController } from "./update-global-settings-controller"
+
+jest.mock("../../../db/db")
+const mockResponse = () => {
+    const res: Partial<Response> = {}
+    res.send = jest.fn().mockReturnValue(res)
+    res.status = jest.fn().mockReturnValue(res)
+    return res
+}
+
+describe("updateGlobalSettingsController", () => {
+    it("should save the new settings into database", async () => {
+        const response = mockResponse()
+        const querySpy = jest.spyOn(require("../../queries/global-settings"), "updateGlobalSettings")
+        const request = {
+            user: {
+                role: AllowedRoles.Admin,
+            },
+            body: {
+                projectAutoProvisioning: true,
+            },
+        }
+        db.none = jest.fn().mockResolvedValueOnce(null)
+
+        await updateGlobalSettingsController(
+            request as unknown as IGetUserAuthInfoRequest,
+            response as unknown as Response)
+        expect(querySpy).toHaveBeenCalledTimes(1)
+        expect(querySpy).toHaveBeenCalledWith(true)
+        expect(response.send).toHaveBeenCalledTimes(1)
+    })
+})

src/server/middleware/project-exists-middleware.spec.ts

@@ -1,6 +1,6 @@
 import { IGetUserAuthInfoRequest } from "./request.model"
 import { Response, NextFunction } from "express"
-import {projectAutoProvisioningMiddleware, projectExistsMiddleware} from "./project-exists-middleware"
+import { projectAutoProvisioningMiddleware, projectExistsMiddleware } from "./project-exists-middleware"
 import { db } from "../../db/db"
 import Boom = require("boom")
 

src/server/routes/item.ts

@@ -33,7 +33,7 @@ import { getItemChartSettingsController } from "../controllers/item/get-item-cha
 import { AllowedRoles, authorizationMiddleware } from "../middleware/authorization-middleware"
 import { authenticationMiddleware } from "../middleware/authentication-middleware"
 import { getRequestStatsExportController } from "../controllers/item/get-request-stats-export-controller"
-import {projectAutoProvisioningMiddleware, projectExistsMiddleware} from "../middleware/project-exists-middleware"
+import { projectAutoProvisioningMiddleware, projectExistsMiddleware } from "../middleware/project-exists-middleware"
 
 export class ItemsRoutes {