Skip to content

Commit

Permalink
Fix undef function call in test_migrations for py26
Browse files Browse the repository at this point in the history
unittest does not have this function until later. This uses the already
imported cinder.test.skip_unless to accomplish the same result without
outputted error.

Change-Id: Id20add5a0d41e5d8cf6f8bc6aa0e547e5e386dab
  • Loading branch information
Thingee committed Feb 18, 2013
1 parent d18519d commit e613426
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions cinder/tests/test_migrations.py
Original file line number Diff line number Diff line change
Expand Up @@ -276,10 +276,9 @@ def test_postgresql_connect_fail(self):
if _is_backend_avail('postgres', user="openstack_cifail"):
self.fail("Shouldn't have connected")

@test.skip_unless(_is_backend_avail('postgres'),
"postgresql not available")
def test_postgresql_opportunistically(self):
# Test postgresql database migration walk
if not _is_backend_avail('postgres'):
self.skipTest("postgresql not available")
# add this to the global lists to make reset work with it, it's removed
# automatically in tearDown so no need to clean it up here.
connect_string = _get_connect_string("postgres")
Expand Down

0 comments on commit e613426

Please sign in to comment.