]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
- use is_() here to more clearly show what's going wrong
authorMike Bayer <mike_mp@zzzcomputing.com>
Mon, 19 Oct 2015 22:48:58 +0000 (18:48 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 19 Oct 2015 22:48:58 +0000 (18:48 -0400)
on occasional pypy test runs

test/orm/inheritance/test_poly_persistence.py

index c6a54c0b590a336129b3e5af5389751cade4694a..361377de8eaeed3590e2bc8694bb117d145e8979 100644 (file)
@@ -1,6 +1,6 @@
 """tests basic polymorphic mapper loading/saving, minimal relationships"""
 
-from sqlalchemy.testing import eq_, assert_raises, assert_raises_message
+from sqlalchemy.testing import eq_, is_, assert_raises, assert_raises_message
 from sqlalchemy import *
 from sqlalchemy.orm import *
 from sqlalchemy.orm import exc as orm_exc
@@ -271,18 +271,30 @@ def _generate_round_trip_test(include_base, lazy_relationship,
         # into the "person_join" conversion.
         palias = people.alias("palias")
         dilbert = session.query(Person).get(dilbert.person_id)
-        assert dilbert is session.query(Person).filter(
-                            (palias.c.name=='dilbert') & \
-                            (palias.c.person_id==Person.person_id)).first()
-        assert dilbert is session.query(Engineer).filter(
-                            (palias.c.name=='dilbert') & \
-                            (palias.c.person_id==Person.person_id)).first()
-        assert dilbert is session.query(Person).filter(
-                            (Engineer.engineer_name=="engineer1") & \
-                            (engineers.c.person_id==people.c.person_id)
-                        ).first()
-        assert dilbert is session.query(Engineer).\
-                        filter(Engineer.engineer_name=="engineer1")[0]
+        is_(
+            dilbert,
+            session.query(Person).filter(
+                (palias.c.name == 'dilbert') &
+                (palias.c.person_id == Person.person_id)).first()
+        )
+        is_(
+            dilbert,
+            session.query(Engineer).filter(
+                (palias.c.name == 'dilbert') &
+                (palias.c.person_id == Person.person_id)).first()
+        )
+        is_(
+            dilbert,
+            session.query(Person).filter(
+                (Engineer.engineer_name == "engineer1") &
+                (engineers.c.person_id == people.c.person_id)
+            ).first()
+        )
+        is_(
+            dilbert,
+            session.query(Engineer).
+            filter(Engineer.engineer_name == "engineer1")[0]
+        )
 
         session.flush()
         session.expunge_all()