diff --git a/migrate/tests/changeset/test_constraint.py b/migrate/tests/changeset/test_constraint.py index e5717f2..42043b4 100644 --- a/migrate/tests/changeset/test_constraint.py +++ b/migrate/tests/changeset/test_constraint.py @@ -118,20 +118,15 @@ class TestConstraint(CommonTestConstraint): """Multicolumn PK constraints can be defined, created, and dropped""" self._define_pk(self.table.c.id, self.table.c.fkey) - @fixture.usedb() + @fixture.usedb(not_supported=['firebird']) def test_drop_cascade(self): """Drop constraint cascaded""" - pk = PrimaryKeyConstraint('fkey', table=self.table, name="id_pkey") pk.create() self.refresh_table() # Drop the PK constraint forcing cascade - try: - pk.drop(cascade=True) - except NotSupportedError: - if self.engine.name == 'firebird': - pass + pk.drop(cascade=True) # TODO: add real assertion if it was added diff --git a/migrate/tests/fixture/database.py b/migrate/tests/fixture/database.py index 7155b26..7739d0e 100644 --- a/migrate/tests/fixture/database.py +++ b/migrate/tests/fixture/database.py @@ -2,6 +2,7 @@ # -*- coding: utf-8 -*- import os +import logging from decorator import decorator from sqlalchemy import create_engine, Table, MetaData @@ -16,6 +17,8 @@ from migrate.tests.fixture.base import Base from migrate.tests.fixture.pathed import Pathed +log = logging.getLogger(__name__) + @Memoize def readurls(): """read URLs from config file return a list""" @@ -74,6 +77,7 @@ def usedb(supported=None, not_supported=None): @decorator def dec(f, self, *a, **kw): for url in my_urls: + log.debug("Running test with engine %s", url) try: self._setup(url) f(self, *a, **kw)