tree b3e9dfa790e57723c22fbc44b4ddbe3e5de2049b
parent 6827cd33c48da6c87dd864c66d87f47fcc6dc11c
author Jordan Pittier <jordan.pittier@scality.com> 1487089831 +0100
committer Jordan Pittier <jordan.pittier@scality.com> 1487617114 +0000

Deprecate the skip_unless_attr decorator.

This decorator was used at only one place (ListServerFiltersTestJSON)
and those tests are skipped in the Gate anyway (SKIPPED: Only one image found)
These tests were poorly written anyway, the resource_setup() method
goes against are principles in [1]: Using discovery for skipping tests
is generally discouraged.

This decorator encourages bad practise, like the usage of class variables.
We should use the generic and well known testtools.SkipIf/Unless decorator
instead.

[1] : https://github.com/openstack/tempest/blob/master/HACKING.rst#skipping-tests

Change-Id: I639f324d5b38cd154b3ecdb89b56ff2ee279c4ff