Changeset 92

Timestamp:

03/27/07 09:52:11 (6 years ago)

Author:

ged

Message:

I guess I really wasn't awake the day I commited r89... So now I actually
commit the fix to the inheritance unit test

Files:

• elixir/trunk/tests/test_inherit.py (modified) (2 diffs)

elixir/trunk/tests/test_inherit.py

@@ -48 +48 @@
     def test_singletable_inheritance(self):
         homer = PersonExtended(firstname="Homer", surname="Simpson", age=36)
-        lisa = PersonExtended(firstname="Lisa", surname="Simpson", 
-                              parent=homer)
+        # lisa needs to be a Person object, not a PersonExtended object because
+        # the sister relationship points to a Person, not a PersonExtendend, so
+        # bart's sister must be a Person. This is to comply with SQLAlchemy's
+        # policy to prevent loading relationships with unintended types, unless 
+        # explicitly enabled (enable_typechecks=False).
+        lisa = Person(firstname="Lisa", surname="Simpson")
         bart = PersonExtended(firstname="Bart", surname="Simpson", 
                               parent=homer, sister=lisa)
@@ -60 +64 @@
         assert p.sister.name == 'Lisa Simpson'
         assert p.parent.age == 36
-        # This doesn't work since the sister relation points to a Person 
-        # object, not a PersonExtended one.
-#        assert p.sister.parent == p.parent
 
         for p in Person.select():