Testing module not save scenarios

[derdeollo]

Describe the bug
Testing module not save scenarios

To Reproduce
Steps to reproduce the behavior:

1. Go to 'Testing'
2. Click on 'Record'
3. Send message
4. Stop recording
5. Save

Expected behavior
Testing scenarios saved

Console output

POST http://localhost:3000/api/v1/bots/tabula-rasa/mod/testing/saveScenario 400 (Bad Request)
(anonymous) @ xhr.js:210
n._execute @ bluebird.js:1011
A._resolveFromExecutor @ bluebird.js:3337
A @ bluebird.js:2922
e.exports @ xhr.js:15
e.exports @ dispatchRequest.js:56
u.request @ Axios.js:112
r.forEach.u.<computed> @ Axios.js:147
(anonymous) @ bind.js:9
(anonymous) @ full.bundle.js:6
l @ analytics.browser.es.js:121
(anonymous) @ analytics.browser.es.js:351
(anonymous) @ analytics.browser.es.js:176
as @ full.bundle.js:6
u @ full.bundle.js:6
(anonymous) @ full.bundle.js:6
n._execute @ bluebird.js:1011
A._resolveFromExecutor @ bluebird.js:3337
A @ bluebird.js:2922
(anonymous) @ full.bundle.js:6
c @ react-dom.production.min.js:14
h @ react-dom.production.min.js:15
(anonymous) @ react-dom.production.min.js:15
O @ react-dom.production.min.js:17
C @ react-dom.production.min.js:18
S @ react-dom.production.min.js:18
A @ react-dom.production.min.js:21
En @ react-dom.production.min.js:84
zs @ react-dom.production.min.js:250
Ie @ react-dom.production.min.js:39
Cn @ react-dom.production.min.js:86
(anonymous) @ react-dom.production.min.js:251
t.unstable_runWithPriority @ scheduler.production.min.js:18
Ls @ react-dom.production.min.js:251
Ln @ react-dom.production.min.js:85
Show 5 more frames
bluebird.js:1590 Unhandled rejection Error: Request failed with status code 400    at e.exports (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:2928123)    at e.exports (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4688056)    at XMLHttpRequest.x (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:2926513)From previous event:    at A.B [as _captureStackTrace] (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:2689263)    at A._resolveFromExecutor (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:2719623)    at new A (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:2712700)    at http://localhost:3000/assets/modules/testing/web/full.bundle.js:6:45535    at Object.c (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4574850)    at h (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4575005)    at http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4575643    at O (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4575732)    at C (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4576168)    at S (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4575980)    at A (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4577067)    at En (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4604467)    at zs (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4672351)    at Ie (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4584344)    at Cn (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4604947)    at http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4672629    at Object.t.unstable_runWithPriority (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4682954)    at Ls (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4672553)    at Ln (http://localhost:3000/assets/studio/ui/public/js/web.cf6726b94d97ce38d011.js?9a5d2a968ad558d7fc01:2:4604690)

Environment (please complete the following information):

• Browser Chrome
• Browser Version latest
• Botpress Version 12.26.10
• Docker

Additional context
Show details: ""

[allardy]

That module is broken since we implemented the web session ID and the messaging client. It usually used the visitor ID of a channel web user (stored in the local storage) to do its task, but since the way the webchat works was changed, it doesn't work anymore.

I'm not sure how this module could be fixed, we'd need a way for the testing module to determine the target of the user currently using the module, and also be able to create a new user/conversation to run the saved testing scenarios.

Since we're slowly removing module support, i'm not sure how it should be addressed. @EFF @laurentlp

[github-actions]

Available on v12.27.0