Merge lp:~ralsina/ubuntu-sso-client/fix_show_gui into lp:ubuntu-sso-client

[Roberto Alsina]

Use CredentialsManagementClient instead of the deprecated API

Use CredentialsManagementClient instead of the deprecated API

[Diego Sarmentero (diegosarmentero)]

+1

(just a comment: is the print statement necessary?
It's not part of this branch changes, but you can see it at the end.)

review: Approve

[Roberto Alsina (ralsina)]

> +1
>
> (just a comment: is the print statement necessary?
> It's not part of this branch changes, but you can see it at the end.)

Not necessary but well, it was there already.

=== modified file 'ubuntu_sso/qt/tests/show_gui.py'
--- ubuntu_sso/qt/tests/show_gui.py	2011-06-30 03:46:20 +0000
+++ ubuntu_sso/qt/tests/show_gui.py	2011-08-02 18:27:38 +0000
@@ -20,6 +20,12 @@
 from twisted.internet import reactor
 from twisted.internet.defer import inlineCallbacks
 from ubuntu_sso.main.windows import UbuntuSSOClient
+from ubuntu_sso.credentials import (
+     TC_URL_KEY,
+     HELP_TEXT_KEY,
+     WINDOW_ID_KEY,
+     UI_MODULE_KEY,
+)
 
 
 def found(*args):
@@ -33,11 +39,16 @@
     """Perform a client request to be logged in."""
     client = UbuntuSSOClient()
     client = yield client.connect()
-    client.sso_cred.on_credentials_found_cb = found
-    yield client.sso_cred.register_to_signals()
-    yield client.sso_cred.login_or_register_to_get_credentials('Ubuntu One',
-                                                    'http://www.google.com',
-                                                    'This is a test.', 0)
+    client.cred_management.on_credentials_found_cb = found
+    yield client.cred_management.register_to_signals()
+    yield client.cred_management.login(
+        'Ubuntu One',
+        {
+            TC_URL_KEY: 'http://www.google.com',
+            HELP_TEXT_KEY: 'This is a test.',
+            WINDOW_ID_KEY: 0,
+            UI_MODULE_KEY: 'ubuntu_sso.qt.gui',
+        })
     print "called ok"