diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2016-04-19 16:11:23 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2016-04-19 18:17:45 -0400 |
| commit | 23825a2591760285d5d42b05f81248f4f7a23631 (patch) | |
| tree | cf50a502e2d688dd3e1906690cd1a4891a43820a /test/ext/declarative | |
| parent | 56dafa6c0dc1ebb7728a2120cce14f8227b2a97e (diff) | |
| download | sqlalchemy-23825a2591760285d5d42b05f81248f4f7a23631.tar.gz | |
Deprecate Mapper.order_by
This is an old parameter no longer relevant to how SQLAlchemy
works, once the Query object was introduced. By deprecating it
we establish that we aren't supporting non-working use cases
and that we encourage applications to move off of the use of this
parameter.
Fixes: #3394
Change-Id: I25b9a38142a1537bbcb27d3e8b66a8b265140072
Diffstat (limited to 'test/ext/declarative')
| -rw-r--r-- | test/ext/declarative/test_reflection.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/ext/declarative/test_reflection.py b/test/ext/declarative/test_reflection.py index c7f7bc05d..84b675b3d 100644 --- a/test/ext/declarative/test_reflection.py +++ b/test/ext/declarative/test_reflection.py @@ -281,7 +281,7 @@ class DeferredReflectionTest(DeferredReflectBase): @decl.declared_attr def __mapper_args__(cls): return { - "order_by": cls.__table__.c.name + "primary_key": cls.__table__.c.id } decl.DeferredReflection.prepare(testing.db) @@ -294,7 +294,7 @@ class DeferredReflectionTest(DeferredReflectBase): ]) sess.commit() eq_( - sess.query(User).all(), + sess.query(User).order_by(User.name).all(), [ User(name='A'), User(name='C'), |