Merge lp:~vila/bzr/413514-bogus-locale into lp:~bzr/bzr/trunk-old

Proposed by Vincent Ladeuil
Status: Merged
Merged at revision: not available
Proposed branch: lp:~vila/bzr/413514-bogus-locale
Merge into: lp:~bzr/bzr/trunk-old
Diff against target: 43 lines
To merge this branch: bzr merge lp:~vila/bzr/413514-bogus-locale
Reviewer Review Type Date Requested Status
bzr-core Pending
Review via email: mp+10287@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Vincent Ladeuil (vila) wrote :

Pretty simple: as explained in #413514 comments, this:
- is specific to karmic which slightly change python behavior when a bogus LANG is specified,
- was already identified as a britle test,
- can be simplified since the important part (among the deleted ones) is covered by
  another test in test_utils_encoding.

Revision history for this message
Robert Collins (lifeless) wrote :

review +1

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'NEWS'
--- NEWS 2009-08-18 01:38:56 +0000
+++ NEWS 2009-08-18 08:35:57 +0000
@@ -9,6 +9,12 @@
9In Development9In Development
10##############10##############
1111
12Bug Fixes
13*********
14
15* Fix a test failure on karmic by making a locale test more robust.
16 (Vincent Ladeuil, #413514)
17
12Improvements18Improvements
13************19************
1420
1521
=== modified file 'bzrlib/tests/blackbox/test_locale.py'
--- bzrlib/tests/blackbox/test_locale.py 2009-03-23 14:59:43 +0000
+++ bzrlib/tests/blackbox/test_locale.py 2009-08-18 08:35:57 +0000
@@ -61,21 +61,7 @@
61 '--no-aliases --no-plugins log -q --log-format=long tree',61 '--no-aliases --no-plugins log -q --log-format=long tree',
62 env_changes={'LANG':'BOGUS', 'BZR_PROGRESS_BAR':'none',62 env_changes={'LANG':'BOGUS', 'BZR_PROGRESS_BAR':'none',
63 'LC_ALL':None, 'LC_CTYPE':None, 'LANGUAGE':None})63 'LC_ALL':None, 'LC_CTYPE':None, 'LANGUAGE':None})
64 # XXX: This depends on the exact formatting of a locale.Error64 self.assertStartsWith(err, 'bzr: warning: unsupported locale setting')
65 # as the first part of the string. It may be a little tempermental
66 self.assertEqualDiff("""\
67bzr: warning: unsupported locale setting
68 bzr could not set the application locale.
69 Although this should be no problem for bzr itself,
70 it might cause problems with some plugins.
71 To investigate the issue, look at the output
72 of the locale(1p) tool available on POSIX systems.
73bzr: warning: unsupported locale setting
74 Could not determine what text encoding to use.
75 This error usually means your Python interpreter
76 doesn't support the locale set by $LANG (BOGUS)
77 Continuing with ascii encoding.
78""", err)
79 self.assertEqualDiff("""\65 self.assertEqualDiff("""\
80------------------------------------------------------------66------------------------------------------------------------
81revno: 167revno: 1