diff options
| -rw-r--r-- | lib/sqlalchemy/sql/clause_compare.py | 2 | ||||
| -rw-r--r-- | test/sql/test_compare.py | 11 |
2 files changed, 12 insertions, 1 deletions
diff --git a/lib/sqlalchemy/sql/clause_compare.py b/lib/sqlalchemy/sql/clause_compare.py index 87f9fb2df..0ea981f1e 100644 --- a/lib/sqlalchemy/sql/clause_compare.py +++ b/lib/sqlalchemy/sql/clause_compare.py @@ -228,7 +228,7 @@ class StructureComparatorStrategy(object): def compare_label(self, elements, left, right, **kw): return left._type._compare_type_affinity(right._type) and ( left.name == right.name - if not isinstance(left, elements._anonymous_label) + if not isinstance(left.name, elements._anonymous_label) else isinstance(right.name, elements._anonymous_label) ) diff --git a/test/sql/test_compare.py b/test/sql/test_compare.py index 67072a640..3608851ed 100644 --- a/test/sql/test_compare.py +++ b/test/sql/test_compare.py @@ -562,6 +562,17 @@ class CompareClausesTest(fixtures.TestBase): is_false(l1.compare(l2)) + def test_compare_labels(self): + is_true(column("q").label(None).compare(column("q").label(None))) + + is_false(column("q").label("foo").compare(column("q").label(None))) + + is_false(column("q").label(None).compare(column("q").label("foo"))) + + is_false(column("q").label("foo").compare(column("q").label("bar"))) + + is_true(column("q").label("foo").compare(column("q").label("foo"))) + def test_compare_binds(self): b1 = bindparam("foo", type_=Integer()) b2 = bindparam("foo", type_=Integer()) |