Remove skipped tests

Author: syrusakbary

src/tests/fastapi/test_context.py

@@ -176,49 +176,6 @@ async def resolve_abc(_root, info) -> str:
     assert response.json() == {"data": {"abc": "abc"}}
 
 
-@pytest.mark.skip(reason="This is no longer supported")
-def test_with_invalid_context_getter():
-    from fastapi import Depends, FastAPI
-    from fastapi.testclient import TestClient
-    from graphql_server.fastapi import GraphQLRouter
-
-    def custom_context_dependency() -> str:
-        return "rocks"
-
-    def get_context(value: str = Depends(custom_context_dependency)) -> str:
-        return value
-
-    async def resolve_abc(_root, info) -> str:
-        assert info.context.get("request") is not None
-        assert info.context.get("graphql_server") is None
-        return "abc"
-
-    app = FastAPI()
-    QueryType = GraphQLObjectType(
-        name="Query",
-        fields={
-            "abc": GraphQLField(
-                GraphQLString,
-                resolve=resolve_abc,
-            )
-        },
-    )
-
-    schema = GraphQLSchema(query=QueryType)
-    graphql_app = GraphQLRouter(schema=schema, context_getter=get_context)
-    app.include_router(graphql_app, prefix="/graphql")
-
-    test_client = TestClient(app)
-    with pytest.raises(
-        Exception,
-        match=(
-            "The custom context must be either a class "
-            "that inherits from BaseContext or a dictionary"
-        ),
-    ):
-        test_client.post("/graphql", json={"query": "{ abc }"})
-
-
 def test_class_context_injects_connection_params_over_transport_ws():
     from fastapi import Depends, FastAPI
     from fastapi.testclient import TestClient

src/tests/litestar/test_context.py

@@ -134,47 +134,6 @@ def resolve_abc(_root, info):
     assert response.json() == {"data": {"abc": "abc"}}
 
 
-@pytest.mark.skip(reason="This is no longer supported")
-def test_with_invalid_context_getter():
-    def custom_context_dependency() -> str:
-        return "rocks"
-
-    async def get_context(custom_context_dependency: str) -> str:
-        return custom_context_dependency
-
-    def resolve_abc(_root, info):
-        assert info.context.get("request") is not None
-        assert info.context.get("teststring") is None
-        return "abc"
-
-    Query = GraphQLObjectType(
-        name="Query",
-        fields={
-            "abc": GraphQLField(
-                GraphQLString,
-                resolve=resolve_abc,
-            )
-        },
-    )
-
-    schema = GraphQLSchema(query=Query)
-    graphql_controller = make_graphql_controller(
-        path="/graphql", schema=schema, context_getter=get_context
-    )
-    app = Litestar(
-        route_handlers=[graphql_controller],
-        dependencies={
-            "custom_context_dependency": Provide(
-                custom_context_dependency, sync_to_thread=True
-            )
-        },
-    )
-    test_client = TestClient(app, raise_server_exceptions=True)
-    response = test_client.post("/graphql", json={"query": "{ abc }"})
-    assert response.status_code == 500
-    assert response.json()["detail"] == "Internal Server Error"
-
-
 def test_custom_context():
     from tests.litestar.app import create_app