Reviewers: mp+139677_code.launchpad.net, Message: Please take a look.
Description: Restored notifications tests. This branch restores the tests previously skipped (as a consequence of the topology refactor) in ``test_application_notifications``. Simplified the way the number of notifications is tested: now, rather than checking a value in the DOM, we check how many model instances are present in the notifications model list. Also added a test for the notifications view, that was no longer exercised due to the change described above. https://code.launchpad.net/~frankban/juju-gui/fix-notification-tests/+merge/139677 (do not edit description out of merge proposal) Please review this at https://codereview.appspot.com/6942045/ Affected files: A [revision details] M test/test_application_notifications.js -- https://code.launchpad.net/~frankban/juju-gui/fix-notification-tests/+merge/139677 Your team Juju GUI Hackers is requested to review the proposed merge of lp:~frankban/juju-gui/fix-notification-tests into lp:juju-gui. -- Mailing list: https://launchpad.net/~yellow Post to : [email protected] Unsubscribe : https://launchpad.net/~yellow More help : https://help.launchpad.net/ListHelp
