serverest 2.26.3 → 2.26.5

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## [2.26.5](https://github.com/ServeRest/ServeRest/compare/v2.26.4...v2.26.5) (2022-10-18)
+
+
+### Code Refactoring
+
+* add concluiCompra method ([0fc3d4c](https://github.com/ServeRest/ServeRest/commit/0fc3d4c217933f1aa73669fc5274c9feaae8b28b))
+* add removeCarrinho method ([8fbbb1b](https://github.com/ServeRest/ServeRest/commit/8fbbb1b28e0ca783e11bce1aff8605ff4a28dc2c))
+* check for carrinhoDoUsuario length and use removeCarrinho method from carrinhosService ([e560be5](https://github.com/ServeRest/ServeRest/commit/e560be539a1a0dc08732f5f863ee912a6afb417d))
+* move all produtosService calls to carrinhosService ([a756731](https://github.com/ServeRest/ServeRest/commit/a756731eb019a17eafeb88880740f345464304b1))
+* move produtosService usage inside carrinhosService ([2837023](https://github.com/ServeRest/ServeRest/commit/28370239995097c4b8cd53fa55e36e17f39def93))
+* remove validation of undefined for returned array ([74a71e2](https://github.com/ServeRest/ServeRest/commit/74a71e20be717e65d5d5b1d1ef6a50466836afa7))
+* use concluiCompra method from carrinhosService ([f5d3bb5](https://github.com/ServeRest/ServeRest/commit/f5d3bb55730a1ab17f813edbff726472e0c33834))
+
+
+### Continuous Integration
+
+* fix sonar and contract jobs to clone PR from Fork ([ed48fe9](https://github.com/ServeRest/ServeRest/commit/ed48fe9d6a2ede2f1c97bf4cc6833e8ec8052480))
+
+
+### Miscellaneous
+
+* Merge pull request #378 from edumaxsantos/refactoring/carrinhos-controller ([325cdf7](https://github.com/ServeRest/ServeRest/commit/325cdf79d16647b6989313439648a269fc389234)), closes [#378](https://github.com/ServeRest/ServeRest/issues/378)
+* Revert "chore(release): use default changelog content" ([d7ec542](https://github.com/ServeRest/ServeRest/commit/d7ec542c3e1b87ce03b80b1f973f8748189685d6))
+
+## [2.26.4](https://github.com/ServeRest/ServeRest/compare/v2.26.3...v2.26.4) (2022-10-14)
+
 ## [2.26.3](https://github.com/ServeRest/ServeRest/compare/v2.26.2...v2.26.3) (2022-10-12)
 
 ## [2.26.2](https://github.com/ServeRest/ServeRest/compare/v2.26.1...v2.26.2) (2022-08-18)

package/docs/swagger.json

@@ -998,7 +998,7 @@
           "type": "number",
           "example": 2
         },
-        "usuarios": {
+        "produtos": {
           "type": "array",
           "items": {
             "$ref": "#/definitions/arrayProduto"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "serverest",
-  "version": "2.26.3",
+  "version": "2.26.5",
   "description": "Servidor REST local de forma rápida e simples para estudo de testes de API",
   "author": "Paulo Gonçalves <paulorochag@hotmail.com> (https://www.linkedin.com/in/paulo-goncalves/)",
   "license": "GPL-3.0",

package/src/controllers/carrinhos-controller.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const constant = require('../utils/constants')
-const produtosService = require('../services/produtos-service')
 const service = require('../services/carrinhos-service')
 
 exports.get = async (req, res) => {
@@ -24,22 +23,16 @@ exports.post = async (req, res) => {
     return res.status(400).send({ message: constant.LIMIT_JUST_ONE_CART })
   }
 
-  const idProdutosDuplicados = service.extrairProdutosDuplicados(req.body.produtos)
-  const temProdutosDuplicados = isNotUndefined(idProdutosDuplicados[0])
-  if (temProdutosDuplicados) {
+  const { produtos } = req.body
+
+  const idProdutosDuplicados = service.extrairProdutosDuplicados(produtos)
+  if (idProdutosDuplicados.length) {
     return res.status(400).send({ message: constant.CART_WITH_DUPLICATE_PRODUCT, idProdutosDuplicados })
   }
 
-  const { produtos } = req.body
-  const produtosComPrecoUnitario = []
-  for (const produto of produtos) {
-    const { precoUnitario: preco, error } = await produtosService.getPrecoUnitarioOuErro(produto)
-    if (error) {
-      const index = produtos.indexOf(produto)
-      const item = { ...error.item, index }
-      return res.status(error.statusCode).send({ message: error.message, item })
-    }
-    produtosComPrecoUnitario.push({ ...produto, precoUnitario: preco })
+  const { produtosComPrecoUnitario, error } = await service.getProdutosComPrecoUnitarioOuErro(produtos)
+  if (error) {
+    return res.status(error.statusCode).send({ message: error.message, item: error.item })
   }
   const precoTotal = await service.precoTotal(produtosComPrecoUnitario)
   const quantidadeTotal = await service.quantidadeTotal(produtosComPrecoUnitario)
@@ -52,18 +45,9 @@ exports.post = async (req, res) => {
 
 exports.cancelarCompra = async (req, res) => {
   const carrinhoDoUsuario = await service.getCarrinhoDoUsuario(req.headers.authorization)
-  const usuarioTemCarrinho = isNotUndefined(carrinhoDoUsuario[0])
-
-  if (usuarioTemCarrinho) {
-    const produtos = carrinhoDoUsuario[0].produtos
 
-    produtos.forEach(async (produto) => {
-      const { idProduto, quantidade } = produto
-      const { quantidade: quantidadeEmEstoque } = await produtosService.getDadosDoProduto({ _id: idProduto })
-      await produtosService.updateById(idProduto, { $set: { quantidade: quantidadeEmEstoque + quantidade } })
-    })
-
-    await service.deleteById(carrinhoDoUsuario[0]._id)
+  if (carrinhoDoUsuario.length) {
+    await service.removeCarrinho(carrinhoDoUsuario[0])
     return res.status(200).send({ message: `${constant.DELETE_SUCCESS}. ${constant.REPLENISHED_STOCK}` })
   }
 
@@ -72,14 +56,10 @@ exports.cancelarCompra = async (req, res) => {
 
 exports.concluirCompra = async (req, res) => {
   const carrinhoDoUsuario = await service.getCarrinhoDoUsuario(req.headers.authorization)
-  const usuarioTemCarrinho = isNotUndefined(carrinhoDoUsuario[0])
-  if (usuarioTemCarrinho) {
-    await service.deleteById(carrinhoDoUsuario[0]._id)
+
+  if (carrinhoDoUsuario.length) {
+    await service.concluiCompra(carrinhoDoUsuario)
     return res.status(200).send({ message: constant.DELETE_SUCCESS })
   }
   res.status(200).send({ message: constant.NO_CART })
 }
-
-const isUndefined = (object) => typeof object === 'undefined'
-
-const isNotUndefined = (object) => !isUndefined(object)

package/src/services/carrinhos-service.js

@@ -38,10 +38,34 @@ exports.extrairProdutosDuplicados = arrayProdutos => {
   return produtosDuplicados
 }
 
+exports.getProdutosComPrecoUnitarioOuErro = async arrayProdutos => {
+  const produtosComPrecoUnitario = []
+  for (const produto of arrayProdutos) {
+    const { precoUnitario: preco, error } = await produtosService.getPrecoUnitarioOuErro(produto)
+    if (error) {
+      const index = arrayProdutos.indexOf(produto)
+      const item = { ...error.item, index }
+      return { error: { ...error, item } }
+    }
+    produtosComPrecoUnitario.push({ ...produto, precoUnitario: preco })
+  }
+  return { produtosComPrecoUnitario }
+}
+
 exports.deleteById = async id => {
   return datastore.remove({ _id: id }, {})
 }
 
+exports.concluiCompra = async carrinho => {
+  await this.deleteById(carrinho._id)
+}
+
+exports.removeCarrinho = async carrinho => {
+  const { produtos, _id } = carrinho
+  this.reabasteceEstoque(produtos)
+  await this.deleteById(_id)
+}
+
 exports.getCarrinhoDoUsuario = async (authorization) => {
   const _id = await idUsuario(authorization)
   return this.getAll({ idUsuario: _id })
@@ -68,6 +92,14 @@ exports.quantidadeTotal = async (produtos) => {
   }, Promise.resolve(0))
 }
 
+exports.reabasteceEstoque = produtos => {
+  produtos.forEach(async (produto) => {
+    const { idProduto, quantidade } = produto
+    const { quantidade: quantidadeEmEstoque } = await produtosService.getDadosDoProduto({ _id: idProduto })
+    await produtosService.updateById(idProduto, { $set: { quantidade: quantidadeEmEstoque + quantidade } })
+  })
+}
+
 const idUsuario = async (authorization) => {
   const { email, password } = authService.verifyToken(authorization)
   const { _id } = await usuariosService.getDadosDoUsuario({ email, password })