diff --git a/releasenotes/notes/deprecate-skip_unless_config-decorator-64c32d588043ab12.yaml b/releasenotes/notes/deprecate-skip_unless_config-decorator-64c32d588043ab12.yaml new file mode 100644 index 0000000000..6285ea6a2e --- /dev/null +++ b/releasenotes/notes/deprecate-skip_unless_config-decorator-64c32d588043ab12.yaml @@ -0,0 +1,5 @@ +--- +deprecations: + - The ``skip_unless_config`` and ``skip_if_config`` decorators in the + ``config`` module have been deprecated and will be removed in the Queens + dev cycle. Use the ``testtools.skipUnless`` (or a variation of) instead. diff --git a/tempest/config.py b/tempest/config.py index bd1996795c..83c5c0ec3e 100644 --- a/tempest/config.py +++ b/tempest/config.py @@ -19,6 +19,7 @@ import functools import os import tempfile +import debtcollector.removals from oslo_concurrency import lockutils from oslo_config import cfg from oslo_log import log as logging @@ -1191,6 +1192,8 @@ class TempestConfigProxy(object): CONF = TempestConfigProxy() +@debtcollector.removals.remove( + message='use testtools.skipUnless instead', removal_version='Queens') def skip_unless_config(*args): """Decorator to raise a skip if a config opt doesn't exist or is False @@ -1229,6 +1232,8 @@ def skip_unless_config(*args): return decorator +@debtcollector.removals.remove( + message='use testtools.skipIf instead', removal_version='Queens') def skip_if_config(*args): """Raise a skipException if a config exists and is True