TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-12-25 03:44:29 +03:00
Code Issues Projects Releases Wiki Activity

Added checks for notification properties

Browse Source 
refs https://linear.app/tryghost/issue/CORE-64/resolve-undissmissable-update-notification-banners

- Before introducing new properties made sure existing ones were well understood and tested in a basic way
This commit is contained in:
Naz 2021-10-07 18:24:05 +02:00 committed by naz
parent 361184fc1c
commit 53ae852a0e
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
test/unit/server/services/notifications/notifications.test.js
View File

@ -175,7 +175,7 @@ describe('Notifications Service', function () {
createdAt: moment().toDate(),
status: 'alert',
type: 'info',
dismissible: true,
dismissible: false,
top: true,
message: 'Hello test world!'
}]
@ -183,6 +183,17 @@ describe('Notifications Service', function () {
allNotifications.length.should.equal(0);
notificationsToAdd.length.should.equal(1);
const createdNotification = notificationsToAdd[0];
createdNotification.id.should.not.be.undefined();
createdNotification.custom.should.be.true();
createdNotification.createdAt.should.not.be.undefined();
createdNotification.status.should.equal('alert');
createdNotification.type.should.equal('info');
createdNotification.dismissible.should.be.false();
createdNotification.top.should.be.true();
createdNotification.message.should.equal('Hello test world!');
});
});