Skip to content

Commit

Permalink
PEP8.
Browse files Browse the repository at this point in the history
  • Loading branch information
dgilland committed Sep 2, 2014
1 parent 8c2f846 commit 60cd28c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 14 deletions.
1 change: 1 addition & 0 deletions alchy/query.py
Original file line number Diff line number Diff line change
Expand Up @@ -591,6 +591,7 @@ def apply_load_options(load, options):

return load


def base_columns_from_subquery(subquery):
"""Return non-aliased, base columns from subquery."""
# base_columns is a set so we need to cast to list.
Expand Down
24 changes: 10 additions & 14 deletions tests/test_query.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from sqlalchemy import orm

from alchy import query
from alchy.query import LoadOption

from .base import TestQueryBase
from .fixtures import Foo, Bar, Baz, Qux
Expand Down Expand Up @@ -221,10 +222,8 @@ def test_join_eager(self):

self.assertEqual(
str((self.db.query(Foo)
.join_eager(
'bars',
options=[query.LoadOption('contains_eager', 'bazs')])
)),
.join_eager('bars',
options=[LoadOption('contains_eager', 'bazs')]))),
str((self.db.query(Foo)
.join('bars')
.options(orm.contains_eager('bars').contains_eager('bazs')))),
Expand Down Expand Up @@ -291,7 +290,7 @@ def test_outerouterjoin_eager(self):
str((self.db.query(Foo)
.outerjoin_eager(
'bars',
options=[query.LoadOption('contains_eager', 'bazs')])
options=[LoadOption('contains_eager', 'bazs')])
)),
str((self.db.query(Foo)
.outerjoin('bars')
Expand Down Expand Up @@ -344,8 +343,7 @@ def test_joinedload(self):
self.assertEqual(
str((self.db.query(Foo)
.joinedload('bars',
options=[query.LoadOption('joinedload', 'bazs')]))
),
options=[LoadOption('joinedload', 'bazs')]))),
str((self.db.query(Foo)
.options(orm.joinedload('bars').joinedload('bazs'))))
)
Expand Down Expand Up @@ -376,8 +374,8 @@ def test_immediateload(self):
self.assertEqual(
str((self.db.query(Foo)
.immediateload('bars',
options=[query.LoadOption('immediateload',
'bazs')]))),
options=[LoadOption('immediateload',
'bazs')]))),
str((self.db.query(Foo)
.options(orm.immediateload('bars').immediateload('bazs'))))
)
Expand Down Expand Up @@ -407,8 +405,7 @@ def test_lazyload(self):

self.assertEqual(
str((self.db.query(Foo)
.lazyload('bars',
options=[query.LoadOption('lazyload', 'bazs')]))),
.lazyload('bars', options=[LoadOption('lazyload', 'bazs')]))),
str((self.db.query(Foo)
.options(orm.lazyload('bars').lazyload('bazs'))))
)
Expand Down Expand Up @@ -438,7 +435,7 @@ def test_noload(self):
self.assertEqual(
str((self.db.query(Foo)
.noload('bars',
options=[query.LoadOption('noload', 'bazs')]))),
options=[LoadOption('noload', 'bazs')]))),
str(self.db.query(Foo).options(orm.noload('bars').noload('bazs')))
)

Expand Down Expand Up @@ -469,8 +466,7 @@ def test_subqueryload(self):
str((self.db.query(Foo)
.subqueryload(
'bars',
options=[query.LoadOption('subqueryload', 'bazs')])
)),
options=[LoadOption('subqueryload', 'bazs')]))),
str((self.db.query(Foo)
.options(orm.subqueryload('bars').subqueryload('bazs'))))
)
Expand Down

0 comments on commit 60cd28c

Please sign in to comment.