- [bug] Fixed the error formatting raised when

a tuple is inadvertently passed to session.query()
[ticket:2297].

diff --git a/CHANGES b/CHANGES
index 89b07c4453f704720f990252c12cbb994533d663..87036000a74c62e431b72ebe1c6bd6dc6f0990c2 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -80,6 +80,10 @@ CHANGES
      a distinct entity when producing certain
      kinds of joined-inh joins.  [ticket:2316]
 
+   - [bug] Fixed the error formatting raised when 
+     a tuple is inadvertently passed to session.query()
+     [ticket:2297].  Also in 0.6.9.
+
 - sql
    - [bug] related to [ticket:2316], made some 
      adjustments to the change from [ticket:2261]

diff --git a/lib/sqlalchemy/orm/query.py b/lib/sqlalchemy/orm/query.py
index a875ad46d06dc667860e32d5e30e9e09e99ba40d..9c7ef7b8eb3f593b68a3c7e16db2706c2cd68342 100644 (file)
--- a/lib/sqlalchemy/orm/query.py
+++ b/lib/sqlalchemy/orm/query.py
@@ -3052,7 +3052,7 @@ class _ColumnEntity(_QueryEntity):
         if not isinstance(column, sql.ColumnElement):
             raise sa_exc.InvalidRequestError(
                 "SQL expression, column, or mapped entity "
-                "expected - got '%r'" % column
+                "expected - got '%r'" % (column, )
             )
 
         # If the Column is unnamed, give it a

diff --git a/test/orm/test_query.py b/test/orm/test_query.py
index 25ab1a29d6bf3a002fa1ba6237ab932348bab59a..9ada53bfcd94ed9422c66352285c5eee543e519b 100644 (file)
--- a/test/orm/test_query.py
+++ b/test/orm/test_query.py
@@ -432,6 +432,13 @@ class InvalidGenerationsTest(QueryTest, AssertsCompiledSQL):
         q = s.query(User)
         assert_raises(sa_exc.InvalidRequestError, q.add_column, object())
 
+    def test_invalid_column_tuple(self):
+        User = self.classes.User
+
+        s = create_session()
+        q = s.query(User)
+        assert_raises(sa_exc.InvalidRequestError, q.add_column, (1, 1))
+
     def test_distinct(self):
         """test that a distinct() call is not valid before 'clauseelement' conditions."""