From c2432d9d190bdc67f274b8da9296ff9ed044bef1 Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Sat, 23 Nov 2019 12:25:20 -0500 Subject: Generalize DescriptorProps.uses_objects Previously, uses_objects was specific to the SynonymAttribute; generalize it so that it defaults to False for other DescriptorProps. Immediate fix is against CompositeProperty. Fixed regression introduced in 1.3.0 related to the association proxy refactor in :ticket:`4351` that prevented :func:`.composite` attributes from working in terms of an association proxy that references them. Add test coverage for association proxies that refer to Composite attributes as endpoints. Fixes: #5000 Change-Id: Iea6fb1bd3314d861a9bc22491b0ae1e6c5e6340d --- lib/sqlalchemy/testing/fixtures.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'lib/sqlalchemy/testing') diff --git a/lib/sqlalchemy/testing/fixtures.py b/lib/sqlalchemy/testing/fixtures.py index e2237fb17..04d09f8dd 100644 --- a/lib/sqlalchemy/testing/fixtures.py +++ b/lib/sqlalchemy/testing/fixtures.py @@ -388,7 +388,10 @@ class DeclarativeMappedTest(MappedTest): cls=DeclarativeBasic, ) cls.DeclarativeBasic = _DeclBase - fn() + + # sets up cls.Basic which is helpful for things like composite + # classes + super(DeclarativeMappedTest, cls)._with_register_classes(fn) if cls.metadata.tables and cls.run_create_tables: cls.metadata.create_all(config.db) -- cgit v1.2.1