Write tests for local notifications
Background
There are no tests for addNotification()
, removeNotification()
, and showNotification()
(upcoming, see #126 (closed)) in test/notifications.js
.
We should test these in relation to showNext()
and markAsShown()
.
What to change
Write tests for the current expected behavior of addNotification()
, removeNotification()
, and showNotification()
.
There are specifics about how these functions are expected to work with showNext()
and markAsShown()
(e.g. https://gitlab.com/eyeo/adblockplus/adblockpluscore/issues/126#changes-in-behavior) and these should be tested as well.
/cc @hfiguiere
Edited by Manish Jethani