Merge lp:~elopio/ubuntuone-testing/fix-payment-typos into lp:ubuntuone-testing

Proposed by Leo Arias
Status: Merged
Approved by: Rick McBride
Approved revision: 31
Merged at revision: 31
Proposed branch: lp:~elopio/ubuntuone-testing/fix-payment-typos
Merge into: lp:ubuntuone-testing
Diff against target: 25 lines (+2/-2)
2 files modified
ubuntuone/web/tests/sst/payments/u1webp001_paymusicstreaming.py (+1/-1)
ubuntuone/web/tests/sst/shared/actions/payment_confirmed.py (+1/-1)
To merge this branch: bzr merge lp:~elopio/ubuntuone-testing/fix-payment-typos
Reviewer Review Type Date Requested Status
Rick McBride (community) Approve
Review via email: mp+76513@code.launchpad.net

Commit message

Fixes two typos on payments tests.

Description of the change

I made two typos on the payments actions. Currrently we really can't test this tests easily because they will make the other tests fail by changing the subscriptions of the user, but it's better to have them as good as possible while we can test them.

To post a comment you must log in.
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/payments/u1webp001_paymusicstreaming.py'
2--- ubuntuone/web/tests/sst/payments/u1webp001_paymusicstreaming.py 2011-09-19 19:08:03 +0000
3+++ ubuntuone/web/tests/sst/payments/u1webp001_paymusicstreaming.py 2011-09-22 05:33:43 +0000
4@@ -30,7 +30,7 @@
5 from datetime import date
6 import actions.services as services_actions
7 import actions.payment as payment_actions
8-import actions.paymet_confirmed as payment_confirmed_actions
9+import actions.payment_confirmed as payment_confirmed_actions
10
11 set_base_url(base_url)
12 login(username, password)
13
14=== modified file 'ubuntuone/web/tests/sst/shared/actions/payment_confirmed.py'
15--- ubuntuone/web/tests/sst/shared/actions/payment_confirmed.py 2011-09-15 07:32:09 +0000
16+++ ubuntuone/web/tests/sst/shared/actions/payment_confirmed.py 2011-09-22 05:33:43 +0000
17@@ -49,7 +49,7 @@
18 expire_date = today.replace(year = today.year + 1)
19 # TODO assert the paid amount.
20 # TODO assert payments in euros.
21- get_element(tag='dt', css_class='purchase_details'
22+ get_element(tag='dt', css_class='purchase_details',
23 text='Expires: %(expire_date)s'
24 % {'expire_date': expire_date.strftime('%B %-d, %Y')})
25 get_element(tag='p', css_class='explanation',

Subscribers

People subscribed via source and target branches