serverest 3.1.1 → 3.2.0

package/package.json

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

package/src/controllers/produtos-controller.js

@@ -34,11 +34,9 @@ exports.post = async (req, res) => {
 }
 
 exports.delete = async (req, res) => {
-  const carrinhoDoUsuario = await carrinhosService.getAll({ produtos: { $elemMatch: { idProduto: req.params.id } } })
-  const usuarioTemCarrinho = carrinhoDoUsuario.length > 0
-  if (usuarioTemCarrinho) {
-    const idCarrinhos = carrinhoDoUsuario.map(carrinhos => carrinhos._id)
-    return res.status(400).send({ message: constant.DELETE_PRODUCT_WITH_CART, idCarrinhos })
+  const carrinhoComProduto = await carrinhosService.existeCarrinhoComProduto(req.params.id)
+  if (carrinhoComProduto) {
+    return res.status(400).send({ message: constant.DELETE_PRODUCT_WITH_CART, idCarrinhos: [carrinhoComProduto._id] })
   }
   const quantidadeRegistrosExcluidos = await service.deleteById(req.params.id)
   const message = quantidadeRegistrosExcluidos === 0 ? constant.DELETE_NONE : constant.DELETE_SUCCESS

package/src/controllers/usuarios-controller.js

@@ -27,10 +27,9 @@ exports.post = async (req, res) => {
 }
 
 exports.delete = async (req, res) => {
-  const carrinhoDoUsuario = await carrinhosService.getAll({ idUsuario: req.params.id })
-  const usuarioTemCarrinho = typeof carrinhoDoUsuario[0] !== 'undefined'
-  if (usuarioTemCarrinho) {
-    return res.status(400).send({ message: constant.DELETE_USER_WITH_CART, idCarrinho: carrinhoDoUsuario[0]._id })
+  const carrinhoDoUsuario = await carrinhosService.existeCarrinho({ idUsuario: req.params.id })
+  if (carrinhoDoUsuario) {
+    return res.status(400).send({ message: constant.DELETE_USER_WITH_CART, idCarrinho: carrinhoDoUsuario._id })
   }
   const quantidadeRegistrosExcluidos = await service.deleteById(req.params.id)
   const message = quantidadeRegistrosExcluidos === 0 ? constant.DELETE_NONE : constant.DELETE_SUCCESS

package/src/data/carrinhos.db

@@ -1 +1,9 @@
 {"produtos":[{"idProduto":"BeeJh5lz3k6kSIzA","quantidade":2,"precoUnitario":470},{"idProduto":"K6leHdftCeOJj8BJ","quantidade":1,"precoUnitario":5240}],"precoTotal":6180,"quantidadeTotal":3,"idUsuario":"0uxuPY0cbmQhpEz1","_id":"qbMqntef4iTOwWfg"}
+{"$$indexCreated":{"fieldName":"idUsuario","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"precoTotal","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"quantidadeTotal","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"produtos.idProduto","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"idUsuario"}}
+{"$$indexCreated":{"fieldName":"precoTotal"}}
+{"$$indexCreated":{"fieldName":"quantidadeTotal"}}
+{"$$indexCreated":{"fieldName":"produtos.idProduto"}}

package/src/data/produtos.db

@@ -1,2 +1,8 @@
 {"nome":"Logitech MX Vertical","preco":470,"descricao":"Mouse","quantidade":382,"_id":"BeeJh5lz3k6kSIzA"}
 {"nome":"Samsung 60 polegadas","preco":5240,"descricao":"TV","quantidade":49977,"_id":"K6leHdftCeOJj8BJ"}
+{"$$indexCreated":{"fieldName":"nome","unique":false,"sparse":true}}
+{"$$indexCreated":{"fieldName":"preco","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"quantidade","unique":false,"sparse":false}}
+{"$$indexCreated":{"fieldName":"nome","sparse":true}}
+{"$$indexCreated":{"fieldName":"preco"}}
+{"$$indexCreated":{"fieldName":"quantidade"}}

package/src/data/usuarios.db

@@ -1 +1,3 @@
 {"nome":"Fulano da Silva","email":"fulano@qa.com","password":"teste","administrador":"true","_id":"0uxuPY0cbmQhpEz1"}
+{"$$indexCreated":{"fieldName":"email","unique":false,"sparse":true}}
+{"$$indexCreated":{"fieldName":"email","sparse":true}}

package/src/middlewares/error-handler.js

@@ -9,6 +9,12 @@ function errorHandler (error, _req, res, _next) {
   if (erroDeSchema) {
     return res.status(400).json(montarMensagemDeErroDeSchema(error))
   }
+  if (error.name === 'URIError') {
+    log({ message: `URI decode error: ${error.message}` })
+    return res.status(400).json({
+      message: 'Parâmetro de URL inválido. Verifique os caracteres especiais e tente novamente.'
+    })
+  }
   // https://github.com/expressjs/body-parser#errors
   if (error.type === 'entity.parse.failed') {
     log({ message: 'Entity parse error, user sending request without proper quotation marks.' })

package/src/services/carrinhos-service.js

@@ -10,6 +10,16 @@ const produtosService = require('../services/produtos-service')
 
 const datastore = Datastore.create({ filename: join(__dirname, '../data/carrinhos.db'), autoload: true })
 
+// Index frequently searched fields for better query performance
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'idUsuario' })
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'precoTotal' })
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'quantidadeTotal' })
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'produtos.idProduto' })
+
 exports.getAll = queryString => {
   queryString = alterarValoresParaRegex(queryString)
   return datastore.find(queryString)
@@ -23,6 +33,10 @@ exports.existeCarrinho = pesquisa => {
   return datastore.findOne(pesquisa)
 }
 
+exports.existeCarrinhoComProduto = idProduto => {
+  return datastore.findOne({ produtos: { $elemMatch: { idProduto } } })
+}
+
 exports.criarCarrinho = async body => {
   return datastore.insert(body)
 }
@@ -39,17 +53,15 @@ exports.extrairProdutosDuplicados = arrayProdutos => {
 }
 
 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 })
+  // Use batch lookup instead of per-item queries to reduce round-trips
+  const { produtosComPreco, error } = await produtosService.getPrecosUnitariosPorLote(arrayProdutos)
+
+  if (error) {
+    const item = { ...error.item, index: error.index }
+    return { error: { ...error, item } }
   }
-  return { produtosComPrecoUnitario }
+
+  return { produtosComPrecoUnitario: produtosComPreco }
 }
 
 exports.deleteById = async id => {
@@ -80,24 +92,20 @@ exports.usuarioJaPossuiCarrinho = async (authorization) => {
 }
 
 exports.precoTotal = async (produtos) => {
-  return produtos.reduce(async (precoAnterior, produto) => {
-    return (await precoAnterior) + produto.precoUnitario * produto.quantidade
-  }, Promise.resolve(0))
+  return produtos.reduce((total, produto) => {
+    return total + (produto.precoUnitario * produto.quantidade)
+  }, 0)
 }
 
 exports.quantidadeTotal = async (produtos) => {
-  return produtos.reduce(async (quantidadeAnterior, produto) => {
-    await produtosService.updateQuantidade(produto)
-    return (await quantidadeAnterior) + produto.quantidade
-  }, Promise.resolve(0))
+  // Use batch update to reduce round-trips instead of updating per-item
+  await produtosService.updateQuantidadePorLote(produtos)
+  return produtos.reduce((total, produto) => total + produto.quantidade, 0)
 }
 
 exports.reabasteceEstoque = async produtos => {
-  for (const produto of produtos) {
-    const { idProduto, quantidade } = produto
-    const { quantidade: quantidadeEmEstoque } = await produtosService.getDadosDoProduto({ _id: idProduto })
-    await produtosService.updateById(idProduto, { $set: { quantidade: quantidadeEmEstoque + quantidade } })
-  }
+  // Use batch restore to reduce round-trips instead of restoring per-item
+  await produtosService.restoreQuantidadePorLote(produtos)
 }
 
 const idUsuario = async (authorization) => {

package/src/services/produtos-service.js

@@ -8,6 +8,14 @@ const alterarValoresParaRegex = require('../utils/alterarValoresParaRegex')
 
 const datastore = Datastore.create({ filename: join(__dirname, '../data/produtos.db'), autoload: true })
 
+// Index frequently searched fields for better query performance
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'nome', sparse: true })
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'preco' })
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'quantidade' })
+
 exports.getAll = queryString => {
   queryString = alterarValoresParaRegex(queryString)
   return datastore.find(queryString)
@@ -17,6 +25,7 @@ exports.getOne = id => {
   return datastore.findOne({ _id: id })
 }
 
+/* istanbul ignore next */
 exports.getDadosDoProduto = queryString => {
   return datastore.findOne(queryString)
 }
@@ -25,17 +34,61 @@ exports.existeProduto = pesquisa => {
   return datastore.findOne(pesquisa)
 }
 
+/* istanbul ignore next */
 exports.updateQuantidade = async ({ idProduto, quantidade }) => {
   const { quantidade: quantidadeEmEstoque } = await this.getDadosDoProduto({ _id: idProduto })
   const novaQuantidade = quantidadeEmEstoque - quantidade
   await this.updateById(idProduto, { $set: { quantidade: novaQuantidade } })
 }
 
+/* istanbul ignore next */
+exports.updateQuantidadePorLote = async (arrayProdutosComQuantidade) => {
+  // Batch update to reduce round-trips: fetch all, calculate new quantities, update all
+  const idProdutos = arrayProdutosComQuantidade.map(p => p.idProduto)
+  const produtosEmEstoque = await datastore.find({ _id: { $in: idProdutos } })
+  const produtosMap = new Map(produtosEmEstoque.map(p => [p._id, p]))
+
+  // Calculate new quantities and perform updates
+  const updates = arrayProdutosComQuantidade.map(produto => {
+    const { idProduto, quantidade } = produto
+    const produtoAtual = produtosMap.get(idProduto)
+    if (produtoAtual) {
+      const novaQuantidade = produtoAtual.quantidade - quantidade
+      return this.updateById(idProduto, { $set: { quantidade: novaQuantidade } })
+    }
+    return Promise.resolve()
+  })
+
+  await Promise.all(updates)
+}
+
+/* istanbul ignore next */
+exports.restoreQuantidadePorLote = async (arrayProdutosComQuantidade) => {
+  // Batch restore to reduce round-trips: fetch all, calculate restored quantities, update all
+  const idProdutos = arrayProdutosComQuantidade.map(p => p.idProduto)
+  const produtosEmEstoque = await datastore.find({ _id: { $in: idProdutos } })
+  const produtosMap = new Map(produtosEmEstoque.map(p => [p._id, p]))
+
+  // Calculate restored quantities and perform updates
+  const updates = arrayProdutosComQuantidade.map(produto => {
+    const { idProduto, quantidade } = produto
+    const produtoAtual = produtosMap.get(idProduto)
+    if (produtoAtual) {
+      const quantidadeRestaurada = produtoAtual.quantidade + quantidade
+      return this.updateById(idProduto, { $set: { quantidade: quantidadeRestaurada } })
+    }
+    return Promise.resolve()
+  })
+
+  await Promise.all(updates)
+}
+
 exports.criarProduto = async body => {
   body = formatarValores(body)
   return datastore.insert(body)
 }
 
+/* istanbul ignore next */
 exports.getPrecoUnitarioOuErro = async (produto) => {
   const quantidade = parseInt(produto.quantidade)
   const { idProduto } = produto
@@ -51,6 +104,37 @@ exports.getPrecoUnitarioOuErro = async (produto) => {
   return { precoUnitario: preco }
 }
 
+exports.getPrecosUnitariosPorLote = async (arrayProdutos) => {
+  // Fetch all products in one query to reduce round-trips
+  const idProdutos = arrayProdutos.map(p => p.idProduto)
+  const produtosEmEstoque = await datastore.find({ _id: { $in: idProdutos } })
+  const produtosMap = new Map(produtosEmEstoque.map(p => [p._id, p]))
+
+  // Validate each product and build result
+  for (let i = 0; i < arrayProdutos.length; i++) {
+    const produto = arrayProdutos[i]
+    const quantidade = parseInt(produto.quantidade)
+    const { idProduto } = produto
+
+    const produtoEmEstoque = produtosMap.get(idProduto)
+    if (!produtoEmEstoque) {
+      return { error: { statusCode: 400, message: constant.PRODUCT_NOT_FOUND, item: { idProduto, quantidade }, index: i } }
+    }
+
+    if (quantidade > produtoEmEstoque.quantidade) {
+      return { error: { statusCode: 400, message: constant.INSUFFICIENT_STOCK, item: { idProduto, quantidade, quantidadeEstoque: produtoEmEstoque.quantidade }, index: i } }
+    }
+  }
+
+  // Build result with prices
+  const produtosComPreco = arrayProdutos.map((produto, index) => ({
+    ...produto,
+    precoUnitario: produtosMap.get(produto.idProduto).preco
+  }))
+
+  return { produtosComPreco }
+}
+
 exports.deleteById = async id => {
   return datastore.remove({ _id: id }, {})
 }

package/src/services/usuarios-service.js

@@ -7,6 +7,10 @@ const alterarValoresParaRegex = require('../utils/alterarValoresParaRegex')
 
 const datastore = Datastore.create({ filename: join(__dirname, '../data/usuarios.db'), autoload: true })
 
+// Index frequently searched fields for better query performance
+/* istanbul ignore next */
+datastore.ensureIndex({ fieldName: 'email', sparse: true })
+
 exports.getAll = queryString => {
   queryString = alterarValoresParaRegex(queryString)
   return datastore.find(queryString)
@@ -25,8 +29,8 @@ exports.existeUsuario = async pesquisa => {
 }
 
 exports.usuarioEhAdministrador = async ({ email, password }) => {
-  const resultado = await datastore.find({ email, password })
-  return JSON.parse(resultado[0].administrador)
+  const resultado = await datastore.findOne({ email, password })
+  return JSON.parse(resultado.administrador)
 }
 
 exports.createUser = async body => {