diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2016-06-13 15:18:13 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2016-06-14 11:48:04 -0400 |
| commit | f38f890849700ee1bf719a31275260e2da455bc3 (patch) | |
| tree | e5d8a958d853f05081e57045baa8c9806ad8f27a /test/orm/inheritance/test_basic.py | |
| parent | 7189d0bc82598c2d6dcbb55b054837416db2ee7d (diff) | |
| download | sqlalchemy-f38f890849700ee1bf719a31275260e2da455bc3.tar.gz | |
Deprecate FromClause.count()
count() here is misleading in that it not only
counts from an arbitrary column in the table, it also
does not make accommodations for DISTINCT, JOIN, etc.
as the ORM-level function does. Core should not be
attempting to provide a function like this.
Change-Id: I9916fc51ef744389a92c54660ab08e9695b8afc2
Fixes: #3724
Diffstat (limited to 'test/orm/inheritance/test_basic.py')
| -rw-r--r-- | test/orm/inheritance/test_basic.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/orm/inheritance/test_basic.py b/test/orm/inheritance/test_basic.py index 3717fafa0..42959a6e3 100644 --- a/test/orm/inheritance/test_basic.py +++ b/test/orm/inheritance/test_basic.py @@ -1104,7 +1104,7 @@ class FlushTest(fixtures.MappedTest): sess.add(a) sess.flush() - assert user_roles.count().scalar() == 1 + eq_(select([func.count('*')]).select_from(user_roles).scalar(), 1) def test_two(self): admins, users, roles, user_roles = (self.tables.admins, @@ -1146,7 +1146,7 @@ class FlushTest(fixtures.MappedTest): a.password = 'sadmin' sess.flush() - assert user_roles.count().scalar() == 1 + eq_(select([func.count('*')]).select_from(user_roles).scalar(), 1) class PassiveDeletesTest(fixtures.MappedTest): |