Skip to content

Commit 21a0730

Browse files
committed
feat: ⚡ lot improvments
1 parent 4ce4dbf commit 21a0730

17 files changed

+286
-113
lines changed

__mocks__/mock.provider.js

+5-7
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
const ProviderClass = require('../packages/bot/provider/provider.class')
2+
const { delay } = require('./env')
23
class MockProvider extends ProviderClass {
34
constructor() {
45
super()
56
}
67

7-
delaySendMessage = (miliseconds, eventName, payload) =>
8-
new Promise((res) =>
9-
setTimeout(() => {
10-
this.emit(eventName, payload)
11-
res
12-
}, miliseconds)
13-
)
8+
delaySendMessage = async (miliseconds, eventName, payload) => {
9+
await delay(miliseconds)
10+
this.emit(eventName, payload)
11+
}
1412

1513
sendMessage = async (userId, message) => {
1614
console.log(`Enviando... ${userId}, ${message}`)

__test__/0.0.0-case.test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ suiteCase(`Responder a "hola"`, async ({ database, provider }) => {
3131
suiteCase(`NO reponder a "pepe"`, async ({ database, provider }) => {
3232
const flow = addKeyword('hola').addAnswer('Buenas!').addAnswer('Como vamos!')
3333

34-
createBot({
34+
await createBot({
3535
database,
3636
provider,
3737
flow: createFlow([flow]),

__test__/0.0.1-case.test.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ suiteCase.after.each(clear)
1111
suiteCase(`Responder a "EVENTS.LOCATION"`, async ({ database, provider }) => {
1212
const flow = addKeyword(EVENTS.LOCATION).addAnswer('Gracias por tu location')
1313

14-
createBot({
14+
await createBot({
1515
database,
1616
provider,
1717
flow: createFlow([flow]),
@@ -51,7 +51,7 @@ suiteCase(`Responder a "EVENTS.DOCUMENT"`, async ({ database, provider }) => {
5151
suiteCase(`Responder a "EVENTS.WELCOME"`, async ({ database, provider }) => {
5252
const flow = addKeyword(EVENTS.WELCOME).addAnswer('Bienvenido!')
5353

54-
createBot({
54+
await createBot({
5555
database,
5656
provider,
5757
flow: createFlow([flow]),
@@ -71,7 +71,7 @@ suiteCase(`Responder a "EVENTS.WELCOME"`, async ({ database, provider }) => {
7171
suiteCase(`Responder a "EVENTS.MEDIA"`, async ({ database, provider }) => {
7272
const flow = addKeyword(EVENTS.MEDIA).addAnswer('gracias por la imagen o video!')
7373

74-
createBot({
74+
await createBot({
7575
database,
7676
provider,
7777
flow: createFlow([flow]),
@@ -91,7 +91,7 @@ suiteCase(`Responder a "EVENTS.MEDIA"`, async ({ database, provider }) => {
9191
suiteCase(`Responder a "EVENTS.VOICE_NOTE"`, async ({ database, provider }) => {
9292
const flow = addKeyword(EVENTS.VOICE_NOTE).addAnswer('gracias por la nota de voz!')
9393

94-
createBot({
94+
await createBot({
9595
database,
9696
provider,
9797
flow: createFlow([flow]),

__test__/0.0.2-case.test.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,13 @@ suiteCase(`Debe saltar de flujo`, async ({ database, provider }) => {
2121
})
2222
.addAnswer('este mensaje no deberia existir')
2323

24-
createBot({
24+
await createBot({
2525
database,
2626
flow: createFlow([flujoBienvenida]),
2727
provider,
2828
})
2929

30-
await provider.delaySendMessage(0, 'message', {
30+
await provider.delaySendMessage(50, 'message', {
3131
from: '000',
3232
body: 'hola',
3333
})
@@ -52,7 +52,7 @@ suiteCase(`Debe saltar de flujo con capture sin flowDynamic`, async ({ database,
5252
})
5353
.addAnswer('este mensaje no deberia existir')
5454

55-
createBot({
55+
await createBot({
5656
database,
5757
flow: createFlow([flujoBienvenida]),
5858
provider,
@@ -90,7 +90,7 @@ suiteCase(`Debe saltar de flujo con capture con flowDynamic`, async ({ database,
9090
})
9191
.addAnswer('este mensaje no deberia existir')
9292

93-
createBot({
93+
await createBot({
9494
database,
9595
flow: createFlow([flujoBienvenida]),
9696
provider,

__test__/0.1.0-case.test.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ suiteCase.after.each(clear)
1111
suiteCase(`Responder a "ole" en minuscula`, async ({ database, provider }) => {
1212
const flow = addKeyword(['ola', 'ole'], { sensitive: true }).addAnswer('Bienvenido a la OLA')
1313

14-
createBot({
14+
await createBot({
1515
database,
1616
provider,
1717
flow: createFlow([flow]),
@@ -31,7 +31,7 @@ suiteCase(`Responder a "ole" en minuscula`, async ({ database, provider }) => {
3131
suiteCase(`NO Responder a "ole" en minuscula`, async ({ database, provider }) => {
3232
const flow = addKeyword(['ola', 'ole'], { sensitive: true }).addAnswer('Bienvenido a la OLA')
3333

34-
createBot({
34+
await createBot({
3535
database,
3636
provider,
3737
flow: createFlow([flow]),

__test__/0.1.1-case.test.js

+3
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,13 @@ suiteCase(`Debe continuar el flujo del hijo`, async ({ database, provider }) =>
5151
body: 'hola',
5252
})
5353

54+
await delay(0)
5455
await provider.delaySendMessage(30, 'message', {
5556
from: '000',
5657
5758
})
5859

60+
await delay(0)
5961
await provider.delaySendMessage(60, 'message', {
6062
from: '000',
6163
body: 'paypal',
@@ -66,6 +68,7 @@ suiteCase(`Debe continuar el flujo del hijo`, async ({ database, provider }) =>
6668
body: 'continue!',
6769
})
6870

71+
await delay(0)
6972
await delay(800)
7073
const getHistory = database.listHistory.map((i) => i.answer)
7174

__test__/0.1.2-case.test.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ suiteCase(`Responder a una expresion regular`, async ({ database, provider }) =>
1515
.addAnswer(`Gracias por proporcionar un numero de tarjeta valido`)
1616
.addAnswer('Fin!')
1717

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

42-
createBot({
42+
await createBot({
4343
database,
4444
provider,
4545
flow: createFlow([flow]),

__test__/0.1.3-case.test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ suiteCase(`Responder a "pregunta"`, async ({ database, provider }) => {
1313
.addAnswer(['Hola como estas?', '¿Cual es tu edad?'], { capture: true })
1414
.addAnswer('Gracias por tu respuesta')
1515

16-
createBot({
16+
await createBot({
1717
database,
1818
provider,
1919
flow: createFlow([flow]),

__test__/0.1.4-case.test.js

+11-11
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ suiteCase(`Responder con un "string"`, async ({ database, provider }) => {
8787
})
8888
.addAnswer('y vos?')
8989

90-
createBot({
90+
await createBot({
9191
database,
9292
provider,
9393
flow: createFlow([flow]),
@@ -113,7 +113,7 @@ suiteCase(`Responder con un "array"`, async ({ database, provider }) => {
113113
})
114114
.addAnswer('y vos?')
115115

116-
createBot({
116+
await createBot({
117117
database,
118118
provider,
119119
flow: createFlow([flow]),
@@ -140,7 +140,7 @@ suiteCase(`Responder con un "object"`, async ({ database, provider }) => {
140140
})
141141
.addAnswer('y vos?')
142142

143-
createBot({
143+
await createBot({
144144
database,
145145
provider,
146146
flow: createFlow([flow]),
@@ -171,23 +171,23 @@ suiteCase(`FlowDynamic con capture`, async ({ database, provider }) => {
171171
)
172172
.addAnswer('fin!')
173173

174-
createBot({
174+
await createBot({
175175
database,
176176
provider,
177177
flow: createFlow([flow]),
178178
})
179179

180-
await provider.delaySendMessage(0, 'message', {
180+
await provider.delaySendMessage(100, 'message', {
181181
from: '000',
182182
body: 'hola',
183183
})
184184

185-
await provider.delaySendMessage(10, 'message', {
185+
await provider.delaySendMessage(100, 'message', {
186186
from: '000',
187187
body: 'mal',
188188
})
189189

190-
await provider.delaySendMessage(20, 'message', {
190+
await provider.delaySendMessage(100, 'message', {
191191
from: '000',
192192
body: 'bien',
193193
})
@@ -232,7 +232,7 @@ suiteCase(`FlowDynamic con capture en hijo`, async ({ database, provider }) => {
232232
[flowDos]
233233
)
234234

235-
createBot({
235+
await createBot({
236236
database,
237237
provider,
238238
flow: createFlow([flow]),
@@ -243,17 +243,17 @@ suiteCase(`FlowDynamic con capture en hijo`, async ({ database, provider }) => {
243243
body: 'hola',
244244
})
245245

246-
await provider.delaySendMessage(10, 'message', {
246+
await provider.delaySendMessage(100, 'message', {
247247
from: '000',
248248
body: 'flowDos',
249249
})
250250

251-
await provider.delaySendMessage(20, 'message', {
251+
await provider.delaySendMessage(100, 'message', {
252252
from: '000',
253253
body: 'flow3',
254254
})
255255

256-
await provider.delaySendMessage(30, 'message', {
256+
await provider.delaySendMessage(100, 'message', {
257257
from: '000',
258258
body: 'flow4',
259259
})

__test__/0.1.5-case.test.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ suiteCase(`Detener el flujo`, async ({ database, provider }) => {
3535
return flowDynamic(data)
3636
})
3737

38-
createBot({
38+
await createBot({
3939
database,
4040
provider,
4141
flow: createFlow([flow]),
@@ -70,7 +70,7 @@ suiteCase(`Detener el flujo flowDynamic`, async ({ database, provider }) => {
7070
})
7171
.addAnswer('Como estas!')
7272

73-
createBot({
73+
await createBot({
7474
database,
7575
provider,
7676
flow: createFlow([flow]),
@@ -117,33 +117,33 @@ suiteCase(`flowDynamic con capture`, async ({ database, provider }) => {
117117
})
118118
.addAnswer('Puedes pasar')
119119

120-
createBot({
120+
await createBot({
121121
database,
122122
provider,
123123
flow: createFlow([flow]),
124124
})
125125

126-
await provider.delaySendMessage(0, 'message', {
126+
await provider.delaySendMessage(100, 'message', {
127127
from: '000',
128128
body: 'hola',
129129
})
130130

131-
await provider.delaySendMessage(10, 'message', {
131+
await provider.delaySendMessage(100, 'message', {
132132
from: '000',
133133
body: 'this is not email value',
134134
})
135135

136-
await provider.delaySendMessage(20, 'message', {
136+
await provider.delaySendMessage(100, 'message', {
137137
from: '000',
138138
139139
})
140140

141-
await provider.delaySendMessage(90, 'message', {
141+
await provider.delaySendMessage(100, 'message', {
142142
from: '000',
143143
body: '20',
144144
})
145145

146-
await provider.delaySendMessage(200, 'message', {
146+
await provider.delaySendMessage(100, 'message', {
147147
from: '000',
148148
body: '18',
149149
})

__test__/0.1.6-case.test.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ suiteCase.after.each(clear)
1818
suiteCase(`WELCOME`, async ({ database, provider }) => {
1919
const flow = addKeyword(EVENTS.WELCOME).addAnswer('Bievenido')
2020

21-
createBot({
21+
await createBot({
2222
database,
2323
provider,
2424
flow: createFlow([flow]),
@@ -38,7 +38,7 @@ suiteCase(`WELCOME`, async ({ database, provider }) => {
3838
suiteCase(`MEDIA`, async ({ database, provider }) => {
3939
const flow = addKeyword(EVENTS.MEDIA).addAnswer('media recibido')
4040

41-
createBot({
41+
await createBot({
4242
database,
4343
provider,
4444
flow: createFlow([flow]),
@@ -57,7 +57,7 @@ suiteCase(`MEDIA`, async ({ database, provider }) => {
5757
suiteCase(`LOCATION`, async ({ database, provider }) => {
5858
const flow = addKeyword(EVENTS.LOCATION).addAnswer('location recibido')
5959

60-
createBot({
60+
await createBot({
6161
database,
6262
provider,
6363
flow: createFlow([flow]),
@@ -76,7 +76,7 @@ suiteCase(`LOCATION`, async ({ database, provider }) => {
7676
suiteCase(`DOCUMENT`, async ({ database, provider }) => {
7777
const flow = addKeyword(EVENTS.DOCUMENT).addAnswer('document recibido')
7878

79-
createBot({
79+
await createBot({
8080
database,
8181
provider,
8282
flow: createFlow([flow]),
@@ -95,7 +95,7 @@ suiteCase(`DOCUMENT`, async ({ database, provider }) => {
9595
suiteCase(`VOICE_NOTE`, async ({ database, provider }) => {
9696
const flow = addKeyword(EVENTS.VOICE_NOTE).addAnswer('voice recibido')
9797

98-
createBot({
98+
await createBot({
9999
database,
100100
provider,
101101
flow: createFlow([flow]),

0 commit comments

Comments
 (0)