converters have access to session

tests/test_converters.py

@@ -1,6 +1,7 @@
 from werkzeug.routing import BaseConverter
 
-from flask import has_request_context
+from flask import request
+from flask import session
 from flask import url_for
 
 
@@ -28,12 +29,13 @@ def test_custom_converters(app, client):
 def test_context_available(app, client):
     class ContextConverter(BaseConverter):
         def to_python(self, value):
-            assert has_request_context()
+            assert request is not None
+            assert session is not None
             return value
 
     app.url_map.converters["ctx"] = ContextConverter
 
-    @app.route("/<ctx:name>")
+    @app.get("/<ctx:name>")
     def index(name):
         return name
 

tests/test_session_interface.py

@@ -2,21 +2,26 @@ import flask
 from flask.sessions import SessionInterface
 
 
-def test_open_session_endpoint_not_none():
-    # Define a session interface that breaks if request.endpoint is None
+def test_open_session_with_endpoint():
+    """If request.endpoint (or other URL matching behavior) is needed
+    while loading the session, RequestContext.match_request() can be
+    called manually.
+    """
+
     class MySessionInterface(SessionInterface):
-        def save_session(self):
+        def save_session(self, app, session, response):
             pass
 
-        def open_session(self, _, request):
+        def open_session(self, app, request):
+            flask._request_ctx_stack.top.match_request()
             assert request.endpoint is not None
 
-    def index():
-        return "Hello World!"
-
-    # Confirm a 200 response, indicating that request.endpoint was NOT None
     app = flask.Flask(__name__)
-    app.route("/")(index)
     app.session_interface = MySessionInterface()
-    response = app.test_client().open("/")
+
+    @app.get("/")
+    def index():
+        return "Hello, World!"
+
+    response = app.test_client().get("/")
     assert response.status_code == 200