summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Passaro <john.a.passaro@gmail.com>2016-09-19 12:23:26 -0400
committerJohn Passaro <jpassaro@zeconomy.com>2016-09-19 12:25:43 -0400
commite432ade7fee69a9e06489b18c4b072240fc9b366 (patch)
tree75436f6b65966b748e287eb5b9732cd7f6b6f5b7
parent65d8deac95b63ea5702a9ce6b5d9a6c9a6a60991 (diff)
downloadsqlalchemy-pr/308.tar.gz
exclude eq and ne from associative operatorspr/308
-rw-r--r--lib/sqlalchemy/sql/operators.py2
-rw-r--r--test/sql/test_operators.py8
2 files changed, 9 insertions, 1 deletions
diff --git a/lib/sqlalchemy/sql/operators.py b/lib/sqlalchemy/sql/operators.py
index bf470710d..142606680 100644
--- a/lib/sqlalchemy/sql/operators.py
+++ b/lib/sqlalchemy/sql/operators.py
@@ -917,7 +917,7 @@ def mirror(op):
return _mirror.get(op, op)
-_associative = _commutative.union([concat_op, and_, or_])
+_associative = _commutative.union([concat_op, and_, or_]).difference([eq, ne])
_natural_self_precedent = _associative.union([
getitem, json_getitem_op, json_path_getitem_op])
diff --git a/test/sql/test_operators.py b/test/sql/test_operators.py
index b6e80de4b..99f8a10ca 100644
--- a/test/sql/test_operators.py
+++ b/test/sql/test_operators.py
@@ -1538,6 +1538,14 @@ class OperatorAssociativityTest(fixtures.TestBase, testing.AssertsCompiledSQL):
f = column('f')
self.assert_compile(f / (f / (f - f)), "f / (f / (f - f))")
+ def test_associativity_22(self):
+ f = column('f')
+ self.assert_compile((f==f) == f, '(f = f) = f')
+
+ def test_associativity_23(self):
+ f = column('f')
+ self.assert_compile((f!=f) != f, '(f != f) != f')
+
class IsDistinctFromTest(fixtures.TestBase, testing.AssertsCompiledSQL):
__dialect__ = 'default'