- mapping to a select() construct now requires that you

make an alias() out of it distinctly.   This to eliminate
confusion over such issues as [ticket:1542]

1 files changed, 7 insertions, 3 deletions

diff --git a/test/orm/test_selectable.py b/test/orm/test_selectable.py
index bfa400895..e46d8bbc8 100644
--- a/test/orm/test_selectable.py
+++ b/test/orm/test_selectable.py
@@ -27,15 +27,19 @@ class SelectableNoFromsTest(_base.MappedTest):
     @testing.resolve_artifact_names
     def test_no_tables(self):
 
-        selectable = select(["x", "y", "z"])
+        selectable = select(["x", "y", "z"]).alias()
         assert_raises_message(sa.exc.InvalidRequestError,
                                  "Could not find any Table objects",
                                  mapper, Subset, selectable)
 
-    @testing.emits_warning('.*creating an Alias.*')
     @testing.resolve_artifact_names
-    def test_basic(self):
+    def test_no_selects(self):
         subset_select = select([common.c.id, common.c.data])
+        assert_raises(sa.exc.InvalidRequestError, mapper, Subset, subset_select)
+        
+    @testing.resolve_artifact_names
+    def test_basic(self):
+        subset_select = select([common.c.id, common.c.data]).alias()
         subset_mapper = mapper(Subset, subset_select)
 
         sess = create_session(bind=testing.db)