Skip to content

feat: implementa leitura de eventos - #541

Merged
EduTiyo merged 4 commits into
532-cadastro-expedicoesfrom
feature/eventos-leitura
Sep 24, 2026
Merged

EduTiyo merged 4 commits into
532-cadastro-expedicoesfrom
feature/eventos-leitura

Conversation

@MariaGabriela-JR

Copy link
Copy Markdown

O que foi feito

Implementados os endpoints de leitura de Eventos da API v2:

  • GET /api/v2/expedicoes/:expedicaoId/eventos

    • Lista os eventos de uma expedição.
  • GET /api/v2/eventos/:eventoId

    • Busca um evento específico pelo ID.

Implementação

  • Criados os use cases para listagem e busca de eventos.
  • Criados os controllers correspondentes.
  • Registradas as novas rotas em create-app.ts.
  • Reutilizado o EventoCollectionKnexAdapter existente.
  • Adicionada validação dos parâmetros expedicaoId e eventoId.
  • Tratados os casos de evento inexistente e erros internos.
  • Autenticação não foi adicionada neste momento, conforme alinhado pela equipe. Foi deixado um TODO para inclusão quando a camada HTTP de autenticação estiver disponível.

Testes

  • Testes unitários dos novos controllers.
  • yarn run test:unit: 79 testes passando.
  • yarn run tsc:check: concluído com sucesso.
  • yarn run lint:eslint: concluído com sucesso.
  • Testes manuais dos endpoints realizados com a API em execução.

@EduTiyo EduTiyo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisão em duas frentes (Standards e Spec) da leitura de Eventos. A busca por id (BuscarEventoController) está correta e, inclusive, evita o bug de mapeamento de erro que peguei no PR do Jordan (#540) — separa falha de infra (500) de "não encontrado" (404), parabéns por isso. Mas a listagem entrega só o roteamento: os três pedidos centrais do spec — paginação (?limite/?pagina), os filtros já existentes em EventoFilters (tipo, capturado_de, capturado_ate) e a ordenação estável (capturado_em desc, id desc, chamada de "a parte difícil" no combinado do time) — não chegam a ser usados. Falta também cobertura de integração para as duas rotas novas.

Comment thread src/application/evento/ListaEventosController.ts Outdated
Comment thread src/application/evento/ListaEventosController.ts
Comment thread src/application/evento/BuscarEventoController.ts
Comment thread src/domain/evento/ListaEventosUseCase.ts
@EduTiyo
EduTiyo merged commit 7fc16b8 into 532-cadastro-expedicoes Sep 24, 2026
5 checks passed
@EduTiyo
EduTiyo deleted the feature/eventos-leitura branch September 24, 2026 22:29
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