Merge pull request #895 from codigoencasa/dev

Dev

.github/PULL_REQUEST_TEMPLATE.md

@@ -12,6 +12,6 @@ Por favor agrega una descripción de tu aporte para tener más contexto y poder
 > Forma parte de este proyecto.
 
 -   [Discord](https://link.codigoencasa.com/DISCORD)
--   [Twitter](https://twitter.com/leifermendez)
+-   [𝕏 (Twitter)](https://twitter.com/leifermendez)
 -   [Youtube](https://www.youtube.com/watch?v=5lEMCeWEJ8o&list=PL_WGMLcL4jzWPhdhcUyhbFU6bC0oJd2BR)
 -   [Telegram](https://t.me/leifermendez)

.github/workflows/releases-dev.yml

@@ -57,3 +57,6 @@ jobs:
 
             - name: Release @bot-whatsapp/portal
               run: node ./scripts/release.js --name=portal --version=  --token="${{ secrets.NPM_TOKEN }}"
+
+            - name: Release eslint-plugin-bot-whatsapp
+              run: node ./scripts/release.js --name=eslint-plugin-bot-whatsapp --version=  --token="${{ secrets.NPM_TOKEN }}"

.github/workflows/releases.yml

@@ -66,5 +66,8 @@ jobs:
             - name: Release @bot-whatsapp/portal
               run: node ./scripts/release.js --name=portal --version="${{ steps.package-version.outputs.current-version}}" --token="${{ secrets.NPM_TOKEN }}"
 
+            - name: Release eslint-plugin-bot-whatsapp
+              run: node ./scripts/release.js --name=eslint-plugin-bot-whatsapp --version="${{ steps.package-version.outputs.current-version}}" --token="${{ secrets.NPM_TOKEN }}"
+
             - name: Release Github
               run: node ./scripts/github.js --version="${{ steps.package-version.outputs.current-version}}" --token="${{ secrets.OCTO_TOKEN }}"

CONTRIBUTING.md

@@ -21,35 +21,34 @@ El lenguaje principal que se utilizó para desarrollar este proyecto fue __JavaS
 
 
 __Requerimientos:__
-- Node v16 o superior __[descargar node](https://nodejs.org/es/download/)__
-- __[Yarn](https://classic.yarnpkg.com/lang/en/docs/install/#windows-stable)__ como gestor de paquetes. En el link conseguirás las intrucciones para instalar yarn.
+- Node v18 o superior __[descargar node](https://nodejs.org/es/download/)__
+- __[pnpm](https://pnpm.io/cli/install)__ como gestor de paquetes. En el link conseguirás las intrucciones para instalar `pnmp`.
 - __[VSCode](https://code.visualstudio.com/download)__ (recomendado): Editor de código con plugins.
 - __[Conventional Commits](https://marketplace.visualstudio.com/items?itemName=vivaxy.vscode-conventional-commits&ssr=false#overview)__ (plugin-vscode) este plugin te ayudará a crear commit semántico.
-- Se usará la rama __dev__ *(https://github.com/leifermendez/bot-whatsapp/tree/dev)* como rama principal hasta que se haga oficialmente el lanzamiento de la V2.
 
 ### 🚀 Iniciando
 
-__Clonar repo rama dev__
+__Clona repositorio (desde tu fork)__
 ```
-git clone --branch dev https://github.com/codigoencasa/bot-whatsapp
+git clone https://github.com/codigoencasa/bot-whatsapp
 ```
 __Instalar dependencias__
 ``` 
 cd bot-whatsapp
-yarn install
+pnpm install
 ```
 
 __Compilar (build)__
 Para compilar la aplicación es necesario ejecutar este comando, el cual genera un directorio `lib` dentro de los paquetes del monorepo.
 
 ```
-yarn build
+pnpm run build
 ```
 
 __Example-app__
 Se ejecuta el CLI (Command Line Interface) para ayudarte a crear un app-bot de ejemplo.
 ```
-yarn run cli
+pnpm run cli
 ```
 
 Selecionas (mediante las flechas arriba y abajo) el proveedor que quieras usar y cuando estes sobre el presiona la barra de espacio, igualmente selecciona la base de datos que quieras usar.

README.md

@@ -26,7 +26,7 @@ Entiende más a fondo sus funcionalidades explicadas en nuestra documentación.
 - [📄 Documentación](https://bot-whatsapp.netlify.app/)
 - [🚀 Roadmap](https://github.com/orgs/codigoencasa/projects/1)
 - [💻 Discord](https://link.codigoencasa.com/DISCORD)
-- [👌 Twitter](https://twitter.com/leifermendez)
+- [👌 𝕏 (Twitter)](https://twitter.com/leifermendez)
 - [🎥 Youtube](https://www.youtube.com/watch?v=5lEMCeWEJ8o&list=PL_WGMLcL4jzWPhdhcUyhbFU6bC0oJd2BR)
 
 ## Comunidad

__mocks__/mock.provider.js

@@ -1,16 +1,14 @@
 const ProviderClass = require('../packages/bot/provider/provider.class')
+const { delay } = require('./env')
 class MockProvider extends ProviderClass {
     constructor() {
         super()
     }
 
-    delaySendMessage = (miliseconds, eventName, payload) =>
-        new Promise((res) =>
-            setTimeout(() => {
-                this.emit(eventName, payload)
-                res
-            }, miliseconds)
-        )
+    delaySendMessage = async (miliseconds, eventName, payload) => {
+        await delay(miliseconds)
+        this.emit(eventName, payload)
+    }
 
     sendMessage = async (userId, message) => {
         console.log(`Enviando... ${userId}, ${message}`)

__test__/0.0.0-case.test.js

@@ -31,7 +31,7 @@ suiteCase(`Responder a "hola"`, async ({ database, provider }) => {
 suiteCase(`NO reponder a "pepe"`, async ({ database, provider }) => {
     const flow = addKeyword('hola').addAnswer('Buenas!').addAnswer('Como vamos!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),

__test__/0.0.1-case.test.js

@@ -11,7 +11,7 @@ suiteCase.after.each(clear)
 suiteCase(`Responder a "EVENTS.LOCATION"`, async ({ database, provider }) => {
     const flow = addKeyword(EVENTS.LOCATION).addAnswer('Gracias por tu location')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -51,7 +51,7 @@ suiteCase(`Responder a "EVENTS.DOCUMENT"`, async ({ database, provider }) => {
 suiteCase(`Responder a "EVENTS.WELCOME"`, async ({ database, provider }) => {
     const flow = addKeyword(EVENTS.WELCOME).addAnswer('Bienvenido!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -71,7 +71,7 @@ suiteCase(`Responder a "EVENTS.WELCOME"`, async ({ database, provider }) => {
 suiteCase(`Responder a "EVENTS.MEDIA"`, async ({ database, provider }) => {
     const flow = addKeyword(EVENTS.MEDIA).addAnswer('gracias por la imagen o video!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -91,7 +91,7 @@ suiteCase(`Responder a "EVENTS.MEDIA"`, async ({ database, provider }) => {
 suiteCase(`Responder a "EVENTS.VOICE_NOTE"`, async ({ database, provider }) => {
     const flow = addKeyword(EVENTS.VOICE_NOTE).addAnswer('gracias por la nota de voz!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),

__test__/0.0.2-case.test.js

@@ -21,13 +21,13 @@ suiteCase(`Debe saltar de flujo`, async ({ database, provider }) => {
         })
         .addAnswer('este mensaje no deberia existir')
 
-    createBot({
+    await createBot({
         database,
         flow: createFlow([flujoBienvenida]),
         provider,
     })
 
-    await provider.delaySendMessage(0, 'message', {
+    await provider.delaySendMessage(50, 'message', {
         from: '000',
         body: 'hola',
     })
@@ -52,7 +52,7 @@ suiteCase(`Debe saltar de flujo con capture sin flowDynamic`, async ({ database,
         })
         .addAnswer('este mensaje no deberia existir')
 
-    createBot({
+    await createBot({
         database,
         flow: createFlow([flujoBienvenida]),
         provider,
@@ -90,7 +90,7 @@ suiteCase(`Debe saltar de flujo con capture con flowDynamic`, async ({ database,
         })
         .addAnswer('este mensaje no deberia existir')
 
-    createBot({
+    await createBot({
         database,
         flow: createFlow([flujoBienvenida]),
         provider,

__test__/0.1.0-case.test.js

@@ -11,7 +11,7 @@ suiteCase.after.each(clear)
 suiteCase(`Responder a "ole" en minuscula`, async ({ database, provider }) => {
     const flow = addKeyword(['ola', 'ole'], { sensitive: true }).addAnswer('Bienvenido a la OLA')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -31,7 +31,7 @@ suiteCase(`Responder a "ole" en minuscula`, async ({ database, provider }) => {
 suiteCase(`NO Responder a "ole" en minuscula`, async ({ database, provider }) => {
     const flow = addKeyword(['ola', 'ole'], { sensitive: true }).addAnswer('Bienvenido a la OLA')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),

__test__/0.1.1-case.test.js

@@ -51,11 +51,13 @@ suiteCase(`Debe continuar el flujo del hijo`, async ({ database, provider }) =>
         body: 'hola',
     })
 
+    await delay(0)
     await provider.delaySendMessage(30, 'message', {
         from: '000',
         body: 'test@test.com',
     })
 
+    await delay(0)
     await provider.delaySendMessage(60, 'message', {
         from: '000',
         body: 'paypal',
@@ -66,6 +68,7 @@ suiteCase(`Debe continuar el flujo del hijo`, async ({ database, provider }) =>
         body: 'continue!',
     })
 
+    await delay(0)
     await delay(800)
     const getHistory = database.listHistory.map((i) => i.answer)
 

__test__/0.1.2-case.test.js

@@ -15,7 +15,7 @@ suiteCase(`Responder a una expresion regular`, async ({ database, provider }) =>
         .addAnswer(`Gracias por proporcionar un numero de tarjeta valido`)
         .addAnswer('Fin!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -39,7 +39,7 @@ suiteCase(`NO Responder a una expresion regular`, async ({ database, provider })
         .addAnswer(`Gracias por proporcionar un numero de tarjeta valido`)
         .addAnswer('Fin!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),

__test__/0.1.3-case.test.js

@@ -13,7 +13,7 @@ suiteCase(`Responder a "pregunta"`, async ({ database, provider }) => {
         .addAnswer(['Hola como estas?', '¿Cual es tu edad?'], { capture: true })
         .addAnswer('Gracias por tu respuesta')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),

__test__/0.1.4-case.test.js

@@ -87,7 +87,7 @@ suiteCase(`Responder con un "string"`, async ({ database, provider }) => {
         })
         .addAnswer('y vos?')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -113,7 +113,7 @@ suiteCase(`Responder con un "array"`, async ({ database, provider }) => {
         })
         .addAnswer('y vos?')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -140,7 +140,7 @@ suiteCase(`Responder con un "object"`, async ({ database, provider }) => {
         })
         .addAnswer('y vos?')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -171,23 +171,23 @@ suiteCase(`FlowDynamic con capture`, async ({ database, provider }) => {
         )
         .addAnswer('fin!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
     })
 
-    await provider.delaySendMessage(0, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'hola',
     })
 
-    await provider.delaySendMessage(10, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'mal',
     })
 
-    await provider.delaySendMessage(20, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'bien',
     })
@@ -232,7 +232,7 @@ suiteCase(`FlowDynamic con capture en hijo`, async ({ database, provider }) => {
         [flowDos]
     )
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -243,17 +243,17 @@ suiteCase(`FlowDynamic con capture en hijo`, async ({ database, provider }) => {
         body: 'hola',
     })
 
-    await provider.delaySendMessage(10, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'flowDos',
     })
 
-    await provider.delaySendMessage(20, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'flow3',
     })
 
-    await provider.delaySendMessage(30, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'flow4',
     })

__test__/0.1.5-case.test.js

@@ -35,7 +35,7 @@ suiteCase(`Detener el flujo`, async ({ database, provider }) => {
             return flowDynamic(data)
         })
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -70,7 +70,7 @@ suiteCase(`Detener el flujo flowDynamic`, async ({ database, provider }) => {
         })
         .addAnswer('Como estas!')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
@@ -117,33 +117,33 @@ suiteCase(`flowDynamic con capture`, async ({ database, provider }) => {
         })
         .addAnswer('Puedes pasar')
 
-    createBot({
+    await createBot({
         database,
         provider,
         flow: createFlow([flow]),
     })
 
-    await provider.delaySendMessage(0, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'hola',
     })
 
-    await provider.delaySendMessage(10, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'this is not email value',
     })
 
-    await provider.delaySendMessage(20, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: 'test@test.com',
     })
 
-    await provider.delaySendMessage(90, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: '20',
     })
 
-    await provider.delaySendMessage(200, 'message', {
+    await provider.delaySendMessage(100, 'message', {
         from: '000',
         body: '18',
     })