diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2014-02-28 18:50:18 -0500 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2014-02-28 18:50:18 -0500 |
| commit | 43f95d3e5e8faec681141212d1e908b06a47e337 (patch) | |
| tree | da277326b72d4de49e9535d02f42b0e9ca545883 /lib/sqlalchemy/testing/suite/test_reflection.py | |
| parent | e8d048f10bb5f5239d1fde543c084b17e81a017f (diff) | |
| download | sqlalchemy-43f95d3e5e8faec681141212d1e908b06a47e337.tar.gz | |
- dont create a self-ref FK if test suite marked as not supporting self ref
- break out reflection of views to support view column reflection separately
from view definition
Diffstat (limited to 'lib/sqlalchemy/testing/suite/test_reflection.py')
| -rw-r--r-- | lib/sqlalchemy/testing/suite/test_reflection.py | 28 |
1 files changed, 15 insertions, 13 deletions
diff --git a/lib/sqlalchemy/testing/suite/test_reflection.py b/lib/sqlalchemy/testing/suite/test_reflection.py index 9f737bc64..f6e82c063 100644 --- a/lib/sqlalchemy/testing/suite/test_reflection.py +++ b/lib/sqlalchemy/testing/suite/test_reflection.py @@ -89,7 +89,7 @@ class ComponentReflectionTest(fixtures.TablesTest): if testing.requires.index_reflection.enabled: cls.define_index(metadata, users) - if testing.requires.view_reflection.enabled: + if testing.requires.view_column_reflection.enabled: cls.define_views(metadata, schema) @classmethod @@ -243,11 +243,11 @@ class ComponentReflectionTest(fixtures.TablesTest): def test_get_columns_with_schema(self): self._test_get_columns(schema='test_schema') - @testing.requires.view_reflection + @testing.requires.view_column_reflection def test_get_view_columns(self): self._test_get_columns(table_type='view') - @testing.requires.view_reflection + @testing.requires.view_column_reflection @testing.requires.schemas def test_get_view_columns_with_schema(self): self._test_get_columns(schema='test_schema', table_type='view') @@ -300,18 +300,20 @@ class ComponentReflectionTest(fixtures.TablesTest): insp = inspect(meta.bind) expected_schema = schema # users - users_fkeys = insp.get_foreign_keys(users.name, - schema=schema) - fkey1 = users_fkeys[0] - - with testing.requires.named_constraints.fail_if(): - self.assert_(fkey1['name'] is not None) - eq_(fkey1['referred_schema'], expected_schema) - eq_(fkey1['referred_table'], users.name) - eq_(fkey1['referred_columns'], ['user_id', ]) if testing.requires.self_referential_foreign_keys.enabled: - eq_(fkey1['constrained_columns'], ['parent_user_id']) + users_fkeys = insp.get_foreign_keys(users.name, + schema=schema) + fkey1 = users_fkeys[0] + + with testing.requires.named_constraints.fail_if(): + self.assert_(fkey1['name'] is not None) + + eq_(fkey1['referred_schema'], expected_schema) + eq_(fkey1['referred_table'], users.name) + eq_(fkey1['referred_columns'], ['user_id', ]) + if testing.requires.self_referential_foreign_keys.enabled: + eq_(fkey1['constrained_columns'], ['parent_user_id']) #addresses addr_fkeys = insp.get_foreign_keys(addresses.name, |