added unit test for validatePhoneNumber method

Author: phshafi

__test__/pet.test.js

@@ -2,7 +2,7 @@ const app = require("../app");
 const request = require("supertest");
 
 describe("Pet Service", () => {
-    describe("Normal scenario", () => {
+    describe("when a valid object is posted", () => {
         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,
@@ -24,7 +24,7 @@ describe("Pet Service", () => {
         });
     });
 
-    describe("Negative scenario", () => {
+    describe("when name and type is missing", () => {
         it("should response with 400 when name or type is missing for the pet", async () => {
             const bodyData = [
                 {

__test__/utils.test.js

@@ -0,0 +1,54 @@
+const { validatePhoneNumber } = require("../helper/utils");
+
+it("should return false if non number string is passed", () => {
+    expect(validatePhoneNumber("onetwothree")).toBe(false);
+});
+
+test("it returns false when a string argument is passed", () => {
+    expect(validatePhoneNumber("onetwothree")).toBe(false);
+});
+
+describe("Validate Phone Number", () => {
+    describe("when valid phone number is passed", () => {
+        it("should return true for a valid Bangladeshi phone number", () => {
+            const phoneNumber = "01622971272";
+            expect(validatePhoneNumber(phoneNumber)).toBe(true);
+        });
+
+        it('should return true for a valid Bangladeshi phone number wth the "+" sign', () => {
+            const phoneNumber = "+8801622971272";
+            expect(validatePhoneNumber(phoneNumber)).toBe(true);
+        });
+        it('should return true for a valid Bangladeshi phone number without the "+" sign', () => {
+            const phoneNumber = "8801622971272";
+            expect(validatePhoneNumber(phoneNumber)).toBe(true);
+        });
+
+        it('should return true for a valid Bangladeshi phone number the "0088" prefix', () => {
+            const phoneNumber = "008801622971272";
+            expect(validatePhoneNumber(phoneNumber)).toBe(true);
+        });
+    });
+
+    describe("when invalid phone number is passed", () => {
+        it("should return false for an invalid phone number", () => {
+            const phoneNumber = "+8801234";
+            expect(validatePhoneNumber(phoneNumber)).toBe(false);
+        });
+
+        it("should return false for a phone number with an incorrect digit", () => {
+            const phoneNumber = "+88012345678A";
+            expect(validatePhoneNumber(phoneNumber)).toBe(false);
+        });
+
+        it("should return false for a phone number with more than 11 digits", () => {
+            const phoneNumber = "+88012345678901";
+            expect(validatePhoneNumber(phoneNumber)).toBe(false);
+        });
+
+        it("should return false for a phone number with less than 11 digits", () => {
+            const phoneNumber = "+88012345678";
+            expect(validatePhoneNumber(phoneNumber)).toBe(false);
+        });
+    });
+});

helper/utils.js

@@ -0,0 +1,5 @@
+exports.validatePhoneNumber = (phoneNumber) => {
+    const regex = /^(?:\+?88|0088)?01[15-9]\d{8}$/;
+    const isValid = regex.test(phoneNumber);
+    return isValid;
+};

package.json

@@ -5,7 +5,7 @@
   "main": "app.js",
   "scripts": {
     "start": "nodemon server.js",
-    "test": "jest --watch"
+    "test": "jest --watch --verbose"
   },
   "keywords": [
     "nodejs",