config: Add parse_override test
This commit is contained in:
parent
51afbe19e1
commit
c5f8e1da19
@ -137,4 +137,4 @@ def parse_override(override):
|
||||
"""Parse section/key=value override."""
|
||||
section, remainder = override.split('/', 1)
|
||||
key, value = remainder.split('=', 1)
|
||||
return (section, key, value)
|
||||
return (section.strip(), key.strip(), value.strip())
|
||||
|
||||
@ -47,3 +47,22 @@ class LoadConfigTest(unittest.TestCase):
|
||||
expected = {'foo': {'bar': 'baz'}, 'foo2': {'bar': 'baz'}}
|
||||
result = config._load([file1, file2], [], [])
|
||||
self.assertEqual(expected, result)
|
||||
|
||||
|
||||
class ParseOverrideTest(unittest.TestCase):
|
||||
def test_valid_override(self):
|
||||
expected = ('section', 'key', 'value')
|
||||
self.assertEqual(expected, config.parse_override('section/key=value'))
|
||||
self.assertEqual(expected, config.parse_override('section/key=value '))
|
||||
self.assertEqual(expected, config.parse_override('section/key =value'))
|
||||
self.assertEqual(expected, config.parse_override('section /key=value'))
|
||||
|
||||
def test_empty_override(self):
|
||||
expected = ('section', 'key', '')
|
||||
self.assertEqual(expected, config.parse_override('section/key='))
|
||||
self.assertEqual(expected, config.parse_override('section/key= '))
|
||||
|
||||
def test_invalid_override(self):
|
||||
self.assertRaises(ValueError, config.parse_override, 'section/key')
|
||||
self.assertRaises(ValueError, config.parse_override, 'section=')
|
||||
self.assertRaises(ValueError, config.parse_override, 'section')
|
||||
|
||||
Loading…
Reference in New Issue
Block a user