diff options
| -rw-r--r-- | openstack/common/cfg.py | 2 | ||||
| -rw-r--r-- | tests/unit/test_cfg.py | 4 |
2 files changed, 5 insertions, 1 deletions
diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py index e967c61..c8a15d7 100644 --- a/openstack/common/cfg.py +++ b/openstack/common/cfg.py @@ -745,7 +745,7 @@ class ListOpt(Opt): """ def __call__(self, parser, namespace, values, option_string=None): if values is not None: - values = values.split(',') + values = [a.strip() for a in values.split(',')] setattr(namespace, self.dest, values) def _get_from_config_parser(self, cparser, section): diff --git a/tests/unit/test_cfg.py b/tests/unit/test_cfg.py index 384a57a..bd9e914 100644 --- a/tests/unit/test_cfg.py +++ b/tests/unit/test_cfg.py @@ -219,6 +219,10 @@ class CliOptsTestCase(BaseTestCase): self._do_cli_test(ListOpt, None, ['--foo', 'blaa,bar'], ['blaa', 'bar']) + def test_list_arg_with_spaces(self): + self._do_cli_test(ListOpt, None, + ['--foo', 'blaa ,bar'], ['blaa', 'bar']) + def test_list_arg_deprecated(self): self._do_cli_test(ListOpt, None, ['--oldfoo', 'blaa,bar'], ['blaa', 'bar']) |