Removing deprecated module neutron.api.versions

'Versions.factory()' is deprecated in version 'Queens'
and was marked to be removed 'Rocky'.
This patch removes it and switches to use
neutron.pecan_wsgi.app.versions_factory instead.

In addition, neutron.api.versions is removed
because this deprecated module isn't used anywhere.

Change-Id: I08d1725a973c734b55771918915428377fb7d3b4
changes/69/556769/7
Vu Cong Tuan 5 years ago
parent c3f2b89601
commit 7988e32202
  1. 27
      neutron/api/versions.py
  2. 10
      neutron/tests/unit/pecan_wsgi/test_app.py

@ -1,27 +0,0 @@
# Copyright 2011 Citrix Systems.
# All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
from debtcollector import removals
from neutron.pecan_wsgi import app as pecan_app
class Versions(object):
@removals.remove(version="Queens", removal_version="Rocky",
message="Use neutron.pecan_wsgi.app:versions_factory")
@classmethod
def factory(cls, global_config, **local_config):
return pecan_app.versions_factory(global_config, **local_config)

@ -12,15 +12,13 @@
import mock
from neutron.api import versions
from neutron import pecan_wsgi
from neutron.tests import base
@mock.patch('neutron.api.versions.Versions.__init__', return_value=None)
@mock.patch('neutron.pecan_wsgi.app.versions_factory')
class TestVersions(base.BaseTestCase):
class TestVersionsFactory(base.BaseTestCase):
def test_pecan_factory(self, pecan_mock, legacy_mock):
versions.Versions.factory({})
def test_versions_factory(self, pecan_mock):
pecan_wsgi.app.versions_factory({})
pecan_mock.assert_called_once_with({})
legacy_mock.assert_not_called()
Loading…
Cancel
Save