refactor: change all toCalledWith to toHaveBeenCalledWith

README.md

@@ -182,7 +182,7 @@ test("should Teste.paragraph is called with valid param", () => {
  sut.paragraph("willian");
 
  // aqui eu verifico se a dependência foi chamada com o parâmetro correto
- expect(makeParagraphSpy).toBeCalledWith("willian");
+ expect(makeParagraphSpy).toHaveBeenCalledWith("willian");
 });
 ```
 

jest-mongodb-config.js

@@ -4,7 +4,7 @@ module.exports = {
  dbName: "jest",
  },
  binary: {
- version: "4.0.3",
+ version: "7.0.0",
  skipMD5: true,
  },
  autoStart: false,

package.json

@@ -15,44 +15,44 @@
  "test:ci": "yarn test -- --coverage"
  },
  "devDependencies": {
- "@shelf/jest-mongodb": "^4.0.0",
+ "@shelf/jest-mongodb": "^4.2.0",
  "@techmmunity/eslint-config": "^5.2.3",
- "@types/bcrypt": "^5.0.0",
- "@types/express": "^4.17.13",
- "@types/jest": "^28.1.6",
- "@types/jsonwebtoken": "^8.5.8",
+ "@types/bcrypt": "^5.0.2",
+ "@types/express": "^4.17.21",
+ "@types/jest": "^29.5.12",
+ "@types/jsonwebtoken": "^9.0.6",
  "@types/pino": "^7.0.5",
- "@types/pino-http": "^5.8.1",
- "@types/pino-pretty": "^4.7.5",
- "@types/request-ip": "^0.0.39",
- "@types/supertest": "^2.0.12",
+ "@types/pino-http": "^5.8.4",
+ "@types/pino-pretty": "^5.0.0",
+ "@types/request-ip": "^0.0.41",
+ "@types/supertest": "^6.0.2",
  "@types/uuidv4": "^5.0.0",
- "@types/validator": "^13.7.4",
- "eslint": "^8.20.0",
- "git-commit-msg-linter": "^4.1.3",
- "husky": "^8.0.1",
- "jest": "^28.1.3",
- "lint-staged": "^13.0.3",
- "prettier": "^2.7.1",
- "supertest": "^6.2.4",
- "ts-jest": "^28.0.7",
+ "@types/validator": "^13.11.9",
+ "eslint": "^8.57.0",
+ "git-commit-msg-linter": "^5.0.7",
+ "husky": "^9.0.11",
+ "jest": "^29.7.0",
+ "lint-staged": "^15.2.2",
+ "prettier": "^3.2.5",
+ "supertest": "^6.3.4",
+ "ts-jest": "^29.1.2",
  "ts-node-dev": "^2.0.0",
- "tsx": "^3.12.7",
- "typescript": "5.1.6"
+ "tsx": "^4.7.1",
+ "typescript": "5.4.3"
  },
  "dependencies": {
- "bcrypt": "^5.0.1",
- "dotenv": "^16.0.3",
- "express": "^4.18.1",
- "express-rate-limit": "^7.1.0",
- "jsonwebtoken": "^8.5.1",
- "mongodb": "^4.8.0",
- "pino": "^8.3.0",
- "pino-http": "^8.1.1",
- "pino-pretty": "^8.1.0",
+ "bcrypt": "^5.1.1",
+ "dotenv": "^16.4.5",
+ "express": "^4.19.2",
+ "express-rate-limit": "^7.2.0",
+ "jsonwebtoken": "^9.0.2",
+ "mongodb": "^6.5.0",
+ "pino": "^8.19.0",
+ "pino-http": "^9.0.0",
+ "pino-pretty": "^11.0.0",
  "request-ip": "^3.3.0",
- "tsup": "^6.7.0",
+ "tsup": "^8.0.2",
  "uid": "^2.0.0",
- "validator": "^13.7.0"
+ "validator": "^13.11.0"
  }
 }

src/data/usecase/add-account/add-account.spec.ts

@@ -79,7 +79,7 @@ describe("DbAddAccount", () => {
  password: "valid_password",
  });
 
- expect(addAccountRepositorySpy).toBeCalledWith({
+ expect(addAccountRepositorySpy).toHaveBeenCalledWith({
  name: "valid_name",
  email: "[email protected]",
  password: "hashed_value",

src/data/usecase/get-account-by-email/get-account-by-email.spec.ts

@@ -40,7 +40,7 @@ describe("GetAccountByEmail", () => {
 
  await sut.get("[email protected]");
 
- expect(getAccountByEmailRepositorySpy).toBeCalledWith(
+ expect(getAccountByEmailRepositorySpy).toHaveBeenCalledWith(
  "[email protected]"
  );
  });

src/data/usecase/reset-refresh-token/reset-refresh-token.spec.ts

@@ -35,7 +35,7 @@ describe("DbResetRefreshToken", () => {
 
  await sut.reset(userId);
 
- expect(resetRefreshTokenRepositorySpy).toBeCalledWith("validUserId");
+ expect(resetRefreshTokenRepositorySpy).toHaveBeenCalledWith("validUserId");
  });
 
  test("should return string if success", async () => {

src/main/factories/refresh-token/refresh-token-validation.spec.ts

@@ -14,6 +14,6 @@ describe("TokenInfo Validation", () => {
  validations.push(new RequiredFieldValidation(field));
  }
 
- expect(ValidationComposite).toBeCalledWith(validations);
+ expect(ValidationComposite).toHaveBeenCalledWith(validations);
  });
 });

src/main/factories/sign-in/sign-in-validation.spec.ts

@@ -21,6 +21,6 @@ describe("TokenInfo Validation", () => {
  new PasswordValidation("password", new UtilPasswordValidator())
  );
 
- expect(ValidationComposite).toBeCalledWith(validations);
+ expect(ValidationComposite).toHaveBeenCalledWith(validations);
  });
 });

src/main/factories/sign-up/sign-up-validation.spec.ts

@@ -27,6 +27,6 @@ describe("TokenInfo Validation", () => {
  new PasswordValidation("password", new UtilPasswordValidator())
  );
 
- expect(ValidationComposite).toBeCalledWith(validations);
+ expect(ValidationComposite).toHaveBeenCalledWith(validations);
  });
 });

src/main/factories/token-info/token-info-validation.spec.ts

@@ -14,6 +14,6 @@ describe("TokenInfo Validation", () => {
  validations.push(new RequiredFieldValidation(field));
  }
 
- expect(ValidationComposite).toBeCalledWith(validations);
+ expect(ValidationComposite).toHaveBeenCalledWith(validations);
  });
 });

src/presentation/controller/refresh-token/refresh-token.spec.ts

@@ -76,15 +76,15 @@ describe("RefreshToken Controller", () => {
  const checkRefreshTokenSpy = checkRefreshTokenStub.check;
  await sut.handle(makeFakeHttpRequest());
 
- expect(checkRefreshTokenSpy).toBeCalledWith("valid_access_token");
+ expect(checkRefreshTokenSpy).toHaveBeenCalledWith("valid_access_token");
  });
 
  test("should create access is called with correct id", async () => {
  const { sut, createAccessTokenStub } = makeSut();
  const createAccessTokenSpy = createAccessTokenStub.create;
  await sut.handle(makeFakeHttpRequest());
 
- expect(createAccessTokenSpy).toBeCalledWith("valid_user_id");
+ expect(createAccessTokenSpy).toHaveBeenCalledWith("valid_user_id");
  });
 
  test("should return 200 if success", async () => {

src/presentation/controller/sign-in/sign-in.spec.ts

@@ -60,14 +60,14 @@ describe("SignIn Controller", () => {
  const { sut, resetRefreshTokenStub } = makeSut();
  await sut.handle(makeFakeHttpRequest());
 
- expect(resetRefreshTokenStub.reset).toBeCalledWith("valid_id");
+ expect(resetRefreshTokenStub.reset).toHaveBeenCalledWith("valid_id");
  });
 
  test("should encrypter.compare is called with correct values", async () => {
  const { sut, encrypterStub } = makeSut();
  await sut.handle(makeFakeHttpRequest());
 
- expect(encrypterStub.compare).toBeCalledWith(
+ expect(encrypterStub.compare).toHaveBeenCalledWith(
  "valid_password",
  "hashed_password"
  );

src/presentation/controller/sign-up/sign-up.spec.ts

@@ -91,7 +91,7 @@ describe("Sign-Up", () => {
  const getAccountByEmailSpy = jest.spyOn(getAccountByEmailStub, "get");
  await sut.handle(makeFakeHttpRequest());
 
- expect(getAccountByEmailSpy).toBeCalledWith("[email protected]");
+ expect(getAccountByEmailSpy).toHaveBeenCalledWith("[email protected]");
  });
 
  test("should returns statusCode 409 if account already exists", async () => {
@@ -124,7 +124,7 @@ describe("Sign-Up", () => {
  const addAccountSpy = jest.spyOn(addAccountStub, "add");
  await sut.handle(makeFakeHttpRequest());
 
- expect(addAccountSpy).toBeCalledWith({
+ expect(addAccountSpy).toHaveBeenCalledWith({
  name: "valid_name",
  email: "[email protected]",
  password: "valid_password",

src/presentation/controller/token-info/token-info.spec.ts

@@ -89,7 +89,7 @@ describe("TokenInfo controller", () => {
  const getTokenInfoSpy = jest.spyOn(getTokenInfoStub, "get");
  await sut.handle(makeFakeHttpRequest());
 
- expect(getTokenInfoSpy).toBeCalledWith("valid_access_token");
+ expect(getTokenInfoSpy).toHaveBeenCalledWith("valid_access_token");
  });
 
  test("should return statusCode 500 if GetTokenInfo dependency throws", async () => {
@@ -122,7 +122,7 @@ describe("TokenInfo controller", () => {
  const validateSpy = jest.spyOn(validationStub, "validate");
  await sut.handle(makeFakeHttpRequest());
 
- expect(validateSpy).toBeCalledWith(makeFakeHttpRequest().header);
+ expect(validateSpy).toHaveBeenCalledWith(makeFakeHttpRequest().header);
  });
 
  test("should return 400 if Validation returns an error", async () => {