Skip to content

test(user): add complete unit test suite and CI validation workflow#8

Merged
lucas-adm merged 9 commits intoNoteHubBR:mainfrom
lucas-adm:tests/user
Nov 19, 2025
Merged

test(user): add complete unit test suite and CI validation workflow#8
lucas-adm merged 9 commits intoNoteHubBR:mainfrom
lucas-adm:tests/user

Conversation

@lucas-adm
Copy link
Copy Markdown
Member

@lucas-adm lucas-adm commented Nov 19, 2025

Sumário

Este PR adiciona a suíte completa de testes unitários para o serviço de usuário e inclui um workflow de CI responsável por executar automaticamente mvn test em pull requests direcionados à branch main.
O objetivo é ampliar a cobertura de testes, garantir maior segurança durante desenvolvimento e preparar o repositório para futuras regras de proteção de branch que exijam a aprovação dos testes antes do merge.

Alterações

  • Adicionados testes unitários para o serviço de usuário, incluindo:
    • criação de usuários;
    • edição de usuários;
    • deleção de usuários;
    • recuperação (single e list);
    • relacionamentos e vinculações;
  • Refatoração interna dos testes para remover duplicação e simplificar chamadas de verificação (verify);
  • Adicionado workflow .github/workflows/tests.yml para:
    • executar mvn test automaticamente em PRs abertos contra main;
    • utilizar actions/setup-java com JDK 21 e Maven embarcado;
    • padronizar o processo de validação antes de merges.

Necessidade

As novas validações garantem que o serviço de usuário se comporte corretamente sob diferentes cenários e estabelecem uma base sólida de confiança no código.
A introdução do workflow de testes aproxima o projeto das boas práticas de CI/CD, permitindo evoluir futuramente para uma política formal de merges dependentes da aprovação dos testes.

Teste manual

  • Abrir um PR direcionado à branch main e verificar:
    • Execução automática do workflow de testes;
    • Status final como success quando os testes forem aprovados;
    • Testes sendo executados com JDK 21 e Maven integrado pela action.

Checklist

  • Código segue o padrão do projeto
  • Documentação atualizada
  • Testes adicionados/atualizados

Breaking Changes

  • Nenhuma

@lucas-adm lucas-adm merged commit 11cc5fc into NoteHubBR:main Nov 19, 2025
@lucas-adm lucas-adm deleted the tests/user branch November 19, 2025 18:33
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.

1 participant