verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

4797-lilium-worker-notifications #1229

Merged
pau merged 37 commits from 4797-lilium-worker-notifications into dev 2023-02-02 08:53:29 +00:00
Conversation 0 Commits 37 Files Changed 8 +22
1 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a425d54bf6 - Show all commits

22
back/models/specs/notificationSubscription.spec.js Normal file
View File

@ -0,0 +1,22 @@
const app = require('vn-loopback/server/server');
pau marked this conversation as resolved Outdated
alexm commented 2023-01-23 06:42:08 +00:00
Outdated
Review
Copy Link

const models = require('vn-loopback/server/server').models;

const models = require('vn-loopback/server/server').models;
describe('loopback model NotificationSubscription', () => {
it('Should fail to delete a notification if the user is not editing itself or a subordinate', async() => {
pau marked this conversation as resolved
alexm commented 2023-01-23 06:43:52 +00:00
Review
Copy Link

Poner un caso en el que te deje si eres tu mismo y otro que te deje si es tu jefe, te hará falta transaccionar la ruta para poder deshacer el delete

Poner un caso en el que te deje si eres tu mismo y otro que te deje si es tu jefe, te hará falta transaccionar la ruta para poder deshacer el delete
const user = 9;
const notificationSubscriptionId = 2;
const ctx = {req: {accessToken: {userId: user}}};
const models = app.models;
const notification = await models.NotificationSubscription.findById(notificationSubscriptionId);
let error;
try {
await models.NotificationSubscription.deleteNotification(ctx, notification.id);
} catch (e) {
error = e;
}
expect(error.message).toContain('You dont have permission to modify this user');
});
});