feat(user-Test): implementing sign up using jest

Author: Tuyisenge2

.github/workflows/node.js.yml

@@ -1,18 +1,16 @@
 # This workflow will install node dependencies, run tests and report coverage
 # For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-nodejs
-
 name: build
 
 on:
   push:
     branches:
-      - feat-test-CI-implementation-187419042
-      - "*"
       - develop
-
+      - "*"
   pull_request:
     branches:
       - develop
+      - "*"
 
 jobs:
   test:
@@ -22,6 +20,11 @@ jobs:
       DB_TEST_URL: ${{ secrets.DB_TEST_URL }}
       DEV_MODE: ${{ secrets.DEV_MODE }}
       DB_HOSTED_MODE: ${{ secrets.DB_HOSTED_MODE }}
+      ACCESS_TOKEN_SECRET: ${{ secrets.ACCESS_TOKEN_SECRET }}
+      SESSION_SECRET: ${{ secrets.SESSION_SECRET }}
+      JWT_SECRET: ${{ secrets.JWT_SECRET }}
+      CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
+      COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
 
     strategy:
       matrix:
@@ -37,15 +40,29 @@ jobs:
       - name: Install dependencies
         run: npm install
 
+      - name: Run tests
+        run: npm run test
+        
       - name: Run tests and build test coverage
         run: npm run test:ci
 
-      - name: Test & publish code climate coverage
-        uses: paambaati/[email protected]
-        env:
-          CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
-        with:
-          coverageCommand: npm run test:ci
-          debug: true
-          coverageLocations: |
-            ${{github.workspace}}/*.lcov:lcov
+      - name: Setup Code Climate test-reporter
+        run: |
+          # Download test reporter as a static binary
+          curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+          chmod +x ./cc-test-reporter
+          ./cc-test-reporter before-build
+      - name: Run tests
+        run: npm run test
+
+      - name: Store coverage report
+        if: always()
+        run: mkdir -p coverage
+      
+      
+      - name: Send coverage report to Code Climate
+        if: always()
+        run: ./cc-test-reporter after-build -t lcov -p coverage
+
+      - name: coveralls
+        run: npx coveralls < coverage/lcov.info

src/__test__/login.test.ts

@@ -1,38 +1,13 @@
 import app from "../app";
 import request from "supertest";
-// import { connectionToDatabase } from "../database/config/db.config";
-
 jest.setTimeout(30000);
 
-describe("USER API TEST", () => {
-	// beforeAll(async () => {
-	//   await connectionToDatabase();
-	// });
-
-	it("Should return 200 and list of all users", async () => {
-		// Your test implementation goes here
-	});
-
-	it("should create a new user", async () => {
-		// Your test implementation goes here
-	});
-
-	it("should return validation errors when required fields are missing", async () => {
-		// Your test implementation goes here
-	});
-
-	it("should return an error if the email is already in use", async () => {
-		// Your test implementation goes here
-	});
-});
-
 import { connectionToDatabase } from "../database/config/db.config";
 import { deleteTableData } from "../utils/database.utils";
 import { User } from "../database/models/User";
 import {
   NewUser,
   user_bad_request,
-  User_without_email,
   exist_user,
 } from "../mock/static";
 import { createUser } from "../services/user.services";
@@ -62,7 +37,6 @@ describe("USER API TEST", () => {
   /**
    * ----------------------------register new user --------------------------------------------
    */
-  describe("FAILED SIGN UP", () => {
     it("it should return a user not found and status 400", async () => {
       const { body } = await Jest_request.post("/api/v1/users/register")
         .send(user_bad_request)
@@ -76,15 +50,12 @@ describe("USER API TEST", () => {
       expect(body.status).toStrictEqual("CONFLICT");
       expect(body.message).toStrictEqual("User already exist!");
     });
-  });
-  describe("SUCCCESSFULL SIGN UP WITH TOKEN CREATED", () => {
     it("it should  register a user and return 201", async () => {
       const { body } = await Jest_request.post("/api/v1/users/register")
         .send(NewUser)
         .expect(201);
       expect(body.status).toStrictEqual("SUCCESS");
       expect(body.message).toStrictEqual("Account Created successfully!");
       expect(body.token).toBeDefined();
-    });
   });
 });

src/__test__/product.test.ts

@@ -36,47 +36,10 @@ const registerUser = async (
   } catch (error) {
     res
       .status(500)
-      .json(new HttpException("SERVER FAIL", "Something went wrong!"));
+      .json(new HttpException("SERVER FAILS", "Something went wrong!"));
   }
 };
 
-const login = async (req: Request, res: Response, next: NextFunction) => {
-	passport.authenticate(
-		"login",
-		(error: Error, user: UserModelAttributes, info: InfoAttribute) => {
-			if (error) {
-				return res
-					.status(400)
-					.json(new HttpException("BAD REQUEST", "Bad Request!"));
-			}
-
-			if (info) {
-				return res
-					.status(409)
-					.json(new HttpException("CONFLICT", info.message));
-			}
-
-			(req as any).login(user, (err: Error) => {
-				if (err) {
-					return res
-						.status(400)
-						.json(new HttpException("BAD REQUEST", "Bad Request!"));
-				}
-
-				const { id, role } = user;
-
-				const token = generateAccessToken({ id, role });
-				const response = new HttpException(
-					"SUCCESS",
-					"Logged in to you account successfully!",
-				).response();
-				return res.status(200).json({ ...response, token });
-			});
-		},
-	)(req, res, next);
-};
-
 export default {
-	login,
-	registerUser,
+	registerUser
 };

src/__test__/users.test.ts

@@ -34,22 +34,21 @@ DB_HOST_MODE === "local"
 export const sequelizeConnection: Sequelize = new Sequelize(db_uri, {
 	dialect: "postgres",
 	dialectOptions: dialect_option,
-	logging: true,
+	logging: false,
 	pool: {
 		max: 10,
 		min: 0,
 		acquire: 30000,
 		idle: 10000,
 	},
 });
-
-export const connectionToDatabase = () =>
-	sequelizeConnection
-		.authenticate()
-		.then(() => {
-			console.log("Database connected successfully.", db_uri);
-		})
-		.catch((error) => {
-			console.error("Unable to connect to the database:", error);
-			process.exit(1);
-		});
+export const connectionToDatabase = async () => {
+	try {
+	  await sequelizeConnection.authenticate();
+	  await sequelizeConnection.sync();
+	  console.log("Database connected successfully.", db_uri);
+	} catch (error) {
+	  console.log("Unable to connect to the database:", error);
+	  process.exit(1);
+	}
+  }

src/controllers/userController.ts

@@ -8,6 +8,5 @@ userRoutes.post(
   userMiddleware.userValid,
   userController.registerUser
 );
-userRoutes.post("/login", userMiddleware.logInValidated, userController.login);
 
 export default userRoutes;

src/database/config/db.config.ts

@@ -1,7 +1,3 @@
-import {
-  connectionToDatabase,
-  sequelizeConnection,
-} from "../database/config/db.config";
 
 export const deleteTableData = async (Model: any, tableName: string) => {
   try {