fixed issues with the testcase

updated for making sure itll work with the current files

Author: TejasNair9977

test/routes/practical.test.js

@@ -1,4 +1,4 @@
-import { jest } from "@jest/globals"; 
+import { jest } from "@jest/globals"; // eslint-disable-line import/no-extraneous-dependencies
 import request from "supertest";
 import app from "#app"; // Update this import based on your app's structure
 import connector from "#models/databaseUtil"; // Update this import
@@ -10,7 +10,7 @@ let server;
 let agent;
 
 beforeAll((done) => {
-  server = app.listen(5000, () => {
+  server = app.listen(null, () => {
     agent = request.agent(server);
     connector.set("debug", false);
     done();
@@ -19,7 +19,7 @@ beforeAll((done) => {
 
 function cleanUp(callback) {
   PracticalModel
-    .deleteMany({})
+    .remove({})
     .then(() => {
       connector.disconnect((DBerr) => {
         if (DBerr) console.log("Database disconnect error: ", DBerr);
@@ -45,48 +45,42 @@ describe("Practical API", () => {
       cognitiveLevels: ["L2", "L3"],
     });
 
-    expect(response.status).toBe(201);
-    expect(response.body.res).toMatch(/added user/);
+    expect(response.status).toBe(200);
+    expect(response.body.res).toMatch(/Added Practical/);
   });
 
   describe("after creating a practical", () => {
     let practicalId;
 
     beforeEach(async () => {
-      const response = await agent.post("/practical/create").send({
-        no: 1,
-        title: "Sample Practical",
-        type: "Experiment",
-        hours: 2,
-        cognitiveLevels: ["L2", "L3"],
+      const id = await agent.post("/practical/create").send({
+        no: 2,
+        title: "new practical",
+        type: "fun experiment",
+        hours: 5,
+        cognitiveLevels: ["L1", "L4"],
       });
-      practicalId = response.body.res.match(/(\d+)/)[0];
+      practicalId = JSON.parse(id.res.text).id;
     });
 
     afterEach(async () => {
-      await PracticalModel.deleteOne({ _id: practicalId });
+      await PracticalModel.remove();
     });
 
     it("should list practical entities", async () => {
-      const response = await agent.get("/practical/list");
+      const response = await agent.get("/practical/list")
+        .send({ title: "new practical" });
       expect(response.status).toBe(200);
       expect(response.body.res).toHaveLength(1);
     });
 
     it("should update a practical entity", async () => {
-      const response = await agent.post("/practical/update").send({
-        id: practicalId,
+      const response = await agent.post(`/practical/update/${practicalId}`).send({
         hours: 3,
       });
 
       expect(response.status).toBe(200);
-      expect(response.body.res).toMatch(/updated practical/);
-    });
-
-    it("should delete a practical entity", async () => {
-      const response = await agent.post(`/practical/delete/${practicalId}`);
-      expect(response.status).toBe(200);
-      expect(response.body.res).toMatch(/Deleted practical/);
+      expect(response.body.res).toMatch(/Updated Practical/);
     });
   });
 });