Crud expedicoes - #540
Crud expedicoes#540JordanBonfim wants to merge 12 commits into
Conversation
EduTiyo
left a comment
EduTiyo
left a comment
There was a problem hiding this comment.
Revisão em duas frentes (Standards e Spec) das mudanças de CRUD de Expedição. Dois achados de gravidade alta: mapeamento de erro invertido (falha de banco virando 404 ou 400 em vez de 500) em quatro controllers. O resto são gaps de cobertura de teste e judgement calls de estilo — nenhum scope creep contra o combinado.
|
|
||
| const result = await this.deletaExpedicaoUseCase.execute(Number(expedicaoId)) | ||
|
|
||
| if (result.left()) { |
There was a problem hiding this comment.
[Standards] Mesmo problema do Busca: result.left() vira 404 incondicionalmente, escondendo falhas reais de banco atrás de um "não encontrado".
| rotas: rotas ?? [] | ||
| }) | ||
|
|
||
| if (result.left()) { |
There was a problem hiding this comment.
[Standards] Mapeamento de erro incorreto (sentido oposto ao de Busca/Deleta): aqui qualquer left vira 400, inclusive falha de infraestrutura do create no adapter. Um erro de banco em POST /expedicoes hoje reporta como pedido inválido do cliente.
| updated_by | ||
| }) | ||
|
|
||
| if (result.left()) { |
There was a problem hiding this comment.
[Standards] Mesmo problema do Cadastra: left do update vira 400 mesmo quando a causa é CollectionError (infra), não validação.
2 participants
CRUD das expedições: Implementação dos 5 endpoints da API v2 (POST, GET lista paginada, GET por ID, PUT e DELETE).