Feat: Implementação listar e buscar fases sucessionais Api.v2 - #512
Feat: Implementação listar e buscar fases sucessionais Api.v2#512JosueModesto wants to merge 5 commits into
Conversation
| buscarFaseSucessionalUseCase: BuscarFaseSucessionalUseCase | ||
| } | ||
|
|
||
| export class BuscarFaseSucessionalController implements RequestHandler { |
There was a problem hiding this comment.
O restante da v2 usa Lista (não Listar). Renomear BuscarFaseSucessionalController, BuscarFaseSucessionalUseCase e os arquivos correspondentes para Busca*.
| }) | ||
| }) | ||
|
|
||
| describe('GET /api/v2/fases-sucessionais/:faseSucessionalId', () => { |
There was a problem hiding this comment.
Este arquivo é de listagem (lista-fases-sucessionais), mas este describe cobre GET …/:faseSucessionalId. Mover para um arquivo próprio (por exemplo busca-fase-sucessional.test.ts), no mesmo estilo de lista-paises / lista-estados.
| ...createPaisRoutes(knex), | ||
| ...createEstadoRoutes(knex), | ||
| ...createFaseSucessionalRoutes(knex), | ||
| ...createFaseSucessionalRoutes(knex), |
There was a problem hiding this comment.
As rotas de fase sucessional estão espalhadas duas vezes. Remover uma das chamadas a createFaseSucessionalRoutes(knex).
| buscarFaseSucessionalUseCase: BuscarFaseSucessionalUseCase | ||
| } | ||
|
|
||
| export class BuscarFaseSucessionalController implements RequestHandler { |
There was a problem hiding this comment.
Este arquivo (e BuscarFaseSucessionalUseCase) ficou sem uso depois do rename. index.ts já aponta para Busca*. Remover os leftovers.
| expect(body.error.message).toMatch(/inválido|invalid/i) | ||
| }) | ||
|
|
||
| test('retorna 400 para id inválido', async () => { |
There was a problem hiding this comment.
Este teste repete o título retorna 400 para id inválido (já usado no caso abc). Distinguir no nome, por exemplo id negativo.
2 participants
Close #495
O que foi feito
Foi implementado o módulo de leitura e busca das fases sucessionais na API v2, preservando o comportamento legado da rota antiga.
Alterações principais
Cobertura de testes