add_route types missing one use case

[patrick91]

Hi there! In Strawberry we have an ASGI view that can be used with Starlette, like this

from starlette.applications import Starlette
from strawberry.asgi import GraphQL

from api.schema import schema

graphql_app = GraphQL(schema)

app = Starlette()
app.add_route("/graphql", graphql_app)

This route is a callable that accepts scope, receive and send, but looks like the types of add_route don't support that use case:

starlette/starlette/applications.py

Line 161 in 6f863b0

route: typing.Callable[[Request], typing.Awaitable[Response] | Response],

is this expected? if not I'm happy to send a PR to update the types :)

[Kludex]

It is expected. Why is the app not mounted?

[patrick91]

@Kludex mmh, not sure I follow, do we need to change our implementation? 😊

[Kludex]

@Kludex mmh, not sure I follow, do we need to change our implementation? 😊

I don't think so... But on the docs, you can just have the "mount" as follows:

from starlette.applications import Starlette
from strawberry.asgi import GraphQL

from api import schema

graphql_app = GraphQL[None, None](schema)

app = Starlette()
app.mount("/graphql", graphql_app)

Or even better:

from starlette.applications import Starlette
from starlette.routing import Mount
from strawberry.asgi import GraphQL

from api import schema

graphql_app = GraphQL[None, None](schema)

app = Starlette(routes=[Mount("/graphql", graphql_app)])