Skip to content

Refatora testes de manter adm com asserts adensados #630 - #666

Open
VeniciusBispo wants to merge 2 commits into
mainfrom
Branch_Venicius_630
Open

Refatora testes de manter adm com asserts adensados #630#666
VeniciusBispo wants to merge 2 commits into
mainfrom
Branch_Venicius_630

Conversation

@VeniciusBispo

Copy link
Copy Markdown
Collaborator

O que foi feito

  • Controller (PessoaControllerTests): asserts adensados em DefinirAdministrador (GET/POST válido, e-mail inválido, e-mail em uso, CPF já admin, ModelState inválido, falha em VerificaEdit e em CreatePessoaIdentityComPapelAsync), EnviarEmailSenha (sucesso com verificação de destinatário/assunto/token, usuário não encontrado, e-mail inválido e e-mail não confirmado) e DeleteConfirmed (redirecionamento p/ DefinirAdministrador com TempData validado).
  • Service (PessoaServiceTests): cobertura de GetAllAdmAsync (filtra não-admin), IsAdmAsync, CreatePessoaIdentityComPapelAsync (ADMINISTRADOR/GESTOR/USUARIO + papel inválido), ValidaEmail, EmailExist, VerificaEdit e Delete (proteção do último admin, remoção de papel, pessoa comum e id inexistente).
  • MockUserManager: rastreio real de papéis (Add/Remove/IsInRole/GetRoles).
  • Correção de verify incorreto: no POST válido o controller redireciona sem chamar GetAllAdmAsync.

@VeniciusBispo VeniciusBispo self-assigned this Sep 4, 2026
@BeatrizSouz
BeatrizSouz self-requested a review September 4, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants