Guardian Register #5

dansenpir · 2023-04-17T15:13:22Z

Introduces the tutor registration feature.

…nal.api into feature/user-register

… is provided

…is not provided

…ided

src/data/use-cases/db-add-guardian.ts

src/infra/criptography/bcrypt-adapter.ts

src/main/adapters/express-route-adapter.ts

src/main/factories/controllers/signup-controller-factory.ts

tests/application/controllers/signup.spec.ts

tests/data/use-cases/db-add-guardian.spec.ts

tests/infra/criptography/bcrypt-adapter.spec.ts

tests/infra/repos/postgresql/guardian-account.spec.ts

tests/main/middlewares/body-parser.test.ts

tests/main/middlewares/content-type.test.ts

tests/main/middlewares/cors.test.ts

tests/main/routes/signup-routes.test.ts

tests/validation/email-validator-adapter.spec.ts

tests/validation/name-validator-adapter.spec.ts

tests/validation/password-validator-adapter.spec.ts

tests/validation/phone-validator-adapter.spec.ts

matheusgondra

Review the commented out code

maztt

Code clarity requested changes

tests/validation/phone-validator-adapter.spec.ts

tests/validation/password-validator-adapter.spec.ts

tests/validation/name-validator-adapter.spec.ts

tests/validation/email-validator-adapter.spec.ts

tests/main/routes/signup-routes.test.ts

tests/infra/repos/postgresql/guardian-account.spec.ts

tests/helpers/prisma-helper.ts

src/application/controllers/signup.ts

src/infra/repos/postgresql/guardian-account-repository.ts

Dan added 30 commits March 26, 2023 14:30

chore: set off @typescript-eslint/no-namespace rule

3ffafe4

chore: remove ts-jest and setup babel

ac83ee9

feat: ensure SignUpController returns 400 if no name is provided

643c21b

test: ensure SignUpController returns 400 if no name is provided

c32a343

feat: ensure SignUpController returns an error if no name is provided

e745ea8

test: ensure SignUpController returns an error if no name is provided

c433bd6

test: ensure SignUpController returns an error if no name is provided

f3f3792

Merge branch 'feature/user-register' of github.com:PetJournal/petjour…

fa7fde1

…nal.api into feature/user-register

Merge branch 'feature/user-register' of github.com:PetJournal/petjour…

eff0071

…nal.api into feature/user-register

Merge branch 'feature/user-register' of github.com:PetJournal/petjour…

25bb142

…nal.api into feature/user-register

chore: set off @typescript-eslint/strict-boolean-expressions

01b0176

feat: ensure SignUpController returns 400 if no lastName is provided

5efe0ea

test: ensure SignUpController returns 400 if no lastName is provided

11ede7d

feat: ensure SignUpController returns 400 if no email is provided

9f5fe63

test: ensure SignUpController returns 400 if no email is provided

df58195

refactor: ensure SignUpController.handle implement correct interfaces

1ca1105

refactor: add customized error for missing param

62c72bd

refactor: move duplicated code to a helper method

aa762c4

refactor: simplify code

2c31eba

feat: ensure SignUpController returns 400 if no password is provided

ef0b614

test: ensure SignUpController returns 400 if no password is provided

a98162d

feat: ensure SignUpController returns 400 if no password confirmation…

90fd226

… is provided

test: ensure SignUpController returns 400 if no password confirmation…

1ff6afd

… is provided

feat: ensure SignUpController returns 400 if isProvicyPolicyAccepted …

6af8e15

…is not provided

test: ensure SignUpController returns 400 if isProvicyPolicyAccepted …

2943747

…is not provided

refactor: ensure SignUpController implements correct interface

c5a3672

refactor: move sut creation to a factory helper method

96c4098

feat: ensure SignUpController returns 400 if an invalid email is prov…

a70d564

…ided

test: ensure SignUpController returns 400 if an invalid email is prov…

077f2ad

…ided

test: ensure SignUpController calls EmailValidator with correct email

4376246

matheusgondra reviewed Apr 17, 2023

View reviewed changes

src/data/use-cases/db-add-guardian.ts Outdated Show resolved Hide resolved

matheusgondra reviewed Apr 17, 2023

View reviewed changes

src/infra/criptography/bcrypt-adapter.ts Outdated Show resolved Hide resolved

matheusgondra reviewed Apr 17, 2023

View reviewed changes

src/main/adapters/express-route-adapter.ts Outdated Show resolved Hide resolved

matheusgondra reviewed Apr 17, 2023

View reviewed changes

src/main/adapters/express-route-adapter.ts Outdated Show resolved Hide resolved

matheusgondra reviewed Apr 17, 2023

View reviewed changes

src/main/factories/controllers/signup-controller-factory.ts Outdated Show resolved Hide resolved