Merge pull request #1144 from bluewave-labs/feat/be/tests-pagespeed-check-module

Remove unused DB method, add tests for pageSpeedCheckModule

Server/db/mongo/MongoDB.js

@@ -118,7 +118,6 @@ import {
 
 import {
 	createPageSpeedCheck,
-	getPageSpeedChecks,
 	deletePageSpeedChecksByMonitorId,
 } from "./modules/pageSpeedCheckModule.js";
 
@@ -212,7 +211,6 @@ export default {
 	updateChecksTTL,
 	deleteMonitorsByUserId,
 	createPageSpeedCheck,
-	getPageSpeedChecks,
 	deletePageSpeedChecksByMonitorId,
 	createHardwareCheck,
 	createMaintenanceWindow,

Server/db/mongo/modules/pageSpeedCheckModule.js

@@ -25,25 +25,6 @@ const createPageSpeedCheck = async (pageSpeedCheckData) => {
 	}
 };
 
-/**
- * Get all PageSpeed checks for a monitor
- * @async
- * @param {string} monitorId
- * @returns {Promise<Array<PageSpeedCheck>>}
- * @throws {Error}
- */
-
-const getPageSpeedChecks = async (monitorId) => {
-	try {
-		const pageSpeedChecks = await PageSpeedCheck.find({ monitorId });
-		return pageSpeedChecks;
-	} catch (error) {
-		error.service = SERVICE_NAME;
-		error.method = "getPageSpeedChecks";
-		throw error;
-	}
-};
-
 /**
  * Delete all PageSpeed checks for a monitor
  * @async
@@ -63,4 +44,4 @@ const deletePageSpeedChecksByMonitorId = async (monitorId) => {
 	}
 };
 
-export { createPageSpeedCheck, getPageSpeedChecks, deletePageSpeedChecksByMonitorId };
+export { createPageSpeedCheck, deletePageSpeedChecksByMonitorId };

Server/tests/db/pageSpeedCheckModule.test.js

@@ -0,0 +1,65 @@
+import sinon from "sinon";
+import PageSpeedCheck from "../../db/models/PageSpeedCheck.js";
+import {
+	createPageSpeedCheck,
+	deletePageSpeedChecksByMonitorId,
+} from "../../db/mongo/modules/pageSpeedCheckModule.js";
+
+const mockPageSpeedCheck = {
+	monitorId: "monitorId",
+	bestPractices: 1,
+	seo: 1,
+	performance: 1,
+};
+
+const mockDeletedResult = { deletedCount: 1 };
+
+describe("pageSpeedCheckModule", () => {
+	let pageSpeedCheckSaveStub, pageSpeedCheckDeleteManyStub;
+	beforeEach(() => {
+		pageSpeedCheckSaveStub = sinon.stub(PageSpeedCheck.prototype, "save");
+		pageSpeedCheckDeleteManyStub = sinon.stub(PageSpeedCheck, "deleteMany");
+	});
+
+	afterEach(() => {
+		sinon.restore();
+	});
+
+	describe("createPageSpeedCheck", () => {
+		it("should return a page speed check", async () => {
+			pageSpeedCheckSaveStub.resolves(mockPageSpeedCheck);
+			const pageSpeedCheck = await createPageSpeedCheck(mockPageSpeedCheck);
+			expect(pageSpeedCheck).to.deep.equal(mockPageSpeedCheck);
+		});
+
+		it("should handle an error", async () => {
+			const err = new Error("test error");
+			pageSpeedCheckSaveStub.rejects(err);
+			try {
+				await expect(createPageSpeedCheck(mockPageSpeedCheck));
+			} catch (error) {
+				expect(error).to.exist;
+				expect(error).to.deep.equal(err);
+			}
+		});
+	});
+
+	describe("deletePageSpeedChecksByMonitorId", () => {
+		it("should return the number of deleted checks", async () => {
+			pageSpeedCheckDeleteManyStub.resolves(mockDeletedResult);
+			const result = await deletePageSpeedChecksByMonitorId("monitorId");
+			expect(result).to.deep.equal(mockDeletedResult.deletedCount);
+		});
+
+		it("should handle an error", async () => {
+			const err = new Error("test error");
+			pageSpeedCheckDeleteManyStub.rejects(err);
+			try {
+				await expect(deletePageSpeedChecksByMonitorId("monitorId"));
+			} catch (error) {
+				expect(error).to.exist;
+				expect(error).to.deep.equal(err);
+			}
+		});
+	});
+});