added a negative scebario for testing

Author: phshafi

__test__/pet.test.js

@@ -3,16 +3,6 @@ const request = require("supertest");
 
 describe("Pet Service", () => {
     describe("Normal scenario", () => {
-        it("should save the pet in the database", async () => {
-            const response = await request(app).post("/api/pet").send({
-                id: 5,
-                name: "Sandy",
-                type: "Hamster"
-            });
-
-            expect(response.headers["content-type"]).toEqual(expect.stringContaining("application/json"));
-        });
-
         it("should response with 201 when new pet created with all the fields", async () => {
             const response = await request(app).post("/api/pet").send({
                 id: 5,
@@ -33,4 +23,27 @@ describe("Pet Service", () => {
             expect(response.headers["content-type"]).toEqual(expect.stringContaining("application/json"));
         });
     });
+
+    describe("Negative scenario", () => {
+        it("should response with 400 when name ro type is missing", async () => {
+            const bodyData = [
+                {
+                    id: 5,
+                    name: "Sandy"
+                },
+                {
+                    id: 5,
+                    type: "Hamster"
+                },
+                {
+                    id: 5
+                }
+            ];
+
+            for (const data of bodyData) {
+                const response = await request(app).post("/api/pet").send(data);
+                expect(response.statusCode).toBe(400);
+            }
+        });
+    });
 });

app.js

@@ -6,8 +6,4 @@ app.use(express.json());
 
 app.use("/api", routes);
 
-app.listen("8080", () => {
-    console.log("Server is up and running");
-});
-
 module.exports = app;

package.json

@@ -4,8 +4,8 @@
   "description": "An introduction of testing nodejs API using jest and supertest",
   "main": "app.js",
   "scripts": {
-    "start": "nodemon app.js",
-    "test": "jest"
+    "start": "nodemon server.js",
+    "test": "jest --watch"
   },
   "keywords": [
     "nodejs",

routes/pet.js

@@ -3,8 +3,14 @@ const pets = require("../data");
 
 router.post("/", async (req, res, next) => {
     const body = req.body;
+    const { name, type } = body;
+    if (!name || !type) {
+        res.status(400).json("Bad Request");
+        return;
+    }
+
     pets.push(body);
-    res.status(200).json("Added successfully");
+    res.status(201).json("Added successfully");
 });
 
 module.exports = router;

server.js

@@ -0,0 +1,5 @@
+const app = require("./app");
+
+app.listen("8080", () => {
+    console.log("Server is up and running");
+});