verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 56 Releases Wiki Activity

4320-notificationQueue #1087

Merged
alexm merged 15 commits from 4320-notificationQueue into dev 2022-10-28 06:06:04 +00:00
Conversation 0 Commits 15 Files Changed 15 -3
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c24b975f26 - Show all commits

3
back/methods/notification/send.js
View File

@ -16,9 +16,6 @@ module.exports = Self => {
}); });
Self.send = async options => { Self.send = async options => {
if (process.env.NODE_ENV == 'test')
throw new UserError(`Action not allowed on the test environment`);
const models = Self.app.models; const models = Self.app.models;
const findStatus = 'pending'; const findStatus = 'pending';
alexm marked this conversation as resolved
juan commented 2022-10-17 08:21:33 +00:00
Review
Copy Link

En test no tiene que lanzar error y debe ejecutar todo el código a excepción de la linea que envía el correo, así se puede comprobar que el proceso funciona correctamente.

En test no tiene que lanzar error y debe ejecutar todo el código a excepción de la linea que envía el correo, así se puede comprobar que el proceso funciona correctamente.