openstack/tempest
Code Issues Proposed changes

Use skip_checks in test_snapshot_pattern

Browse Source 
In TestSnapshotPattern, Snapshotting feature is necessary, it is
the same to add test cases in the future, so use skip_checks
instead of skipUnless.

Change-Id: Ic48f4f2eb9cbb93c08df561e70bc9db0cc2db510
This commit is contained in:
guo yunxian 2016-07-25 11:19:36 +08:00
parent 99debda8e9
commit 13b456adf3
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tempest/scenario/test_snapshot_pattern.py
View File

@ -13,8 +13,6 @@
# License for the specific language governing permissions and limitations
# under the License.
import testtools
from tempest import config
from tempest.scenario import manager
from tempest import test
@ -33,9 +31,13 @@ class TestSnapshotPattern(manager.ScenarioTest):
"""
@classmethod
def skip_checks(cls):
super(TestSnapshotPattern, cls).skip_checks()
if not CONF.compute_feature_enabled.snapshot:
raise cls.skipException("Snapshotting is not available.")
@test.idempotent_id('608e604b-1d63-4a82-8e3e-91bc665c90b4')
@testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
'Snapshotting is not available.')
@test.services('compute', 'network', 'image')
def test_snapshot_pattern(self):
# prepare for booting an instance