serverest 2.26.0 → 2.26.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [2.26.1](https://github.com/ServeRest/ServeRest/compare/v2.26.0...v2.26.1) (2022-07-08)
+
+
+### Bug Fixes
+
+* show user that request ended with timeout ([1f432c4](https://github.com/ServeRest/ServeRest/commit/1f432c429647c871b2075dc8a4259ab52e714e04))
+
 # [2.26.0](https://github.com/ServeRest/ServeRest/compare/v2.25.4...v2.26.0) (2022-06-19)
 
 

package/README.md

@@ -178,7 +178,15 @@ Sua empresa usa o ServeRest? Pergunte ao seu gerente ou equipe de marketing se s
 1. Subdomínio próprio (_nome-escolhido.serverest.dev_)
 1. Acesso a todas as requests e respostas feitas nos últimos 7 dias no subdomínio
 
-[![Apoiador individual - Open Collective](https://opencollective.com/serverest/tiers/patrocinador.svg)](https://opencollective.com/serverest)
+[![Empresas - Open Collective](https://opencollective.com/serverest/tiers/patrocinador.svg)](https://opencollective.com/serverest)
+
+Empresas que apoiam o ServeRest:
+
+<p align="center">
+ <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">
+</p>
 
 ### Individuais
 
@@ -210,6 +218,8 @@ Veja aqui [como você pode contribuir](https://github.com/ServeRest/ServeRest/bl
     <td align="center"><a href="https://about.me/rustnnes"><img src="https://avatars1.githubusercontent.com/u/638445?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Diego Bandeira</b></sub></a><br /><a href="#infra-rustnnes" title="Infrastructure (Hosting, Build-Tools, etc)">🚇</a></td>
     <td align="center"><a href="https://github.com/maximilianoalves"><img src="https://avatars3.githubusercontent.com/u/11561118?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Maximiliano Alves</b></sub></a><br /><a href="#talk-maximilianoalves" title="Talks">📢</a></td>
     <td align="center"><a href="https://github.com/murilomaiaa"><img src="https://avatars.githubusercontent.com/u/56596799?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Murilo Maia</b></sub></a><br /><a href="https://github.com/ServeRest/ServeRest/commits?author=murilomaiaa" title="Code">💻</a></td>
+    <td align="center"><a href="https://github.com/crisnazario"><img src="https://avatars.githubusercontent.com/u/37200398?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Cristina Nazário</b></sub></a><br /><a href="#ideas-crisnazario" title="Ideas, Planning, & Feedback">🤔</a></td>
+    <td align="center"><a href="http://www.eduardosantos.dev"><img src="https://avatars.githubusercontent.com/u/10568807?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Eduardo Santos</b></sub></a><br /><a href="https://github.com/ServeRest/ServeRest/commits?author=edumaxsantos" title="Code">💻</a></td>
   </tr>
 </table>
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "serverest",
-  "version": "2.26.0",
+  "version": "2.26.1",
   "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",
@@ -34,7 +34,7 @@
     "dev": "nodemon --legacy-watch -e json,js ./src/server.js --nodoc",
     "test:mutation": "stryker run ./test/stryker.conf.js",
     "test:mutation:diff": "stryker-diff-runner --path ./test/stryker.conf.js --branch trunk",
-    "test:unit": "nyc --report-dir ./coverage-unit mocha --config test/unit/.mocharc.js",
+    "test:unit": "nyc --report-dir ./coverage-unit --check-coverage false mocha --config test/unit/.mocharc.js",
     "test:integration": "nyc --report-dir ./coverage-integration mocha --config test/integration/.mocharc.js",
     "test:e2e": "mocha --config test/integration/.mocharc.js --fgrep @skipE2E --invert",
     "test:smoke": "mocha --config test/integration/.mocharc.js --grep @smokeE2E",

package/src/controllers/carrinhos-controller.js

@@ -1,10 +1,8 @@
 'use strict'
 
-const authService = require('../services/auth-service')
 const constant = require('../utils/constants')
 const produtosService = require('../services/produtos-service')
 const service = require('../services/carrinhos-service')
-const usuariosService = require('../services/usuarios-service')
 
 exports.get = async (req, res) => {
   const carrinhos = await service.getAll(req.query)
@@ -21,53 +19,40 @@ exports.getOne = async (req, res) => {
 }
 
 exports.post = async (req, res) => {
-  const { email, password } = authService.verifyToken(req.headers.authorization)
-  const { _id } = await usuariosService.getDadosDoUsuario({ email, password })
-  const usuarioJaPossuiCarrinho = await service.existeCarrinho({ idUsuario: _id })
-  if (usuarioJaPossuiCarrinho) {
+  const { idUsuario, possuiCarrinho } = await service.usuarioJaPossuiCarrinho(req.headers.authorization)
+  if (possuiCarrinho) {
     return res.status(400).send({ message: constant.LIMITE_1_CARRINHO })
   }
 
   const idProdutosDuplicados = service.extrairProdutosDuplicados(req.body.produtos)
-  const temProdutosDuplicados = typeof idProdutosDuplicados[0] !== 'undefined'
+  const temProdutosDuplicados = isNotUndefined(idProdutosDuplicados[0])
   if (temProdutosDuplicados) {
     return res.status(400).send({ message: constant.CARRINHO_COM_PRODUTO_DUPLICADO, idProdutosDuplicados })
   }
 
-  const produtos = req.body.produtos
-  for (let index = 0; index < produtos.length; index++) {
-    produtos[index].quantidade = parseInt(produtos[index].quantidade)
-    const { idProduto, quantidade } = produtos[index]
-    if (!await produtosService.existeProduto({ _id: idProduto })) {
-      return res.status(400).send({ message: constant.IDPRODUTO_INVALIDO, item: { index, idProduto, quantidade } })
+  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 })
     }
-
-    const { quantidade: quantidadeEstoque, preco } = await produtosService.getDadosDoProduto({ _id: idProduto })
-    if (quantidade > quantidadeEstoque) {
-      return res.status(400).send({ message: constant.ESTOQUE_INSUFICIENTE, item: { index, idProduto, quantidade, quantidadeEstoque } })
-    }
-    Object.assign(produtos[index], { precoUnitario: preco })
-  }
-  let precoTotal = 0
-  let quantidadeTotal = 0
-  for (let index = 0; index < produtos.length; index++) {
-    const { idProduto, quantidade } = produtos[index]
-    const { quantidade: quantidadeEmEstoque, preco } = await produtosService.getDadosDoProduto({ _id: idProduto })
-    const novaQuantidade = quantidadeEmEstoque - quantidade
-    await produtosService.updateById(idProduto, { $set: { quantidade: novaQuantidade } })
-    precoTotal += preco * quantidade
-    quantidadeTotal += quantidade
+    produtosComPrecoUnitario.push({ ...produto, precoUnitario: preco })
   }
+  const precoTotal = await service.precoTotal(produtosComPrecoUnitario)
+  const quantidadeTotal = await service.quantidadeTotal(produtosComPrecoUnitario)
 
-  Object.assign(req.body, { precoTotal, quantidadeTotal, idUsuario: _id })
+  const carrinho = { produtos: produtosComPrecoUnitario, precoTotal, quantidadeTotal, idUsuario }
 
-  const dadosCadastrados = await service.criarCarrinho(req.body)
+  const dadosCadastrados = await service.criarCarrinho(carrinho)
   res.status(201).send({ message: constant.POST_SUCESS, _id: dadosCadastrados._id })
 }
 
 exports.cancelarCompra = async (req, res) => {
   const carrinhoDoUsuario = await service.getCarrinhoDoUsuario(req.headers.authorization)
-  const usuarioTemCarrinho = typeof carrinhoDoUsuario[0] !== 'undefined'
+  const usuarioTemCarrinho = isNotUndefined(carrinhoDoUsuario[0])
 
   if (usuarioTemCarrinho) {
     const produtos = carrinhoDoUsuario[0].produtos
@@ -87,10 +72,14 @@ exports.cancelarCompra = async (req, res) => {
 
 exports.concluirCompra = async (req, res) => {
   const carrinhoDoUsuario = await service.getCarrinhoDoUsuario(req.headers.authorization)
-  const usuarioTemCarrinho = typeof carrinhoDoUsuario[0] !== 'undefined'
+  const usuarioTemCarrinho = isNotUndefined(carrinhoDoUsuario[0])
   if (usuarioTemCarrinho) {
     await service.deleteById(carrinhoDoUsuario[0]._id)
     return res.status(200).send({ message: constant.DELETE_SUCESS })
   }
   res.status(200).send({ message: constant.SEM_CARRINHO })
 }
+
+const isUndefined = (object) => typeof object === 'undefined'
+
+const isNotUndefined = (object) => !isUndefined(object)

package/src/middlewares/error-handler.js

@@ -1,4 +1,4 @@
-const { INTERNAL_ERROR } = require('../utils/constants')
+const { INTERNAL_ERROR, TIMEOUT } = require('../utils/constants')
 const montarMensagemDeErroDeSchema = require('../utils/montarMensagemDeErroDeSchema')
 
 function errorHandler (error, _req, res, _next) {
@@ -10,9 +10,13 @@ function errorHandler (error, _req, res, _next) {
   if (error.type === 'entity.parse.failed') {
     console.log('Erro 500')
     return res.status(500).json({
-      message: 'Adicione aspas em todos os valores. Esse problema já está sendo investigado na issue https://github.com/ServeRest/ServeRest/issues/225'
+      message: 'Adicione aspas em todos os valores. Para mais informações acesse a issue https://github.com/ServeRest/ServeRest/issues/225'
     })
   }
+  /* istanbul ignore if */
+  if (error.code === 'ETIMEDOUT') {
+    return res.status(408).json({ message: TIMEOUT })
+  }
   return res.status(500).json({ message: INTERNAL_ERROR, error })
 }
 

package/src/services/carrinhos-service.js

@@ -6,6 +6,7 @@ const { join } = require('path')
 const alterarValoresParaRegex = require('../utils/alterarValoresParaRegex')
 const authService = require('../services/auth-service')
 const usuariosService = require('../services/usuarios-service')
+const produtosService = require('../services/produtos-service')
 
 const datastore = Datastore.create({ filename: join(__dirname, '../data/carrinhos.db'), autoload: true })
 
@@ -42,7 +43,33 @@ exports.deleteById = async id => {
 }
 
 exports.getCarrinhoDoUsuario = async (authorization) => {
+  const _id = await idUsuario(authorization)
+  return this.getAll({ idUsuario: _id })
+}
+
+exports.usuarioJaPossuiCarrinho = async (authorization) => {
+  const _id = await idUsuario(authorization)
+  return {
+    idUsuario: _id,
+    possuiCarrinho: await this.existeCarrinho({ idUsuario: _id })
+  }
+}
+
+exports.precoTotal = async (produtos) => {
+  return produtos.reduce(async (precoAnterior, produto) => {
+    return (await precoAnterior) + produto.precoUnitario * produto.quantidade
+  }, Promise.resolve(0))
+}
+
+exports.quantidadeTotal = async (produtos) => {
+  return produtos.reduce(async (quantidadeAnterior, produto) => {
+    await produtosService.updateQuantidade(produto)
+    return (await quantidadeAnterior) + produto.quantidade
+  }, Promise.resolve(0))
+}
+
+const idUsuario = async (authorization) => {
   const { email, password } = authService.verifyToken(authorization)
-  const { _id: idUsuario } = await usuariosService.getDadosDoUsuario({ email, password })
-  return this.getAll({ idUsuario })
+  const { _id } = await usuariosService.getDadosDoUsuario({ email, password })
+  return _id
 }

package/src/services/produtos-service.js

@@ -1,5 +1,6 @@
 'use strict'
 
+const constant = require('../utils/constants')
 const Datastore = require('nedb-promises')
 const { join } = require('path')
 
@@ -24,11 +25,32 @@ exports.existeProduto = pesquisa => {
   return datastore.count(pesquisa)
 }
 
+exports.updateQuantidade = async ({ idProduto, quantidade }) => {
+  const { quantidade: quantidadeEmEstoque } = await this.getDadosDoProduto({ _id: idProduto })
+  const novaQuantidade = quantidadeEmEstoque - quantidade
+  await this.updateById(idProduto, { $set: { quantidade: novaQuantidade } })
+}
+
 exports.criarProduto = async body => {
   body = formatarValores(body)
   return datastore.insert(body)
 }
 
+exports.getPrecoUnitarioOuErro = async (produto) => {
+  const quantidade = parseInt(produto.quantidade)
+  const { idProduto } = produto
+  const existePedido = await this.existeProduto({ _id: idProduto })
+  if (!existePedido) {
+    return { error: { statusCode: 400, message: constant.IDPRODUTO_INVALIDO, item: { idProduto, quantidade } } }
+  }
+
+  const { quantidade: quantidadeEstoque, preco } = await this.getDadosDoProduto({ _id: idProduto })
+  if (quantidade > quantidadeEstoque) {
+    return { error: { statusCode: 400, message: constant.ESTOQUE_INSUFICIENTE, item: { idProduto, quantidade, quantidadeEstoque } } }
+  }
+  return { precoUnitario: preco }
+}
+
 exports.deleteById = async id => {
   return datastore.remove({ _id: id }, {})
 }

package/src/utils/constants.js

@@ -2,6 +2,7 @@
 
 module.exports = {
   INTERNAL_ERROR: 'Abra uma issue informando essa resposta. https://github.com/ServeRest/ServeRest/issues',
+  TIMEOUT: 'Timeout da requisição atingido',
   POST_SUCESS: 'Cadastro realizado com sucesso',
   DELETE_SUCESS: 'Registro excluído com sucesso',
   DELETE_NONE: 'Nenhum registro excluído',