From 4cbaa060316441b1d3dae58c6d0928789037213b Mon Sep 17 00:00:00 2001 From: longqianzhao Date: Fri, 7 Dec 2018 23:18:36 +0800 Subject: [PATCH] Add test cases: invalidate CIDR Add the invalid CIDR for verifing the modified method is valid. Co-Authored-By: Allain Legacy Depends-On: https://review.openstack.org/623415/ Change-Id: I0f051130e4708682104bd935b23991b534fc99d2 Related-Bug: #1805991 Story: 2004567 --- neutron/tests/unit/db/test_db_base_plugin_v2.py | 4 +++- neutron/tests/unit/extensions/test_extraroute.py | 8 ++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/neutron/tests/unit/db/test_db_base_plugin_v2.py b/neutron/tests/unit/db/test_db_base_plugin_v2.py index fb9af9f6915..7bb5923c790 100644 --- a/neutron/tests/unit/db/test_db_base_plugin_v2.py +++ b/neutron/tests/unit/db/test_db_base_plugin_v2.py @@ -3792,7 +3792,9 @@ class TestSubnetsV2(NeutronDbPluginV2TestCase): [{'nexthop': '10.0.2.20', 'destination': '100.0.0.0/8'}, {'nexthop': '10.0.2.20', - 'destination': '100.0.0.0/8'}]] + 'destination': '100.0.0.0/8'}], + [{'destination': '100.1.1.1/8', + 'nexthop': '10.0.2.20'}]] tenant_id = network['network']['tenant_id'] for hostroutes in hostroute_pools: data = {'subnet': {'network_id': network['network']['id'], diff --git a/neutron/tests/unit/extensions/test_extraroute.py b/neutron/tests/unit/extensions/test_extraroute.py index 162403d06c7..ee0bb51970e 100644 --- a/neutron/tests/unit/extensions/test_extraroute.py +++ b/neutron/tests/unit/extensions/test_extraroute.py @@ -377,6 +377,14 @@ class ExtraRouteDBTestCaseBase(object): routes}}, expected_code=exc.HTTPBadRequest.code) + routes = [{'destination': '1.1.1.1/24', + 'nexthop': '10.0.1.3'}] + + self._update('routers', r['router']['id'], + {'router': {'routes': + routes}}, + expected_code=exc.HTTPBadRequest.code) + # clean-up self._router_interface_action('remove', r['router']['id'],