new test decorator to skip test if specific ext enabled

[alex kang]

In some deployments enabled features are required to turn-off some of their functions.
For example on provisioning a subnet, we can specify allocation_pools, dns_nameservers and host_routes, ..., etc. In this propose we would like to skip tests testing host_routes if nsxv-router-type extension is enabled.

Propose to add skip_if_ext_enabled() to test.py and TestSkipIfExtEnabledDecorator to tests/test_decorators.py.

example usage of this decorator:

@test.skip_if_ext_enabled(extension='nsxv-router-type', service='network')

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/215819

[Jordan Pittier (jordan-pittier)]

Instead of adding a new decorator, could you just decorate the existing tests with "@testtools.skipIf(is_extension_enabled(...))" ?

[alex kang (akang)]

I think it is better to view from Tempest test perspective, not from the testtools which is the parent class -- should focus on test logic not how testtools can do which should be provided from the viewpoint of Tempest.test.

2nd semantically speaking it is more straight forward to say skip this test if:
   service='network' and extension="nsxv-router-type"

[OpenStack Infra (hudson-openstack)]

Change abandoned by Alex Kang (<email address hidden>) on branch: master
Review: https://review.openstack.org/215819

[Ken'ichi Ohmichi (oomichi)]

This seems that we have gotten agreement with using @testtools.skipIf(is_extension_enabled(...)) instead on the review.
So it is not necessary to keep this report now.