openstack/tempest
Code Issues Proposed changes

uses skip_because where appropriate instead of regular skip

Browse Source 
Change-Id: If37582fa6b4ea9ec36bf446de305a97ea525b1f9
This commit is contained in:
Giulio Fidente
2013-10-07 14:29:27 +02:00
parent a2a0c9066c
commit 386ac8f3d8
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tempest/cli/simple_read_only/test_nova.py
View File

@@ -22,6 +22,7 @@ import testtools
import tempest.cli
from tempest.openstack.common import log as logging
import tempest.test
CONF = cfg.CONF
@@ -73,7 +74,7 @@ class SimpleReadOnlyNovaClientTest(tempest.cli.ClientTestBase):
def test_admin_dns_domains(self):
self.nova('dns-domains')
@testtools.skip("Test needs parameters, Bug #1157349")
@tempest.test.skip_because(bug="1157349")
def test_admin_dns_list(self):
self.nova('dns-list')
@@ -111,7 +112,7 @@ class SimpleReadOnlyNovaClientTest(tempest.cli.ClientTestBase):
def test_admin_image_list(self):
self.nova('image-list')
@testtools.skip("Test needs parameters, Bug #1157349")
@tempest.test.skip_because(bug="1157349")
def test_admin_interface_list(self):
self.nova('interface-list')
@@ -136,7 +137,7 @@ class SimpleReadOnlyNovaClientTest(tempest.cli.ClientTestBase):
def test_admin_secgroup_list(self):
self.nova('secgroup-list')
@testtools.skip("Test needs parameters, Bug #1157349")
@tempest.test.skip_because(bug="1157349")
def test_admin_secgroup_list_rules(self):
self.nova('secgroup-list-rules')

3
tempest/scenario/test_stamp_pattern.py
View File

@@ -18,7 +18,6 @@
import time
from cinderclient import exceptions as cinder_exceptions
import testtools
from tempest.common.utils.data_utils import rand_name
from tempest import exceptions
@@ -142,7 +141,7 @@ class TestStampPattern(manager.OfficialClientTest):
got_timestamp = ssh_client.exec_command('sudo cat /mnt/timestamp')
self.assertEqual(self.timestamp, got_timestamp)
@testtools.skip("Skipped until the Bug #1205344 is resolved.")
@tempest.test.skip_because(bug="1205344")
@tempest.test.services('compute', 'network', 'volume', 'image')
def test_stamp_pattern(self):
# prepare for booting a instance