Skip to content

[Fix] Corrige escopo de cobertura do JaCoCo e elimina falsos zeros - #908

Open
roberto-bento wants to merge 1 commit into
devfrom
894-corrigir-o-escopo-do-jacoco
Open

[Fix] Corrige escopo de cobertura do JaCoCo e elimina falsos zeros#908
roberto-bento wants to merge 1 commit into
devfrom
894-corrigir-o-escopo-do-jacoco

Conversation

@roberto-bento

Copy link
Copy Markdown
Contributor

O que mudou?

Este PR resolve o problema de divergência na medição de cobertura de testes pelo JaCoCo. Anteriormente, as execuções de prepare-agent e report possuíam regras conflitantes, o que gerava relatórios com "falsos zeros" (classes com testes passando que apareciam zeradas) e limitava a visão do relatório aos pacotes antigos, ignorando a nova estrutura de pastas da migração DDD. A configuração foi unificada para varrer todo o sistema.

Tarefas Relacionadas

Mudanças Realizadas

  • Centralização das regras de escopo (<includes> e <excludes>) no bloco global <configuration> do jacoco-maven-plugin, garantindo que ambas as etapas do plugin operem sob as mesmas condições.
  • Substituição do filtro restrito de controllers por br/org/apae/api/**, fazendo com que o relatório meça toda a base de código e detecte automaticamente pacotes migrados (como o PatientRecordController).
  • Inclusão de <excludes> para arquivos sem lógica testável (classe ApiApplication, arquivos *DTO.* e MinioConfiguration.class), evitando impacto injusto no baseline.
  • Nenhuma trava de build (como jacoco:check) foi adicionada nesta etapa.

Evidências

Em anexo, as imagens comprovando a alteração nos relatórios locais:

  • Antes: image O relatório media uma parte muito restrita da aplicação e gerava os falsos zeros.
  • Depois: image O relatório agora exibe a árvore completa do projeto. As 6 classes apontadas na issue (como AbsenceController, AuthController e DashboardController) saíram do falso 0% e agora refletem a cobertura real de 100%.

Nota sobre a validação: O escopo agora mede um total de 12.167 Instruções de Bytecode (Instructions) e 3.229 Linhas (Lines). O valor de "~12.000" apontado na issue original refere-se à contagem de instruções. O novo baseline realista de cobertura foi estabilizado em 19% das instruções, devidamente enxuto após a exclusão dos DTOs e classes de configuração.

@roberto-bento roberto-bento self-assigned this Aug 28, 2026
@roberto-bento roberto-bento added QA Lista de tarefas para análise de qualidade 3 Complexidade 3 build labels Aug 28, 2026
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0df1c04a-7980-4d23-919b-764d3a686b87

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 Complexidade 3 build QA Lista de tarefas para análise de qualidade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant