User entitie#1
Conversation
lseixas
left a comment
lseixas
left a comment
There was a problem hiding this comment.
@PedroOSanches reverte as alterações que você fez nas rotas, não vamos usa-las do jeito que estão agora
Deixa alterações apenas nos arquivos: user.py, test_user.py, user_repo_mock.py, test_user_repo_mock.py, user_repository_interface.py
| class UserRepositoryMock(IUserRepository): | ||
| users: List[User] | ||
| user_counter: int |
There was a problem hiding this comment.
Pode remover esse user counter, inclusive a função dele no mock também
A interface IUserRepository não está aparecendo aqui no diff, provavelmente ta com a assinatura antiga dos métodos.
| def get_user(self, user_email: str) -> User: | ||
| for user in self.users: | ||
| if user.user_id == user_id: | ||
| if user.email == user_email: | ||
| return user | ||
| raise NoItemsFound("user_id") | ||
| raise NoItemsFound("user_email") |
There was a problem hiding this comment.
Troca o get_user para pedir um id e buscar no banco por um id ao invés de email
| def delete_user(self, user_email: str) -> User: | ||
| for idx, user in enumerate(self.users): | ||
| if user.user_id == user_id: | ||
| if user.email == user_email: | ||
| self.user_counter -= 1 | ||
| return self.users.pop(idx) |
There was a problem hiding this comment.
mesma coisa aqui, coloca para puxar por id
| def update_user(self, user_email: str, new_password_hash: str) -> User: | ||
| for user in self.users: | ||
| if user.user_id == user_id: | ||
| user.name = new_name | ||
| if user.email == user_email: | ||
| user.password_hash = new_password_hash | ||
| return user |
There was a problem hiding this comment.
aqui também. Outra coisa, adiciona aqui a opção do método alterar a Role do usuário também
There was a problem hiding this comment.
dentro desse arquivo, troca os raise NoItemsFound para um return None.
There was a problem hiding this comment.
Reverte esse requirements para o que estava antes
2 participants
No description provided.