tyler.durden
/
compose
mirror of https://github.com/docker/compose.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1852 from mnowster/fix-testsuite-post-some-merges 

Fix ports validation
This commit is contained in:
Aanand Prasad 2015-08-12 15:31:59 +01:00
parent eea3b1cd6f 5e2ecff8a1
commit 2134800cad
2 changed files with 16 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
compose/config/schema.json
View File

@ -75,22 +75,20 @@
"pid": {"type": "string"}, "pid": {"type": "string"},
"ports": { "ports": {
"oneOf": [ "type": "array",
{ "items": {
"type": "array", "oneOf": [
"items": {"type": "string"}, {
"uniqueItems": true, "type": "string",
"format": "ports" "format": "ports"
}, },
{ {
"type": "string", "type": "number",
"format": "ports" "format": "ports"
}, }
{ ]
"type": "number", },
"format": "ports" "uniqueItems": true
}
]
}, },
"privileged": {"type": "string"}, "privileged": {"type": "string"},

4
tests/unit/config_test.py
View File

@ -77,7 +77,7 @@ class ConfigTest(unittest.TestCase):
def test_config_invalid_ports_format_validation(self): def test_config_invalid_ports_format_validation(self):
expected_error_msg = "Service 'web' configuration key 'ports' contains an invalid type" expected_error_msg = "Service 'web' configuration key 'ports' contains an invalid type"
with self.assertRaisesRegexp(ConfigurationError, expected_error_msg): with self.assertRaisesRegexp(ConfigurationError, expected_error_msg):
for invalid_ports in [{"1": "8000"}, False, 0]: for invalid_ports in [{"1": "8000"}, False, 0, "8000", 8000, ["8000", "8000"]]:
config.load( config.load(
config.ConfigDetails( config.ConfigDetails(
{'web': {'image': 'busybox', 'ports': invalid_ports}}, {'web': {'image': 'busybox', 'ports': invalid_ports}},
@ -87,7 +87,7 @@ class ConfigTest(unittest.TestCase):
) )
def test_config_valid_ports_format_validation(self): def test_config_valid_ports_format_validation(self):
valid_ports = [["8000", "9000"], ["8000/8050"], ["8000"], "8000", 8000] valid_ports = [["8000", "9000"], ["8000/8050"], ["8000"], [8000], ["49153-49154:3002-3003"]]
for ports in valid_ports: for ports in valid_ports:
config.load( config.load(
config.ConfigDetails( config.ConfigDetails(