search

SharebookBR / sharebook-backend

Projeto backend de código livre para o app Sharebook.
117 stars 75 forks source link

Avaliar resultado antes de retornar ação create/update #483

Closed fernandesdiego closed 2 years ago

fernandesdiego commented 2 years ago

Ao tentar cadastrar um livro, não há verificação se a operação ocorreu como deveria, sempre retornando 200 e mensagem de sucesso. Este pull request adiciona a verificação da propriedade Result.IsSuccess gerado pelo serviço, para que retorne 400 com a mensagem de erro quando necessário.

Exemplo de um POST de um livro inválido que retorna 200 mas não é cadastrado pois falha na validação de formato de imagem:

{ "title": "Um livro aleatório", "author": "Um autor desconhecido", "categoryId": "B1A34609-A85C-4B36-893F-BA56AE911C2D", "imageName": "string", "imageBytes": "00000000", "freightOption": "0", "synopsis": "string", "type": "1", "eBookDownloadLink": "", "eBookPdfFile": "" }

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

AppVeyorBot commented 2 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.83-quqfevbo completed (commit https://github.com/SharebookBR/sharebook-backend/commit/fa326b5aeb by @)

raffacabofrio commented 2 years ago

Como a gente deixou passar isso? 😱

Obrigado pela contribuição Man. Tmj 👊