Restored notifications tests.

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)

[frankban, 2012-12-13 11:58:04]

Please take a look.

[gary.poster, 2012-12-13 13:29:13]

Land as is.

Really nice, Francesco. Thank you.

Gary

[benjamin.saller, 2012-12-13 13:58:41]

As a fly by review, this branch made me happier than it should have, Thanks.

[frankban, 2012-12-13 14:44:41]

Hi Gary and Ben,

thanks for the reviews.

[frankban, 2012-12-13 14:53:51]

*** Submitted:

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.

R=gary.poster, benjamin.saller
CC=
https://codereview.appspot.com/6942045