diff options
| author | kkania@chromium.org <kkania@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-08 21:04:01 +0000 |
|---|---|---|
| committer | kkania@chromium.org <kkania@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-08 21:04:01 +0000 |
| commit | e04975a7279393c8ba849cac4d96b6583bf024d5 (patch) | |
| tree | 6716557ed199110c0cd88929ad15959dd51fd506 | |
| parent | b835c68f6c1e4e0152ac7a061093aa4dcb62ab25 (diff) | |
| download | chromium_src-e04975a7279393c8ba849cac4d96b6583bf024d5.zip chromium_src-e04975a7279393c8ba849cac4d96b6583bf024d5.tar.gz chromium_src-e04975a7279393c8ba849cac4d96b6583bf024d5.tar.bz2 | |
Fix notifications.NotificationsTest.testNotificationOrderAfterClosingOne.
The third notification was not showing because it did not fit on the screen.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/5552007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68635 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/test/functional/notifications.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/chrome/test/functional/notifications.py b/chrome/test/functional/notifications.py index 5440dbb..29bf31b 100644 --- a/chrome/test/functional/notifications.py +++ b/chrome/test/functional/notifications.py @@ -15,8 +15,6 @@ class NotificationsTest(pyauto.PyUITest): def __init__(self, methodName='runTest'): super(NotificationsTest, self).__init__(methodName) self.NO_SUCH_URL = 'http://no_such_url_exists/' - self.NO_SUCH_URL2 = 'http://no_such_url_exists2/' - self.NO_SUCH_URL3 = 'http://no_such_url_exists3/' # Content settings for default notification permission. self.ALLOW_ALL_SETTING = 1 self.DENY_ALL_SETTING = 2 @@ -514,15 +512,16 @@ class NotificationsTest(pyauto.PyUITest): """ self._AllowAllOrigins() self.NavigateToURL(self.TEST_PAGE_URL) - self._CreateHTMLNotification(self.NO_SUCH_URL) - self._CreateHTMLNotification(self.NO_SUCH_URL2) - self._CreateHTMLNotification(self.NO_SUCH_URL3) + self._CreateSimpleNotification('', 'Title1', '') + self._CreateSimpleNotification('', 'Title2', '') + self._CreateSimpleNotification('', 'Title3', '') self.assertEquals(3, len(self.GetActiveNotifications())) + old_notifications = self.GetActiveNotifications() self.CloseNotification(1) self.assertEquals(2, len(self.GetActiveNotifications())) - notifications = self.GetActiveNotifications() - self.assertEquals(self.NO_SUCH_URL, notifications[0]['content_url']) - self.assertEquals(self.NO_SUCH_URL3, notifications[1]['content_url']) + new_notifications = self.GetActiveNotifications() + self.assertEquals(old_notifications[0], new_notifications[0]) + self.assertEquals(old_notifications[2], new_notifications[1]) def testNotificationReplacement(self): """Test that we can replace a notification using the replaceId.""" |