diff options
author | Federico Caselli <cfederico87@gmail.com> | 2020-02-15 12:34:37 +0100 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2020-03-25 10:49:27 -0400 |
commit | cdbe3f84ba4656fd54205212b5adcd5ad9c8e8d2 (patch) | |
tree | ba5cc03910274939dbb19b123df9236bceb44ecd /lib/sqlalchemy/testing/assertsql.py | |
parent | 7ab1a62d886a9fe40eb368bbbe73b6436b9cbb4b (diff) | |
download | sqlalchemy-cdbe3f84ba4656fd54205212b5adcd5ad9c8e8d2.tar.gz |
Deprecate add of columns in order by with distinct
Deprecate automatic addition of order by column in a query with a distinct
Fixes: #5134
Change-Id: I467a39379c496be7e84a05f11ba9f8ca2bcc6e32
Diffstat (limited to 'lib/sqlalchemy/testing/assertsql.py')
-rw-r--r-- | lib/sqlalchemy/testing/assertsql.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/lib/sqlalchemy/testing/assertsql.py b/lib/sqlalchemy/testing/assertsql.py index f0da69400..8876c2304 100644 --- a/lib/sqlalchemy/testing/assertsql.py +++ b/lib/sqlalchemy/testing/assertsql.py @@ -184,8 +184,8 @@ class CompiledSQL(SQLMatchRule): def _failure_message(self, expected_params): return ( - "Testing for compiled statement %r partial params %s, " - "received %%(received_statement)r with params " + "Testing for compiled statement\n%r partial params %s, " + "received\n%%(received_statement)r with params " "%%(received_parameters)r" % ( self.statement.replace("%", "%%"), @@ -343,6 +343,9 @@ class SQLExecuteObserved(object): self.parameters = _distill_params(multiparams, params) self.statements = [] + def __repr__(self): + return str(self.statements) + class SQLCursorExecuteObserved( collections.namedtuple( @@ -373,7 +376,7 @@ class SQLAsserter(object): elif rule.errormessage: assert False, rule.errormessage if observed: - assert False, "Additional SQL statements remain" + assert False, "Additional SQL statements remain:\n%s" % observed elif not rule.is_consumed: rule.no_more_statements() |