Merge lp:~diegosarmentero/ubuntu-sso-client/930720 into lp:ubuntu-sso-client

Proposed by Diego Sarmentero
Status: Merged
Approved by: Natalia Bidart
Approved revision: 866
Merged at revision: 868
Proposed branch: lp:~diegosarmentero/ubuntu-sso-client/930720
Merge into: lp:ubuntu-sso-client
Diff against target: 117 lines (+39/-8)
4 files modified
ubuntu_sso/qt/success_page.py (+5/-2)
ubuntu_sso/qt/tests/test_qt_views.py (+2/-5)
ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py (+25/-0)
ubuntu_sso/qt/ubuntu_sso_wizard.py (+7/-1)
To merge this branch: bzr merge lp:~diegosarmentero/ubuntu-sso-client/930720
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Approve
Review via email: mp+92815@code.launchpad.net

Commit message

- Fixed: Qt UI: congratulations page is empty (LP: #930720).

To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Looks and works great!

review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :
Download full text (63.9 KiB)

The attempt to merge lp:~diegosarmentero/ubuntu-sso-client/930720 into lp:ubuntu-sso-client failed. Below is the output from the failed tests.

*** Running GTK test suite for ubuntu_sso ***
ubuntu_sso.tests.test_account
  AccountTestCase
    test_generate_captcha ... [OK]
    test_is_not_validated ... [OK]
    test_is_not_validated_empty_result ... [OK]
    test_is_validated ... [OK]
    test_login_if_http_error ... [OK]
    test_login_if_no_error ... [OK]
    test_register_user_checks_valid_email ... [OK]
    test_register_user_checks_valid_password ... [OK]
    test_register_user_if_status_error ... [OK]
    test_register_user_if_status_error_with_string_message ... [OK]
    test_register_user_if_status_ok ... [OK]
    test_register_user_if_status_unknown ... [OK]
    test_request_password_reset_token_if_http_error ... [OK]
    test_request_password_reset_token_if_status_ok ... [OK]
    test_request_password_reset_token_if_status_unknown ... [OK]
    test_set_new_password_if_http_error ... [OK]
    test_set_new_password_if_status_ok ... [OK]
    test_set_new_password_if_status_unknown ... [OK]
    test_validate_email_if_status_error ... [OK]
    test_validate_email_if_status_error_with_string_message ... [OK]
    test_validate_email_if_status_ok ... [OK]
    test_validate_email_if_status_unknown ... [OK]
  EnvironOverridesTestCase
    test_no_override_service_url ... [OK]
    test_override_service_url ... [OK]
    test_service_url_as_parameter ... [OK]
twisted.trial.unittest
  TestCase
    runTest ... [OK]
ubuntu_sso.tests.test_account
  TimestampedAuthorizerTestCase
    test_authorize_request_includes_timestamp ... [OK]
ubuntu_sso.tests.test_credentials
  BasicTestCase
    runTest ... [OK]
  ClearCredentialsTestCase
    test_clear_credentials ... [OK]
    test_keyring_failure ... [OK]
  CredentialsCallbacksTestCase
    test_creation_parameters_are_stored ... [OK]
    test_help_text_defaults_to_empty_string ... [OK]
    test_ping_url_defaults_to_none ... [OK]
    test_policy_url_defaults_to_none ... [OK]
    test_tc...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubuntu_sso/qt/success_page.py'
--- ubuntu_sso/qt/success_page.py 2012-02-07 18:59:36 +0000
+++ ubuntu_sso/qt/success_page.py 2012-02-13 18:15:24 +0000
@@ -17,11 +17,14 @@
17"""Success page UI."""17"""Success page UI."""
1818
19from ubuntu_sso.qt.gui import SSOWizardPage19from ubuntu_sso.qt.gui import SSOWizardPage
20from ubuntu_sso.utils.ui import SUCCESS
2021
2122
22class SuccessPage(SSOWizardPage):23class SuccessPage(SSOWizardPage):
23 """Page used to display success message."""24 """Page used to display success message."""
2425
25 def __init__(self, ui, *args, **kwargs):26 def __init__(self, ui, image_pixmap, *args, **kwargs):
26 super(SuccessPage, self).__init__(ui, *args, **kwargs)27 super(SuccessPage, self).__init__(ui, *args, **kwargs)
27 self.next = -128 self.ui.image_label.setPixmap(image_pixmap)
29 message = SUCCESS % {'app_name': self.app_name}
30 self.ui.success_message_body.setText(message)
2831
=== modified file 'ubuntu_sso/qt/tests/test_qt_views.py'
--- ubuntu_sso/qt/tests/test_qt_views.py 2012-02-13 16:05:13 +0000
+++ ubuntu_sso/qt/tests/test_qt_views.py 2012-02-13 18:15:24 +0000
@@ -75,18 +75,15 @@
75 """Setup tests."""75 """Setup tests."""
76 super(SuccessPageTestCase, self).setUp()76 super(SuccessPageTestCase, self).setUp()
77 self.ui = Ui_SuccessPage()77 self.ui = Ui_SuccessPage()
78 self.controller = FakeController()78 self.widget = SuccessPage(self.ui, QtGui.QPixmap())
79 self.widget = SuccessPage(self.ui, self.controller)
8079
81 def test_constructor(self):80 def test_constructor(self):
82 """Test that the constructor works as expected."""81 """Test that the constructor works as expected."""
83 mocker = Mocker()82 mocker = Mocker()
84 ui = mocker.mock()83 ui = mocker.mock()
85 controller = mocker.mock()
86 ui.setupUi(MATCH(lambda x: isinstance(x, SuccessPage)))84 ui.setupUi(MATCH(lambda x: isinstance(x, SuccessPage)))
87 controller.setupUi(MATCH(lambda x: isinstance(x, SuccessPage)))
88 mocker.replay()85 mocker.replay()
89 self.widget = SuccessPage(self.ui, self.controller)86 self.widget = SuccessPage(self.ui, QtGui.QPixmap())
9087
9188
92class ErrorPageTestCase(TestCase):89class ErrorPageTestCase(TestCase):
9390
=== modified file 'ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py'
--- ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py 2012-02-13 15:19:57 +0000
+++ ubuntu_sso/qt/tests/test_ubuntu_sso_wizard.py 2012-02-13 18:15:24 +0000
@@ -16,6 +16,7 @@
1616
17"""Test Ubunto SSO Wizard."""17"""Test Ubunto SSO Wizard."""
1818
19from PyQt4 import QtGui
19from twisted.internet import defer20from twisted.internet import defer
2021
21from ubuntu_sso.qt import ubuntu_sso_wizard22from ubuntu_sso.qt import ubuntu_sso_wizard
@@ -47,3 +48,27 @@
47 self.assertEqual(self._called,48 self.assertEqual(self._called,
48 ((ubuntu_sso_wizard.USER_SUCCESS,), {}))49 ((ubuntu_sso_wizard.USER_SUCCESS,), {}))
49 # pylint: enable=W021250 # pylint: enable=W0212
51
52
53class UbuntuSSOWizardTestCase(BaseTestCase):
54
55 """Test Case for UbuntuSSOWizard"""
56
57 @defer.inlineCallbacks
58 def setUp(self):
59 yield super(UbuntuSSOWizardTestCase, self).setUp()
60 self.app_name = 'app_name'
61 self.ui = ubuntu_sso_wizard.UbuntuSSOWizard(self.app_name)
62
63 # pylint: disable=W0212
64 def test_move_to_success_page_state(self):
65 """Test _move_to_success_page method."""
66 self.patch(self.ui, 'setButtonLayout', self._set_called)
67 self.ui._move_to_success_page(self.app_name)
68 self.assertEqual(self.ui._next_id, -1)
69 self.assertEqual(self.ui._next_id, -1)
70 self.assertEqual(self._called,
71 (([QtGui.QWizard.Stretch, QtGui.QWizard.FinishButton],), {}))
72 self.assertTrue(self.ui.button(QtGui.QWizard.FinishButton).isEnabled())
73 self.assertEqual(self.ui._done, ubuntu_sso_wizard.USER_SUCCESS)
74 # pylint: enable=W0212
5075
=== modified file 'ubuntu_sso/qt/ubuntu_sso_wizard.py'
--- ubuntu_sso/qt/ubuntu_sso_wizard.py 2012-02-13 17:12:09 +0000
+++ ubuntu_sso/qt/ubuntu_sso_wizard.py 2012-02-13 18:15:24 +0000
@@ -65,6 +65,7 @@
65 parent = kwargs.get('parent')65 parent = kwargs.get('parent')
66 super(UbuntuSSOWizard, self).__init__(parent)66 super(UbuntuSSOWizard, self).__init__(parent)
67 self._next_id = -167 self._next_id = -1
68 self.setOption(QWizard.HaveFinishButtonOnEarlyPages, True)
68 self._done = USER_CANCELLATION69 self._done = USER_CANCELLATION
6970
70 # store common useful data provided by the app71 # store common useful data provided by the app
@@ -117,7 +118,7 @@
117 self.email_verification.registrationSuccess.connect(118 self.email_verification.registrationSuccess.connect(
118 self._move_to_success_page)119 self._move_to_success_page)
119120
120 self.success = SuccessPage(Ui_SuccessPage(),121 self.success = SuccessPage(Ui_SuccessPage(), QPixmap(),
121 app_name=self.app_name, parent=self)122 app_name=self.app_name, parent=self)
122 self.error = ErrorPage(Ui_ErrorPage(),123 self.error = ErrorPage(Ui_ErrorPage(),
123 app_name=self.app_name)124 app_name=self.app_name)
@@ -199,6 +200,11 @@
199 """Move to the success page wizard."""200 """Move to the success page wizard."""
200 self._next_id = self.success_page_id201 self._next_id = self.success_page_id
201 self.next()202 self.next()
203 self.setButtonLayout([
204 QWizard.Stretch,
205 QWizard.FinishButton])
206 self.button(QWizard.FinishButton).setEnabled(True)
207 self.button(QWizard.FinishButton).setFocus()
202 self._done = USER_SUCCESS208 self._done = USER_SUCCESS
203 self._next_id = -1209 self._next_id = -1
204210

Subscribers

People subscribed via source and target branches