Rev 6524: Fix some test issues raised by mgz. in file:///home/vila/src/bzr/cleanup/test-fixes/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/cleanup/test-fixes/

------------------------------------------------------------
revno: 6524
revision-id: v.ladeuil+lp at free.fr-20120516085421-dd91h9qm3yb2j7xb
parent: pqm at pqm.ubuntu.com-20120419102945-4fmzcqhvnpwbw61b
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: test-fixes
timestamp: Wed 2012-05-16 10:54:21 +0200
message:
  Fix some test issues raised by mgz.
-------------- next part --------------
=== modified file 'bzrlib/config.py'
--- a/bzrlib/config.py	2012-03-29 12:27:52 +0000
+++ b/bzrlib/config.py	2012-05-16 08:54:21 +0000
@@ -2398,7 +2398,8 @@
                 value = self.default()
                 if not isinstance(value, unicode):
                     raise AssertionError(
-                    'Callable default values should be unicode')
+                        "Callable default value for '%s' should be unicode"
+                        % (self.name))
             else:
                 value = self.default
         return value

=== modified file 'bzrlib/tests/test_config.py'
--- a/bzrlib/tests/test_config.py	2012-03-29 12:27:52 +0000
+++ b/bzrlib/tests/test_config.py	2012-05-16 08:54:21 +0000
@@ -2477,7 +2477,6 @@
 
     def test_help_is_set(self):
         option_help = self.registry.get_help(self.option_name)
-        self.assertNotEquals(None, option_help)
         # Come on, think about the user, he really wants to know what the
         # option is about
         self.assertIsNot(None, option_help)
@@ -3584,6 +3583,7 @@
         return config.MemoryStack(content)
 
     def test_override_value_from_env(self):
+        self.overrideEnv('FOO', None)
         self.registry.register(
             config.Option('foo', default='bar', override_from_env=['FOO']))
         self.overrideEnv('FOO', 'quux')
@@ -3592,6 +3592,9 @@
         self.assertEquals('quux', conf.get('foo'))
 
     def test_first_override_value_from_env_wins(self):
+        self.overrideEnv('NO_VALUE', None)
+        self.overrideEnv('FOO', None)
+        self.overrideEnv('BAZ', None)
         self.registry.register(
             config.Option('foo', default='bar',
                           override_from_env=['NO_VALUE', 'FOO', 'BAZ']))