Changeset 216

Timestamp:

09/25/07 18:31:21 (6 years ago)

Author:

ged

Message:

added query to the attributes triggering the setup_all call

Location:

elixir/trunk

Files:

• elixir/entity.py (modified) (1 diff)
• tests/test_autosetup.py (modified) (1 diff)

elixir/trunk/elixir/entity.py

@@ -645 +645 @@
     def __getattribute__(cls, name):
         if type.__getattribute__(cls, "_ready"):
-            #TODO: we need to add all assign_mapper methods
-            if name in ('c', 'table', 'mapper'):
+            #TODO: we might want to add all columns that will be available as
+            #attributes on the class itself (in SA 0.4). This would be a pretty
+            #rare usecase, as people will hit the query attribute before the
+            #column attributes, but still...
+            if name in ('c', 'table', 'mapper', 'query'):
                 elixir.setup_all()
         return type.__getattribute__(cls, name)

elixir/trunk/tests/test_autosetup.py

@@ -51 +51 @@
         assert isinstance(metadata.tables['person'], Table)
 
+    def test_query(self):
+        class Person(Entity):
+            has_field('name', Unicode(30))
+            using_options(tablename='person')
+
+        q = Person.query
+        assert isinstance(metadata.tables['person'], Table)
+