Skip to content

Feature/product image#134

Merged
PauloGoncalvesBH merged 3 commits intoServeRest:trunkfrom
eliasreis54:feature/productImage
Oct 2, 2020
Merged

Feature/product image#134
PauloGoncalvesBH merged 3 commits intoServeRest:trunkfrom
eliasreis54:feature/productImage

Conversation

@eliasreis54
Copy link
Copy Markdown
Contributor

@eliasreis54 eliasreis54 commented Oct 1, 2020

Description

Add field to add product image on create product

I can't update the documentation. Is there some workflow to do that?

How can the user experience this change?

  • Create a new product with field imagem
  • Get all products and see your field saved in database

Documentation

Related Issues

Fixes #107

PR Tasks

  • Has been related to an issue?
  • Have tests been added/updated?
  • Has Aglio documentation been added/updated?

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 1, 2020 •
edited
Loading

Codecov Report

Merging #134 into trunk will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             trunk      #134   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        23    +1     
  Lines          389       382    -7     
=========================================
- Hits           389       382    -7
Impacted Files Coverage Δ
src/models/produtos-model.js 100.00% <ø> (ø)
src/services/produtos-service.js 100.00% <100.00%> (ø)
src/app.js 100.00% <0.00%> (ø)
src/controllers/login-controller.js 100.00% <0.00%> (ø)
src/controllers/produtos-controller.js 100.00% <0.00%> (ø)
src/controllers/usuarios-controller.js 100.00% <0.00%> (ø)
src/controllers/carrinhos-controller.js 100.00% <0.00%> (ø)
src/middlewares/error-handler.js 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46229c6...64f0204. Read the comment docs.

PauloGoncalvesBH
PauloGoncalvesBH requested changes Oct 2, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@PauloGoncalvesBH PauloGoncalvesBH left a comment

Choose a reason for hiding this comment

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

@eliasreis54 gostei bastante da alteração feita.

Sobre os testes, tem 2 cenários que acho interessante incluir:

  1. Cadastro com imagem do produto com sucesso
    • Esse cenário é devido de que 'imagem' é opcional, então precisamos garantir cenários passando imagem e sem passar.
  2. Bad request - imagem deve ser string
    • Nesse cenário é para garantir que a mensagem de erro que o campo deve ser string será retornada.

Esses 2 ajustes podem ser feitos no arquivo /test/produtos/post.test.js, e o cenário 2 pode ser inspirado nos outros cenários de bad request.

@eliasreis54
Copy link
Copy Markdown
Contributor Author

eliasreis54 commented Oct 2, 2020

@PauloGoncalvesBH obrigado pelo feedback, acredito que o caso de teste número 1 já está sendo contemplado.
Criei um teste chamado Cadastro sem imagem do produto com sucesso onde é feito um cadastro do produto com imagem e o cadastro default é preenchido todos os campos, visto que, alterei a função dadosProduto para retornar um objeto contendo uma key chamada imagem
Minha explicação sobre o teste 1 ficou clara ou realmente esses testes não estão coerentes?

O teste número 2 irei incluir.

@PauloGoncalvesBH
Copy link
Copy Markdown
Member

PauloGoncalvesBH commented Oct 2, 2020

@eliasreis54 realmente, concordo totalmente com o que disse e não tinha me atentado a isso.

@eliasreis54
Copy link
Copy Markdown
Contributor Author

eliasreis54 commented Oct 2, 2020

@PauloGoncalvesBH fiz o ajuste do teste que estava faltando, realmente fica bem mais completo com esse teste =)

PauloGoncalvesBH
PauloGoncalvesBH approved these changes Oct 2, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@PauloGoncalvesBH PauloGoncalvesBH left a comment

Choose a reason for hiding this comment

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

Ótima contribuição 🎉

@PauloGoncalvesBH PauloGoncalvesBH merged commit 6669cf2 into ServeRest:trunk Oct 2, 2020
@PauloGoncalvesBH
Copy link
Copy Markdown
Member

PauloGoncalvesBH commented Oct 2, 2020

🎉 This PR is included in version 2.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@PauloGoncalvesBH PauloGoncalvesBH mentioned this pull request Oct 12, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PauloGoncalvesBH PauloGoncalvesBH PauloGoncalvesBH approved these changes

Assignees

No one assigned

Labels

Hacktoberfest released on @latest released on v2.11.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Adicionar imagens dos produtos

3 participants

@eliasreis54 @codecov-commenter @PauloGoncalvesBH