Merge lp:~nataliabidart/ubuntu-sso-client/captcha-reload-on-error into lp:ubuntu-sso-client

Proposed by Natalia Bidart
Status: Merged
Merge reported by: dobey
Merged at revision: not available
Proposed branch: lp:~nataliabidart/ubuntu-sso-client/captcha-reload-on-error
Merge into: lp:ubuntu-sso-client
Diff against target: 0 lines
To merge this branch: bzr merge lp:~nataliabidart/ubuntu-sso-client/captcha-reload-on-error
Reviewer Review Type Date Requested Status
Matt Griffin (community) Approve
Alejandro J. Cura (community) Approve
Review via email: mp+39769@code.launchpad.net

Commit message

* Captcha is reloaded when any registration error occurs (LP: #634593).
* When there is a captcha generation error, a warning message is shown and captcha is reloaded.
* Removing GtkWarning when running tests.

Description of the change

* Verification token entry is now centered.

* Removed the following warning from the test run:

GtkWarning: Attempting to add a widget with type GtkAlignment to a GtkEventBox, but as a GtkBin subclass a GtkEventBox can only contain one widget at a time; it already contains a widget of type GtkAlignment

To test, please run within this branch:

* run the dbus daemon with this command:

killall ubuntu-sso-login; DEBUG=True PYTHONPATH=. ./bin/ubuntu-sso-login

* run the SSO gui by invoking in d-feet the com.ubuntu.sso bus name, /com/ubuntu/sso/credentials object path, CredentialsManagement interface, 'register' method. Parameters to pass are:

'Ubuntu Two', {}

* Try to register with setting a wrong captcha. You'll see the error in the registration page and the captcha will be reloaded. You can also try to use an already registered email and you'll get the error and the captcha will be reloaded as well.

To post a comment you must log in.
Revision history for this message
Alejandro J. Cura (alecu) wrote :

When merging the branch with trunk:
Text conflict in ubuntu_sso/tests/test_gui.py

review: Needs Fixing
Revision history for this message
Alejandro J. Cura (alecu) wrote :

Other than the merge problem mentioned above, the branch looks and works fine.
I've found that the best way to test it seems to try to register using an email address that is already registered, so an error will show up on the registration dialog, and the captcha would be reloaded.

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Conflict fixed!

Revision history for this message
Alejandro J. Cura (alecu) wrote :

Approved!

review: Approve
Revision history for this message
Matt Griffin (mattgriffin) wrote :

Tested and worked :) Approved!

review: Approve
Revision history for this message
dobey (dobey) wrote :

Attempt to merge into lp:ubuntu-sso-client failed due to conflicts:

text conflict in ubuntu_sso/tests/test_gui.py

Revision history for this message
dobey (dobey) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Preview Diff

Empty

Subscribers

People subscribed via source and target branches