Skip to content
This repository has been archived by the owner on Nov 5, 2022. It is now read-only.

Add tests for notifications.getAll #3

Open
m-alger opened this issue Jan 15, 2016 · 0 comments
Open

Add tests for notifications.getAll #3

m-alger opened this issue Jan 15, 2016 · 0 comments
Labels

Comments

@m-alger
Copy link
Contributor

m-alger commented Jan 15, 2016

This will require adding some way to reset the internal state of the notifications polyfill — otherwise, the output of getAll will be unpredictable, since it will return all notifications that have been involved in tests thus far.

Following the example of the storage polyfill tests (currently in a CL here), such a reset function should be included in the chrome.caterpillar.notifications namespace and should include a ForTests suffix, e.g. chrome.caterpillar.notifications.resetNotificationsForTests.

@m-alger m-alger added the tests label Jan 15, 2016
@m-alger m-alger changed the title Add tests for notifications.getAll. Add tests for notifications.getAll Jan 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant