From 4d4824772d92c1695767da4da9040bde6b7b2023 Mon Sep 17 00:00:00 2001 From: Andrew Laski Date: Tue, 31 May 2016 11:14:46 -0400 Subject: [PATCH] Remove unused _vlan_is_disabled test flag The FakeNetworkAPI was setting the _vlan_is_disabled class member to False and there was no override of that member anywhere else in the code. It was checked in one place to determine whether or not to raise a NotImplementedError. But since it was never True that was never raised. Change-Id: I830bf1e5c62ec842701fed48b6ecc9ddf7deb2ba --- nova/tests/unit/api/openstack/compute/test_networks.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/nova/tests/unit/api/openstack/compute/test_networks.py b/nova/tests/unit/api/openstack/compute/test_networks.py index 70cfbdecf5d2..2064de384a75 100644 --- a/nova/tests/unit/api/openstack/compute/test_networks.py +++ b/nova/tests/unit/api/openstack/compute/test_networks.py @@ -117,7 +117,6 @@ NEW_NETWORK = { class FakeNetworkAPI(object): _sentinel = object() - _vlan_is_disabled = False def __init__(self, skip_policy_check=False): self.networks = copy.deepcopy(FAKE_NETWORKS) @@ -153,8 +152,6 @@ class FakeNetworkAPI(object): def add_network_to_project(self, context, project_id, network_uuid=None): - if self._vlan_is_disabled: - raise NotImplementedError() if network_uuid: for network in self.networks: if network.get('project_id', None) is None: