Merge lp:~dobey/ubuntuone-control-panel/update-13-10 into lp:ubuntuone-control-panel/stable-13-10

Proposed by dobey
Status: Merged
Approved by: dobey
Approved revision: 408
Merged at revision: 407
Proposed branch: lp:~dobey/ubuntuone-control-panel/update-13-10
Merge into: lp:ubuntuone-control-panel/stable-13-10
Diff against target: 86 lines (+17/-7)
5 files modified
setup.py (+1/-1)
ubuntuone/controlpanel/__init__.py (+5/-1)
ubuntuone/controlpanel/backend.py (+8/-3)
ubuntuone/controlpanel/gui/__init__.py (+1/-1)
ubuntuone/controlpanel/tests/test_web_client.py (+2/-1)
To merge this branch: bzr merge lp:~dobey/ubuntuone-control-panel/update-13-10
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve
Review via email: mp+186152@code.launchpad.net

Commit message

[Rodney Dawes]

    - Include the scheme and host in the timestamp test, as new oauthlib bails on signing if the values are not included in the URI.

[Michael McCracken]

    - Fix a lint issue that was causing build failure on saucy.

[Natalia B. Bidart]

    - Allow Ubuntu One base URL to be overridden using an environment variable.

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

The attempt to merge lp:~dobey/ubuntuone-control-panel/update-13-10 into lp:ubuntuone-control-panel/stable-13-10 failed. Below is the output from the failed tests.

*** Running DBus test suite ***

Traceback (most recent call last):
  File "/usr/bin/u1trial", line 40, in <module>
    main()
  File "/usr/lib/python2.7/dist-packages/ubuntuone-dev-tools/ubuntuone/devtools/runners/__init__.py", line 277, in main
    suite = test_runner.get_suite(options)
  File "/usr/lib/python2.7/dist-packages/ubuntuone-dev-tools/ubuntuone/devtools/runners/__init__.py", line 173, in get_suite
    config['ignore-paths']))
  File "/usr/lib/python2.7/dist-packages/ubuntuone-dev-tools/ubuntuone/devtools/runners/__init__.py", line 157, in _collect_tests
    module_suite = self._load_unittest(filepath)
  File "/usr/lib/python2.7/dist-packages/ubuntuone-dev-tools/ubuntuone/devtools/runners/__init__.py", line 98, in _load_unittest
    module = __import__(modpath, None, None, [""])
  File "/mnt/tarmac/cache/ubuntuone-control-panel/stable-13-10/ubuntuone/controlpanel/dbustests/__init__.py", line 28, in <module>
    from ubuntuone.controlpanel import dbus_service
  File "/mnt/tarmac/cache/ubuntuone-control-panel/stable-13-10/ubuntuone/controlpanel/dbus_service.py", line 27, in <module>
    from ubuntuone.syncdaemon.interaction_interfaces import bool_str
ImportError: No module named syncdaemon.interaction_interfaces

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

The attempt to merge lp:~dobey/ubuntuone-control-panel/update-13-10 into lp:ubuntuone-control-panel/stable-13-10 failed. Below is the output from the failed tests.

*** Running DBus test suite ***
ubuntuone.controlpanel.dbustests.test_dbus_service
  BaseTestCase
    runTest ... [OK]
  DBusServiceMainTestCase
    test_dbus_service_cant_register ... Control panel backend already running.
                                   [OK]
    test_dbus_service_main ... [OK]
  DBusServiceTestCase
    test_cant_register_twice ... [SKIPPED]
    test_dbus_busname_created ... [OK]
    test_error_handler_default ... [OK]
    test_error_handler_with_exception ... [OK]
    test_error_handler_with_failure ... [OK]
    test_error_handler_with_non_string_dict ... [OK]
    test_error_handler_with_string_dict ... [OK]
    test_register_service ... [OK]
  FileSyncTestCase
    test_file_sync_status_changed ... [OK]
    test_file_sync_status_disabled ... [OK]
    test_file_sync_status_disconnected ... [OK]
    test_file_sync_status_error ... [OK]
    test_file_sync_status_idle ... [OK]
    test_file_sync_status_starting ... [OK]
    test_file_sync_status_stopped ... [OK]
    test_file_sync_status_syncing ... [OK]
    test_file_sync_status_unknown ... [OK]
    test_status_changed_handler ... [OK]
    test_status_changed_handler_after_status_requested ... [OK]
    test_status_changed_handler_after_status_requested_twice ... [OK]
  OperationsAuthErrorTestCase
    test_account_info_returned ... [OK]
    test_change_device_settings ... [OK]
    test_change_replication_settings ... [OK]
    test_change_volume_settings ... [OK]
    test_connect_files ... [OK]
    test_devices_info_returned ... [OK]
    test_disable_files ... [OK]
    test_disconnect_files ... [OK]
    test_enable_files ... [OK]
    test_remove_device ... [OK]
    test_replications_info ... [OK]
    test_restart_files ... [OK]
 ...

408. By dobey

Merge test fix.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'setup.py'
2--- setup.py 2013-08-05 20:19:28 +0000
3+++ setup.py 2013-09-18 16:51:21 +0000
4@@ -113,7 +113,7 @@
5 (qrc_file, py_file)) > 0:
6 self.warn('Unable to generate python module {py_file}'
7 ' for resource file {qrc_file}'.format(
8- py_file=py_file, qrc_file=qrc_file))
9+ py_file=py_file, qrc_file=qrc_file))
10 if not os.path.exists(py_file) or not file(py_file).read():
11 raise SystemExit(1)
12 else:
13
14=== modified file 'ubuntuone/controlpanel/__init__.py'
15--- ubuntuone/controlpanel/__init__.py 2012-11-29 19:36:53 +0000
16+++ ubuntuone/controlpanel/__init__.py 2013-09-18 16:51:21 +0000
17@@ -24,9 +24,13 @@
18
19 """
20
21+import os
22+
23 # constants
24 DBUS_BUS_NAME = "com.ubuntuone.controlpanel"
25 DBUS_PREFERENCES_PATH = "/preferences"
26 DBUS_PREFERENCES_IFACE = "com.ubuntuone.controlpanel.Preferences"
27
28-WEBSERVICE_BASE_URL = u"https://one.ubuntu.com/api/"
29+UBUNTUONE_BASE_URL = os.environ.get(
30+ 'UONE_BASE_URL', u'https://one.ubuntu.com').rstrip('/') + '/'
31+WEBSERVICE_BASE_URL = UBUNTUONE_BASE_URL + u'api/'
32
33=== modified file 'ubuntuone/controlpanel/backend.py'
34--- ubuntuone/controlpanel/backend.py 2013-02-22 19:54:41 +0000
35+++ ubuntuone/controlpanel/backend.py 2013-09-18 16:51:21 +0000
36@@ -28,7 +28,12 @@
37 DeferredLock)
38 from ubuntuone.platform.credentials import CredentialsManagementTool
39
40-from ubuntuone.controlpanel import sd_client, replication_client
41+from ubuntuone.controlpanel import (
42+ UBUNTUONE_BASE_URL,
43+ WEBSERVICE_BASE_URL,
44+ replication_client,
45+ sd_client,
46+)
47 from ubuntuone.controlpanel.logger import setup_logging, log_call
48 from ubuntuone.controlpanel.web_client import WebClient
49 from ubuntu_sso.utils.webclient.common import (
50@@ -73,8 +78,8 @@
51
52 CONTACTS_PKG = 'thunderbird-couchdb'
53
54-UBUNTUONE_FROM_OAUTH = u'https://one.ubuntu.com/api/1.0/from_oauth/'
55-UBUNTUONE_LINK = u'https://one.ubuntu.com/'
56+UBUNTUONE_FROM_OAUTH = WEBSERVICE_BASE_URL + u'1.0/from_oauth/'
57+UBUNTUONE_LINK = UBUNTUONE_BASE_URL
58
59
60 def append_path_sep(path):
61
62=== modified file 'ubuntuone/controlpanel/gui/__init__.py'
63--- ubuntuone/controlpanel/gui/__init__.py 2013-06-12 14:32:34 +0000
64+++ ubuntuone/controlpanel/gui/__init__.py 2013-09-18 16:51:21 +0000
65@@ -70,7 +70,7 @@
66
67 FILE_URI_PREFIX = u'file://'
68
69-ACCEPT_SHARES = 'https://one.ubuntu.com/files/shareoffer/%s/'
70+ACCEPT_SHARES = UBUNTUONE_LINK + u'files/shareoffer/%s/'
71 CONTACTS_LINK = UBUNTUONE_LINK
72 DASHBOARD = UBUNTUONE_LINK + u'dashboard/'
73 EDIT_ACCOUNT_LINK = UBUNTUONE_LINK + u'account/'
74
75=== modified file 'ubuntuone/controlpanel/tests/test_web_client.py'
76--- ubuntuone/controlpanel/tests/test_web_client.py 2013-05-31 16:49:34 +0000
77+++ ubuntuone/controlpanel/tests/test_web_client.py 2013-09-18 16:51:21 +0000
78@@ -199,7 +199,8 @@
79 @defer.inlineCallbacks
80 def test_uses_timestamper(self):
81 """Test that the signed url is using the serverrelative timestamp."""
82- signed = yield self.wc.build_signed_iri(u'/blah?foo=bar')
83+ signed = yield self.wc.build_signed_iri(
84+ SAMPLE_TARGET + u'/blah?foo=bar')
85 parsed_signed = urlparse(signed)
86 signed_query = parse_qs(parsed_signed.query)
87

Subscribers

People subscribed via source and target branches

to all changes: