- Fixed a regression from 0.9.0 due to :ticket:`2736` where the

:meth:`.Query.select_from` method no longer set up the "from
entity" of the :class:`.Query` object correctly, so that
subsequent :meth:`.Query.filter_by` or :meth:`.Query.join`
calls would fail to check the appropriate "from" entity when
searching for attributes by string name.
fixes #3083

2 files changed, 47 insertions, 0 deletions

diff --git a/test/orm/test_froms.py b/test/orm/test_froms.py
index 64e98e745..7c06836d2 100644
--- a/test/orm/test_froms.py
+++ b/test/orm/test_froms.py
@@ -1930,6 +1930,52 @@ class SelectFromTest(QueryTest, AssertsCompiledSQL):
             ]
         )
 
+    def test_join_relname_from_selected_from(self):
+        User, Address = self.classes.User, self.classes.Address
+        users, addresses = self.tables.users, self.tables.addresses
+        mapper(User, users, properties=
+            {'addresses': relationship(mapper(Address, addresses),
+                backref='user')})
+
+        sess = create_session()
+
+        self.assert_compile(
+            sess.query(User).select_from(Address).join("user"),
+            "SELECT users.id AS users_id, users.name AS users_name "
+            "FROM addresses JOIN users ON users.id = addresses.user_id"
+        )
+
+    def test_filter_by_selected_from(self):
+        User, Address = self.classes.User, self.classes.Address
+        users, addresses = self.tables.users, self.tables.addresses
+        mapper(User, users, properties=
+            {'addresses': relationship(mapper(Address, addresses))})
+
+        sess = create_session()
+
+        self.assert_compile(
+            sess.query(User).select_from(Address).
+            filter_by(email_address='ed').join(User),
+            "SELECT users.id AS users_id, users.name AS users_name "
+            "FROM addresses JOIN users ON users.id = addresses.user_id "
+            "WHERE addresses.email_address = :email_address_1"
+        )
+
+    def test_join_ent_selected_from(self):
+        User, Address = self.classes.User, self.classes.Address
+        users, addresses = self.tables.users, self.tables.addresses
+        mapper(User, users, properties=
+            {'addresses': relationship(mapper(Address, addresses))})
+
+        sess = create_session()
+
+        self.assert_compile(
+            sess.query(User).select_from(Address).join(User),
+            "SELECT users.id AS users_id, users.name AS users_name "
+            "FROM addresses JOIN users ON users.id = addresses.user_id"
+        )
+
+
     def test_join(self):
         users, Address, addresses, User = (self.tables.users,
                                 self.classes.Address,
diff --git a/test/orm/test_query.py b/test/orm/test_query.py
index bac39b46b..f7cd732ee 100644
--- a/test/orm/test_query.py
+++ b/test/orm/test_query.py
@@ -1486,6 +1486,7 @@ class FilterTest(QueryTest, AssertsCompiledSQL):
         assert [User(name='chuck')] == sess.query(User).filter_by(addresses = None).all()
         assert [User(name='chuck')] == sess.query(User).filter_by(addresses = null()).all()
 
+
     def test_filter_by_tables(self):
         users = self.tables.users
         addresses = self.tables.addresses