Merge pull request #198 from agiledev-students-fall2023/update-tests

Updated tests for login, signup, forgot password, and logout
agiledev-students-fall2023 · Dec 5, 2023 · 107af8e · 107af8e
2 parents 02a371e + d13f59e
commit 107af8e
Show file tree

Hide file tree

Showing 4 changed files with 73 additions and 10 deletions.
diff --git a/back-end/test/forgotPasswordRoute.test.js b/back-end/test/forgotPasswordRoute.test.js
@@ -11,11 +11,35 @@ describe("POST for Forgot Password", () => {
       chai
         .request(app)
         .post("/forgot-password")
-        .send({ username: "testuser", password: "testnewpassword" })
+        .send({
+          email: "[email protected]",
+          username: "candy",
+          newPassword: "candy01",
+        })
         .end((err, res) => {
           expect(res).to.have.status(200);
           expect(res.body).to.be.a("object");
-          expect(res.body).to.have.property("status", "Success");
+          expect(res.body).to.have.property("success", true);
+          expect(res.body).to.have.property("message");
+          done();
+        });
+    });
+  });
+
+  describe("POST /forgot-password", () => {
+    it("should return an unsuccessful response", (done) => {
+      chai
+        .request(app)
+        .post("/forgot-password")
+        .send({
+          email: "[email protected]",
+          username: "candy",
+          newPassword: "candy01",
+        })
+        .end((err, res) => {
+          expect(res).to.have.status(404);
+          expect(res.body).to.be.a("object");
+          expect(res.body).to.have.property("success", false);
           expect(res.body).to.have.property("message");
           done();
         });

diff --git a/back-end/test/loginRoute.test.js b/back-end/test/loginRoute.test.js
@@ -3,19 +3,37 @@ const chaiHttp = require("chai-http");
 const app = require("../app");
 const expect = chai.expect;
 
+const should = chai.should(); // the same assertion library in the style using the word 'should'
+
 chai.use(chaiHttp);
 
 describe("POST for Login", () => {
-  describe("POST /", () => {
-    it("should return a success response", (done) => {
+  describe("POST / with correct login credentials", () => {
+    it("should return a 200 success response", (done) => {
       chai
         .request(app)
         .post("/")
-        .send({ username: "testuser", password: "testpassword" })
+        .send({ username: "cindy", password: "hi" })
         .end((err, res) => {
           expect(res).to.have.status(200);
           expect(res.body).to.be.a("object");
-          expect(res.body).to.have.property("status", "Success");
+          expect(res.body).to.have.property("success", true);
+          expect(res.body).to.have.property("message");
+          done();
+        });
+    });
+  });
+
+  describe("POST / with incorrect login credentials", () => {
+    it("should return a 401 unauthorized response", (done) => {
+      chai
+        .request(app)
+        .post("/")
+        .send({ username: "incorrect", password: "incorrect" })
+        .end((err, res) => {
+          expect(res).to.have.status(401);
+          expect(res.body).to.be.a("object");
+          expect(res.body).to.have.property("success", false);
           expect(res.body).to.have.property("message");
           done();
         });

diff --git a/back-end/test/logoutRoute.test.js b/back-end/test/logoutRoute.test.js
@@ -14,6 +14,7 @@ describe(" GET Endpoint for logout", () => {
         .end((err, res) => {
           expect(res).to.have.status(200);
           expect(res.body).to.be.a("object");
+          expect(res.body).to.have.property("success", true);
           done();
         });
     });

diff --git a/back-end/test/signupRoute.test.js b/back-end/test/signupRoute.test.js
@@ -12,14 +12,34 @@ describe("POST for Signup", () => {
         .request(app)
         .post("/signup")
         .send({
-          email: "testemail",
-          username: "testuser",
-          password: "testpassword",
+          email: "[email protected]",
+          username: "wrongnewtestuser01129",
+          password: "wrongnewtestpassword",
         })
         .end((err, res) => {
           expect(res).to.have.status(200);
           expect(res.body).to.be.a("object");
-          expect(res.body).to.have.property("status", "Success");
+          expect(res.body).to.have.property("success", true);
+          expect(res.body).to.have.property("message");
+          done();
+        });
+    });
+  });
+
+  describe("POST /signup", () => {
+    it("should return a unsuccessful response", (done) => {
+      chai
+        .request(app)
+        .post("/signup")
+        .send({
+          email: "[email protected]",
+          username: "cindy",
+          password: "hi",
+        })
+        .end((err, res) => {
+          expect(res).to.have.status(409);
+          expect(res.body).to.be.a("object");
+          expect(res.body).to.have.property("success", false);
           expect(res.body).to.have.property("message");
           done();
         });