Merge "Make port_range validator accept an integer"
This commit is contained in:
commit
e68ec1fbf2
|
@ -925,10 +925,7 @@ def validate_port_range_or_none(data, valid_values=None):
|
|||
"""
|
||||
if data is None:
|
||||
return
|
||||
if validate_string_or_none(data):
|
||||
msg = _("Port range must be a string.")
|
||||
LOG.debug(msg)
|
||||
return msg
|
||||
data = str(data)
|
||||
ports = data.split(':')
|
||||
if len(ports) > 2:
|
||||
msg = _("Port range must be two integers separated by a colon.")
|
||||
|
|
|
@ -1102,6 +1102,10 @@ class TestPortRangeValidation(base.BaseTestCase):
|
|||
result = validators.validate_port_range_or_none("80")
|
||||
self.assertIsNone(result)
|
||||
|
||||
def test_valid_port_integer(self):
|
||||
result = validators.validate_port_range_or_none(80)
|
||||
self.assertIsNone(result)
|
||||
|
||||
def test_valid_range(self):
|
||||
result = validators.validate_port_range_or_none("80:8888")
|
||||
self.assertIsNone(result)
|
||||
|
@ -1133,7 +1137,7 @@ class TestPortRangeValidation(base.BaseTestCase):
|
|||
|
||||
def test_range_bad_input(self):
|
||||
result = validators.validate_port_range_or_none(['a', 'b', 'c'])
|
||||
self.assertEqual(u"Port range must be a string.", result)
|
||||
self.assertEqual(u"Invalid port: ['a', 'b', 'c'].", result)
|
||||
|
||||
def test_range_colon(self):
|
||||
result = validators.validate_port_range_or_none(":")
|
||||
|
|
Loading…
Reference in New Issue