Merge "Migrate cli tests to resource_* fixtures"
This commit is contained in:
@@ -94,11 +94,11 @@ def min_client_version(*args, **kwargs):
|
||||
|
||||
class ClientTestBase(tempest.test.BaseTestCase):
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.cli.enabled:
|
||||
msg = "cli testing disabled"
|
||||
raise cls.skipException(msg)
|
||||
super(ClientTestBase, cls).setUpClass()
|
||||
super(ClientTestBase, cls).resource_setup()
|
||||
|
||||
def __init__(self, *args, **kwargs):
|
||||
self.parser = tempest.cli.output_parser
|
||||
|
||||
@@ -41,11 +41,11 @@ class SimpleReadOnlyNovaClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.nova:
|
||||
msg = ("%s skipped as Nova is not available" % cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyNovaClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyNovaClientTest, cls).resource_setup()
|
||||
|
||||
def test_admin_fake_action(self):
|
||||
self.assertRaises(exceptions.CommandFailed,
|
||||
|
||||
@@ -36,7 +36,7 @@ class SimpleReadOnlyNovaManageTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.nova:
|
||||
msg = ("%s skipped as Nova is not available" % cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
@@ -44,7 +44,7 @@ class SimpleReadOnlyNovaManageTest(cli.ClientTestBase):
|
||||
msg = ("%s skipped as *-manage commands not available"
|
||||
% cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyNovaManageTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyNovaManageTest, cls).resource_setup()
|
||||
|
||||
def test_admin_fake_action(self):
|
||||
self.assertRaises(exceptions.CommandFailed,
|
||||
|
||||
@@ -34,11 +34,11 @@ class SimpleReadOnlySaharaClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.sahara:
|
||||
msg = "Skipping all Sahara cli tests because it is not available"
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlySaharaClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlySaharaClientTest, cls).resource_setup()
|
||||
|
||||
@test.attr(type='negative')
|
||||
def test_sahara_fake_action(self):
|
||||
|
||||
@@ -34,11 +34,11 @@ class SimpleReadOnlyGlanceClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.glance:
|
||||
msg = ("%s skipped as Glance is not available" % cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyGlanceClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyGlanceClientTest, cls).resource_setup()
|
||||
|
||||
def test_glance_fake_action(self):
|
||||
self.assertRaises(exceptions.CommandFailed,
|
||||
|
||||
@@ -35,11 +35,11 @@ class SimpleReadOnlyNeutronClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if (not CONF.service_available.neutron):
|
||||
msg = "Skipping all Neutron cli tests because it is not available"
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyNeutronClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyNeutronClientTest, cls).resource_setup()
|
||||
|
||||
@test.attr(type='smoke')
|
||||
def test_neutron_fake_action(self):
|
||||
|
||||
@@ -31,11 +31,11 @@ class SimpleReadOnlySwiftClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.swift:
|
||||
msg = ("%s skipped as Swift is not available" % cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlySwiftClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlySwiftClientTest, cls).resource_setup()
|
||||
|
||||
def test_swift_fake_action(self):
|
||||
self.assertRaises(exceptions.CommandFailed,
|
||||
|
||||
@@ -32,12 +32,12 @@ class SimpleReadOnlyHeatClientTest(tempest.cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if (not CONF.service_available.heat):
|
||||
msg = ("Skipping all Heat cli tests because it is "
|
||||
"not available")
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyHeatClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyHeatClientTest, cls).resource_setup()
|
||||
cls.heat_template_path = os.path.join(os.path.dirname(
|
||||
os.path.dirname(os.path.realpath(__file__))),
|
||||
'heat_templates/heat_minimal.yaml')
|
||||
|
||||
@@ -32,12 +32,12 @@ class SimpleReadOnlyCeilometerClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if (not CONF.service_available.ceilometer):
|
||||
msg = ("Skipping all Ceilometer cli tests because it is "
|
||||
"not available")
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyCeilometerClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyCeilometerClientTest, cls).resource_setup()
|
||||
|
||||
def test_ceilometer_meter_list(self):
|
||||
self.ceilometer('meter-list')
|
||||
|
||||
@@ -35,11 +35,11 @@ class SimpleReadOnlyCinderClientTest(cli.ClientTestBase):
|
||||
"""
|
||||
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
def resource_setup(cls):
|
||||
if not CONF.service_available.cinder:
|
||||
msg = ("%s skipped as Cinder is not available" % cls.__name__)
|
||||
raise cls.skipException(msg)
|
||||
super(SimpleReadOnlyCinderClientTest, cls).setUpClass()
|
||||
super(SimpleReadOnlyCinderClientTest, cls).resource_setup()
|
||||
|
||||
def test_cinder_fake_action(self):
|
||||
self.assertRaises(exceptions.CommandFailed,
|
||||
|
||||
Reference in New Issue
Block a user