Merge branch 'tests'

backend/src/__tests__/unit/User/AuthUserService.spec.ts

@@ -0,0 +1,66 @@
+import faker from "faker";
+import AppError from "../../../errors/AppError";
+import AuthUserService from "../../../services/UserServices/AuthUserSerice";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("Auth", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be able to login with an existing user", async () => {
+    await CreateUserService({
+      name: faker.name.findName(),
+      email: "mail@test.com",
+      password: "hardpassword"
+    });
+
+    const response = await AuthUserService({
+      email: "mail@test.com",
+      password: "hardpassword"
+    });
+
+    expect(response).toHaveProperty("token");
+  });
+
+  it("should not be able to login with not registered email", async () => {
+    try {
+      await AuthUserService({
+        email: faker.internet.email(),
+        password: faker.internet.password()
+      });
+    } catch (err) {
+      expect(err).toBeInstanceOf(AppError);
+      expect(err.statusCode).toBe(401);
+      expect(err.message).toBe("ERR_INVALID_CREDENTIALS");
+    }
+  });
+
+  it("should not be able to login with incorret password", async () => {
+    await CreateUserService({
+      name: faker.name.findName(),
+      email: "mail@test.com",
+      password: "hardpassword"
+    });
+
+    try {
+      await AuthUserService({
+        email: "mail@test.com",
+        password: faker.internet.password()
+      });
+    } catch (err) {
+      expect(err).toBeInstanceOf(AppError);
+      expect(err.statusCode).toBe(401);
+      expect(err.message).toBe("ERR_INVALID_CREDENTIALS");
+    }
+  });
+});

backend/src/__tests__/unit/User/CreateUserService.spec.ts

@@ -0,0 +1,47 @@
+import faker from "faker";
+import AppError from "../../../errors/AppError";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("User", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be able to create a new user", async () => {
+    const user = await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    expect(user).toHaveProperty("id");
+  });
+
+  it("should not be able to create a user with duplicated email", async () => {
+    await CreateUserService({
+      name: faker.name.findName(),
+      email: "teste@sameemail.com",
+      password: faker.internet.password()
+    });
+
+    try {
+      await CreateUserService({
+        name: faker.name.findName(),
+        email: "teste@sameemail.com",
+        password: faker.internet.password()
+      });
+    } catch (err) {
+      expect(err).toBeInstanceOf(AppError);
+      expect(err.statusCode).toBe(400);
+    }
+  });
+});

backend/src/__tests__/unit/User/DeleteUserService.spec.ts

@@ -0,0 +1,35 @@
+import faker from "faker";
+import AppError from "../../../errors/AppError";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import DeleteUserService from "../../../services/UserServices/DeleteUserService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("User", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be delete a existing user", async () => {
+    const { id } = await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    expect(DeleteUserService(id)).resolves.not.toThrow();
+  });
+
+  it("to throw an error if tries to delete a non existing user", async () => {
+    expect(DeleteUserService(faker.random.number())).rejects.toBeInstanceOf(
+      AppError
+    );
+  });
+});

backend/src/__tests__/unit/User/ListUserService.spec.ts

@@ -0,0 +1,34 @@
+import faker from "faker";
+import User from "../../../models/User";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import ListUsersService from "../../../services/UserServices/ListUsersService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("User", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be able to list users", async () => {
+    await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    const response = await ListUsersService({
+      pageNumber: 1
+    });
+
+    expect(response).toHaveProperty("users");
+    expect(response.users[0]).toBeInstanceOf(User);
+  });
+});

backend/src/__tests__/unit/User/ShowUserService.spec.ts

@@ -0,0 +1,39 @@
+import faker from "faker";
+import AppError from "../../../errors/AppError";
+import User from "../../../models/User";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import ShowUserService from "../../../services/UserServices/ShowUserService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("User", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be able to find a user", async () => {
+    const newUser = await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    const user = await ShowUserService(newUser.id);
+
+    expect(user).toHaveProperty("id");
+    expect(user).toBeInstanceOf(User);
+  });
+
+  it("should not be able to find a inexisting user", async () => {
+    expect(ShowUserService(faker.random.number())).rejects.toBeInstanceOf(
+      AppError
+    );
+  });
+});

backend/src/__tests__/unit/User/UpdateUserService.spec.ts

@@ -0,0 +1,68 @@
+import faker from "faker";
+import AppError from "../../../errors/AppError";
+import CreateUserService from "../../../services/UserServices/CreateUserService";
+import UpdateUserService from "../../../services/UserServices/UpdateUserService";
+import { disconnect, truncate } from "../../utils/database";
+
+describe("User", () => {
+  beforeEach(async () => {
+    await truncate();
+  });
+
+  afterEach(async () => {
+    await truncate();
+  });
+
+  afterAll(async () => {
+    await disconnect();
+  });
+
+  it("should be able to find a user", async () => {
+    const newUser = await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    const updatedUser = await UpdateUserService({
+      userId: newUser.id,
+      userData: {
+        name: "New name",
+        email: "newmail@email.com"
+      }
+    });
+
+    expect(updatedUser).toHaveProperty("name", "New name");
+    expect(updatedUser).toHaveProperty("email", "newmail@email.com");
+  });
+
+  it("should not be able to updated a inexisting user", async () => {
+    const userId = faker.random.number();
+    const userData = {
+      name: faker.name.findName(),
+      email: faker.internet.email()
+    };
+
+    expect(UpdateUserService({ userId, userData })).rejects.toBeInstanceOf(
+      AppError
+    );
+  });
+
+  it("should not be able to updated an user with invalid data", async () => {
+    const newUser = await CreateUserService({
+      name: faker.name.findName(),
+      email: faker.internet.email(),
+      password: faker.internet.password()
+    });
+
+    const userId = newUser.id;
+    const userData = {
+      name: faker.name.findName(),
+      email: "test.worgn.email"
+    };
+
+    expect(UpdateUserService({ userId, userData })).rejects.toBeInstanceOf(
+      AppError
+    );
+  });
+});

backend/src/__tests__/utils/database.ts

@@ -0,0 +1,11 @@
+import database from "../../database";
+
+const truncate = async (): Promise<void> => {
+  await database.truncate({ force: true, cascade: true });
+};
+
+const disconnect = async (): Promise<void> => {
+  return database.connectionManager.close();
+};
+
+export { truncate, disconnect };

backend/src/app.ts

@@ -0,0 +1,41 @@
+import "./bootstrap";
+import "reflect-metadata";
+import "express-async-errors";
+import express, { Request, Response, NextFunction } from "express";
+import cors from "cors";
+import cookieParser from "cookie-parser";
+import * as Sentry from "@sentry/node";
+
+import "./database";
+import uploadConfig from "./config/upload";
+import AppError from "./errors/AppError";
+import routes from "./routes";
+
+Sentry.init({ dsn: process.env.SENTRY_DSN });
+
+const app = express();
+
+app.use(
+  cors({
+    credentials: true,
+    origin: process.env.FRONTEND_URL
+  })
+);
+app.use(cookieParser());
+app.use(express.json());
+app.use(Sentry.Handlers.requestHandler());
+app.use("/public", express.static(uploadConfig.directory));
+app.use(routes);
+
+app.use(Sentry.Handlers.errorHandler());
+
+app.use(async (err: Error, req: Request, res: Response, _: NextFunction) => {
+  if (err instanceof AppError) {
+    return res.status(err.statusCode).json({ error: err.message });
+  }
+
+  console.error(err);
+  return res.status(500).json({ error: "Internal server error" });
+});
+
+export default app;

backend/src/bootstrap.ts

@@ -0,0 +1,5 @@
+import dotenv from "dotenv";
+
+dotenv.config({
+  path: process.env.NODE_ENV === "test" ? ".env.test" : ".env"
+});

backend/src/config/database.ts

@@ -1,4 +1,4 @@
-require("dotenv/config");
+require("../bootstrap");
 
 module.exports = {
   define: {

backend/src/server.ts

@@ -1,48 +1,10 @@
-import "reflect-metadata";
-import "dotenv/config";
-import "express-async-errors";
-import express, { Request, Response, NextFunction } from "express";
-import cors from "cors";
-import cookieParser from "cookie-parser";
-import * as Sentry from "@sentry/node";
-
-import uploadConfig from "./config/upload";
-import "./database";
-import AppError from "./errors/AppError";
-import routes from "./routes";
+import app from "./app";
 import { initIO } from "./libs/socket";
 import { StartWhatsAppSessions } from "./services/WbotServices/StartWhatsAppSessions";
 
-Sentry.init({ dsn: process.env.SENTRY_DSN });
-
-const app = express();
-
-app.use(
-  cors({
-    credentials: true,
-    origin: process.env.FRONTEND_URL
-  })
-);
-app.use(cookieParser());
-app.use(express.json());
-app.use(Sentry.Handlers.requestHandler());
-app.use("/public", express.static(uploadConfig.directory));
-app.use(routes);
-
 const server = app.listen(process.env.PORT, () => {
   console.log(`Server started on port: ${process.env.PORT}`);
 });
 
 initIO(server);
 StartWhatsAppSessions();
-
-app.use(Sentry.Handlers.errorHandler());
-
-app.use(async (err: Error, req: Request, res: Response, _: NextFunction) => {
-  if (err instanceof AppError) {
-    return res.status(err.statusCode).json({ error: err.message });
-  }
-
-  console.error(err);
-  return res.status(500).json({ error: "Internal server error" });
-});

backend/src/services/UserServices/CreateUserService.ts

@@ -32,13 +32,10 @@ const CreateUserService = async ({
         "Check-email",
         "An user with this email already exists.",
         async value => {
-          if (value) {
-            const emailExists = await User.findOne({
-              where: { email: value }
-            });
-            return !emailExists;
-          }
-          return false;
+          const emailExists = await User.findOne({
+            where: { email: value! }
+          });
+          return !emailExists;
         }
       ),
     password: Yup.string().required().min(5)

backend/src/services/UserServices/DeleteUserService.ts

@@ -3,7 +3,7 @@ import AppError from "../../errors/AppError";
 import Ticket from "../../models/Ticket";
 import UpdateDeletedUserOpenTicketsStatus from "../../helpers/UpdateDeletedUserOpenTicketsStatus";
 
-const DeleteUserService = async (id: string): Promise<void> => {
+const DeleteUserService = async (id: string | number): Promise<void> => {
   const user = await User.findOne({
     where: { id }
   });

backend/src/services/UserServices/ListUsersService.ts

@@ -3,7 +3,7 @@ import User from "../../models/User";
 
 interface Request {
   searchParam?: string;
-  pageNumber?: string;
+  pageNumber?: string | number;
 }
 
 interface Response {

backend/src/services/UserServices/UpdateUserService.ts

@@ -12,7 +12,7 @@ interface UserData {
 
 interface Request {
   userData: UserData;
-  userId: string;
+  userId: string | number;
 }
 
 interface Response {