Merge lp:~elopio/ubuntuone-testing/stabilize-notes into lp:ubuntuone-testing

Proposed by Leo Arias
Status: Merged
Approved by: Rick McBride
Approved revision: 85
Merged at revision: 77
Proposed branch: lp:~elopio/ubuntuone-testing/stabilize-notes
Merge into: lp:ubuntuone-testing
Prerequisite: lp:~elopio/ubuntuone-testing/stabilize-contacts
Diff against target: 20 lines (+2/-2)
1 file modified
ubuntuone/web/tests/sst/shared/actions/notes.py (+2/-2)
To merge this branch: bzr merge lp:~elopio/ubuntuone-testing/stabilize-notes
Reviewer Review Type Date Requested Status
Rick McBride (community) Approve
Review via email: mp+91191@code.launchpad.net

Commit message

Stabilize notes tests.

Description of the change

Fixes and updates to stabilize the notes tests.

To post a comment you must log in.
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :

The prerequisite lp:~elopio/ubuntuone-testing/stabilize-contacts has not yet been merged into lp:ubuntuone-testing.

Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0, Needs Fixing == 0, Needs Information == 0, Resubmit == 0, Pending == 0. Got: 1 Pending.

Revision history for this message
Rick McBride (rmcbride) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntuone/web/tests/sst/shared/actions/notes.py'
2--- ubuntuone/web/tests/sst/shared/actions/notes.py 2011-12-09 18:19:11 +0000
3+++ ubuntuone/web/tests/sst/shared/actions/notes.py 2012-02-02 00:58:18 +0000
4@@ -64,7 +64,7 @@
5
6
7 def _get_note_title():
8- return get_element_by_css('#notes-main > h1').text
9+ return get_element_by_css('#notes-main > h2').text
10
11
12 def _get_note_body():
13@@ -119,7 +119,7 @@
14 """Delete the note"""
15 select_note(title)
16 delete_button = get_element_by_css('li.selected > a.delete-button')
17- click_element(delete_button)
18+ click_element(delete_button, wait=False)
19 accept_alert()
20 wait_for_action_to_complete()
21

Subscribers

People subscribed via source and target branches