Merge lp:~alecu/ubuntuone-control-panel/fix-997326 into lp:ubuntuone-control-panel

Proposed by Alejandro J. Cura
Status: Merged
Approved by: dobey
Approved revision: 394
Merged at revision: 393
Proposed branch: lp:~alecu/ubuntuone-control-panel/fix-997326
Merge into: lp:ubuntuone-control-panel
Diff against target: 49 lines (+10/-2)
2 files modified
ubuntuone/controlpanel/gui/qt/main/__init__.py (+3/-1)
ubuntuone/controlpanel/gui/qt/main/tests/test_main.py (+7/-1)
To merge this branch: bzr merge lp:~alecu/ubuntuone-control-panel/fix-997326
Reviewer Review Type Date Requested Status
Manuel de la Peña (community) Approve
Michał Karnicki (community) Approve
Review via email: mp+143206@code.launchpad.net

Commit message

- Use a standard LC_CTYPE locale when the LANG is turkish. (LP: #997326).

Description of the change

This branch has the following SSO branch as a pre-requisite:
lp:~alecu/ubuntu-sso-client/fix-997326

To post a comment you must log in.
Revision history for this message
Michał Karnicki (karni) :
review: Approve
Revision history for this message
Manuel de la Peña (mandel) :
review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :
Download full text (54.8 KiB)

The attempt to merge lp:~alecu/ubuntuone-control-panel/fix-997326 into lp:ubuntuone-control-panel 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]
    test_start_f...

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

The attempt to merge lp:~alecu/ubuntuone-control-panel/fix-997326 into lp:ubuntuone-control-panel 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]
    test_start_f...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntuone/controlpanel/gui/qt/main/__init__.py'
2--- ubuntuone/controlpanel/gui/qt/main/__init__.py 2012-12-20 00:17:43 +0000
3+++ ubuntuone/controlpanel/gui/qt/main/__init__.py 2013-01-15 22:25:25 +0000
4@@ -1,6 +1,6 @@
5 # -*- coding: utf-8 -*-
6 #
7-# Copyright 2011 Canonical Ltd.
8+# Copyright 2011-2013 Canonical Ltd.
9 #
10 # This program is free software: you can redistribute it and/or modify it
11 # under the terms of the GNU General Public License version 3, as published
12@@ -21,6 +21,7 @@
13 import sys
14
15 from PyQt4 import QtCore
16+from ubuntu_sso.utils.locale import fix_turkish_locale
17
18 from ubuntuone.controlpanel.gui.qt.gui import start
19 # Module used to include the resources into this file
20@@ -86,6 +87,7 @@
21
22 def main(args, install_reactor_darwin=False):
23 """Start the Qt mainloop and open the main window."""
24+ fix_turkish_locale()
25
26 # Disable the overlay-scrollbar GTK module that was
27 # added in Ubuntu 12.10 because it breaks Qt (LP:1007421)
28
29=== modified file 'ubuntuone/controlpanel/gui/qt/main/tests/test_main.py'
30--- ubuntuone/controlpanel/gui/qt/main/tests/test_main.py 2012-11-08 19:25:16 +0000
31+++ ubuntuone/controlpanel/gui/qt/main/tests/test_main.py 2013-01-15 22:25:25 +0000
32@@ -1,6 +1,6 @@
33 # -*- coding: utf-8 -*-
34 #
35-# Copyright 2012 Canonical Ltd.
36+# Copyright 2012-2013 Canonical Ltd.
37 #
38 # This program is free software: you can redistribute it and/or modify it
39 # under the terms of the GNU General Public License version 3, as published
40@@ -325,3 +325,9 @@
41
42 main.main([sys.argv[0], '--with-icon'], install_reactor_darwin=False)
43 self.assertEqual(self._called, ((self.app,), {'quit_kills_sd': False}))
44+
45+ def test_fix_turkish_locale_called(self):
46+ """The fix_turkish_locale function is called, always."""
47+ self.patch(main, 'fix_turkish_locale', self._set_called)
48+ main.main([sys.argv[0]])
49+ self.assertEqual(self._called, ((), {}))

Subscribers

People subscribed via source and target branches