From 481c314d6a54965fe6e0972995c6ad9afa86a908 Mon Sep 17 00:00:00 2001 From: "Mauro S. M. Rodrigues" Date: Fri, 15 Feb 2013 08:19:00 -0500 Subject: Standarize ip validation along the code Details: - Use the nova/utils.py validations along the code - refactor ipv4 validator to looks like ipv6 validator - interprets ip validations exceptions as Invalid IP values Change-Id: Ia506c00510a066e167d3dcd9dd371a371129a9dd --- nova/tests/api/test_validator.py | 8 -------- nova/tests/test_utils.py | 4 ++++ 2 files changed, 4 insertions(+), 8 deletions(-) (limited to 'nova/tests') diff --git a/nova/tests/api/test_validator.py b/nova/tests/api/test_validator.py index 132e67e95..93a7deb66 100644 --- a/nova/tests/api/test_validator.py +++ b/nova/tests/api/test_validator.py @@ -73,14 +73,6 @@ class ValidatorTestCase(test.TestCase): self.assertFalse(validator.validate_ec2_id(1234)) self.assertTrue(validator.validate_ec2_id('i-284f3a41')) - def test_validate_ipv4(self): - self.assertTrue(validator.validate_ipv4('4.2.2.4')) - self.assertFalse(validator.validate_ipv4('foobar')) - self.assertFalse( - validator.validate_ipv4('2001:5a8:4:68e0:e6ce:8fff:fe27:d116')) - self.assertFalse(validator.validate_ipv4(123)) - self.assertFalse(validator.validate_ipv4('')) - def test_validate_url_path(self): self.assertTrue(validator.validate_url_path('/path/to/file')) self.assertFalse(validator.validate_url_path('path/to/file')) diff --git a/nova/tests/test_utils.py b/nova/tests/test_utils.py index 2c2c58db9..e71090ceb 100644 --- a/nova/tests/test_utils.py +++ b/nova/tests/test_utils.py @@ -515,6 +515,8 @@ class GenericUtilsTestCase(test.TestCase): self.assertTrue(utils.is_valid_ipv4('127.0.0.1')) self.assertFalse(utils.is_valid_ipv4('::1')) self.assertFalse(utils.is_valid_ipv4('bacon')) + self.assertFalse(utils.is_valid_ipv4("")) + self.assertFalse(utils.is_valid_ipv4(10)) def test_is_valid_ipv6(self): self.assertTrue(utils.is_valid_ipv6("::1")) @@ -524,6 +526,8 @@ class GenericUtilsTestCase(test.TestCase): "0000:0000:0000:0000:0000:0000:0000:0001")) self.assertFalse(utils.is_valid_ipv6("foo")) self.assertFalse(utils.is_valid_ipv6("127.0.0.1")) + self.assertFalse(utils.is_valid_ipv6("")) + self.assertFalse(utils.is_valid_ipv6(10)) def test_is_valid_ipv6_cidr(self): self.assertTrue(utils.is_valid_ipv6_cidr("2600::/64")) -- cgit