serverest 2.26.4 → 2.26.6

package/CHANGELOG.md

@@ -1,5 +1,42 @@
 # Changelog
 
+## [2.26.6](https://github.com/ServeRest/ServeRest/compare/v2.26.5...v2.26.6) (2022-10-28)
+
+
+### Chores
+
+* **patch:** add new sponsor Cesar School ([fc0df2f](https://github.com/ServeRest/ServeRest/commit/fc0df2feacb529d618609a43b2cf941c85a97fdb))
+
+
+### Continuous Integration
+
+* avoid concurrency while creating release and deploying ([4da150c](https://github.com/ServeRest/ServeRest/commit/4da150ca545a2cbe0032504e391f704141df38ac))
+* bump actions version to avoid deprecated node version ([9dbd693](https://github.com/ServeRest/ServeRest/commit/9dbd693bd353f163bd96e548183cfa20ee26434b))
+
+## [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)

package/README.md

@@ -188,6 +188,7 @@ Empresas que apoiam o ServeRest:
  <img alt="Logo da EBAC" src="https://user-images.githubusercontent.com/29241659/177436481-2a6a3324-1b0e-4d28-8a40-d885f54291c0.png#gh-light-mode-only" height="120">
  <img alt="Logo da EBAC" src="https://user-images.githubusercontent.com/29241659/177436489-5d2f50f8-2fb3-4091-b822-446d24c83722.png#gh-dark-mode-only" height="120">
  <img alt="Logo da Agilizei" src="https://user-images.githubusercontent.com/29241659/177436678-8187f90f-bb4a-4978-87ab-a03f2f80820f.png" height="124">
+ <img alt="Logo da Cesar School" src="https://user-images.githubusercontent.com/29241659/198725645-3e42d795-4393-4d9e-abf6-1655ecb83cfa.png" height="150">
 </p>
 
 ### Individuais

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "serverest",
-  "version": "2.26.4",
+  "version": "2.26.6",
   "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/app.js

@@ -68,6 +68,9 @@ switch (formaDeExecucao()) {
   case 'compassuol':
     swaggerDocument.host = 'compassuol.serverest.dev'
     break
+  case 'cesarschool':
+    swaggerDocument.host = 'cesarschool.serverest.dev'
+    break
 }
 
 swaggerDocument.info.version = packageJson.version

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 })

package/src/utils/ambiente.js

@@ -7,6 +7,7 @@ function formaDeExecucao () {
       process.env.USERNAME === 'serverest.dev' ||
       process.env.USERNAME === 'staging.serverest.dev' ||
       process.env.USERNAME === 'agilizei' ||
+      process.env.USERNAME === 'cesarschool' ||
       process.env.USERNAME === 'compassuol') {
     return process.env.USERNAME
   }
@@ -27,6 +28,8 @@ function urlDocumentacao () {
       return 'https://agilizei.serverest.dev'
     case 'compassuol':
       return 'https://compassuol.serverest.dev'
+    case 'cesarschool':
+      return 'https://cesarschool.serverest.dev'
     default:
       return `http://localhost:${conf.porta}`
   }