Merge "Add test for BoolOpt with no default"

This commit is contained in:
Jenkins 2016-02-02 16:51:46 +00:00 committed by Gerrit Code Review
commit ea53c55864
1 changed files with 12 additions and 1 deletions

View File

@ -878,7 +878,18 @@ class ConfigFileOptsTestCase(BaseTestCase):
self.assertTrue(hasattr(self.conf, 'foo'))
self.assertEqual(self.conf.foo, 'bar')
def test_conf_file_bool_default(self):
def test_conf_file_bool_default_none(self):
self.conf.register_opt(cfg.BoolOpt('foo'))
paths = self.create_tempfiles([('test',
'[DEFAULT]\n')])
self.conf(['--config-file', paths[0]])
self.assertTrue(hasattr(self.conf, 'foo'))
self.assertIsNone(self.conf.foo)
def test_conf_file_bool_default_false(self):
self.conf.register_opt(cfg.BoolOpt('foo', default=False))
paths = self.create_tempfiles([('test',