Remove all remaining removed_in_20 warnings slated for removal

Finalize all remaining removed-in-2.0 changes so that we
can begin doing pep-484 typing without old things
getting in the way (we will also have to do public_factory).

note there are a few "moved_in_20()" and "became_legacy_in_20()"
warnings still in place.  The SQLALCHEMY_WARN_20 variable
is now removed.

Also removed here are the legacy "in place mutators" for Select
statements, and some keyword-only argument signatures in Core
have been added.

Also in the big change department, the ORM mapper() function
is removed entirely; the Mapper class is otherwise unchanged,
just the public-facing API function.  Mappers are now always
given a registry in which to participate, however the
argument signature of Mapper is not changed. ideally "registry"
would be the first positional argument.

Fixes: #7257

Change-Id: Ic70c57b9f1cf7eb996338af5183b11bdeb3e1623

1 files changed, 6 insertions, 14 deletions

diff --git a/lib/sqlalchemy/sql/coercions.py b/lib/sqlalchemy/sql/coercions.py
index d11ab6712..fe4fc4b40 100644
--- a/lib/sqlalchemy/sql/coercions.py
+++ b/lib/sqlalchemy/sql/coercions.py
@@ -99,14 +99,15 @@ def _document_text_coercion(paramname, meth_rst, param_rst):
 def _expression_collection_was_a_list(attrname, fnname, args):
     if args and isinstance(args[0], (list, set, dict)) and len(args) == 1:
         if isinstance(args[0], list):
-            util.warn_deprecated_20(
-                'The "%s" argument to %s(), when referring to a sequence '
+            raise exc.ArgumentError(
+                f'The "{attrname}" argument to {fnname}(), when '
+                "referring to a sequence "
                 "of items, is now passed as a series of positional "
-                "elements, rather than as a list. " % (attrname, fnname)
+                "elements, rather than as a list. "
             )
         return args[0]
-    else:
-        return args
+
+    return args
 
 
 def expect(
@@ -883,15 +884,6 @@ class StatementImpl(_CoerceLiterals, RoleImpl):
                 original_element, resolved, argname=argname, **kw
             )
 
-    def _text_coercion(self, element, argname=None):
-        util.warn_deprecated_20(
-            "Using plain strings to indicate SQL statements without using "
-            "the text() construct is  "
-            "deprecated and will be removed in version 2.0.  Ensure plain "
-            "SQL statements are passed using the text() construct."
-        )
-        return elements.TextClause(element)
-
 
 class SelectStatementImpl(_NoTextCoercion, RoleImpl):
     __slots__ = ()