completed test coverage

Signed-off-by: Oleg Höfling <[email protected]>

Author: hoefling

aiohttp_graphql/graphqlview.py

@@ -164,12 +164,6 @@ async def __call__(self, request):
             )
 
         except HttpQueryError as err:
-            if err.headers and 'Allow' in err.headers:
-                # bug in graphql_server.execute_graphql_request
-                # https://github.com/graphql-python/graphql-server-core/pull/4
-                if isinstance(err.headers['Allow'], list):
-                    err.headers['Allow'] = ', '.join(err.headers['Allow'])
-
             return web.Response(
                 text=self.encoder({
                     'errors': [self.error_formatter(err)]

tests/test_graphqlview.py

@@ -544,12 +544,18 @@ async def test_passes_request_into_request_context(client, url_builder):
 
 class TestCustomContext:
     @pytest.fixture
-    def view_kwargs(self, view_kwargs):
+    def view_kwargs(self, request, view_kwargs):
         # pylint: disable=no-self-use
         # pylint: disable=redefined-outer-name
-        view_kwargs.update(context='CUSTOM CONTEXT')
+        view_kwargs.update(context=request.param)
         return view_kwargs
 
+    @pytest.mark.parametrize(
+        'view_kwargs',
+        ['CUSTOM CONTEXT', {'CUSTOM_CONTEXT': 'test'}],
+        indirect=True,
+        ids=repr
+    )
     @pytest.mark.asyncio
     async def test_context_remapped(self, client, url_builder):
         response = await client.get(url_builder(query='{context}'))
@@ -560,6 +566,19 @@ async def test_context_remapped(self, client, url_builder):
         assert 'CUSTOM CONTEXT' not in _json['data']['context']
 
 
+    @pytest.mark.parametrize(
+        'view_kwargs', [{'request': 'test'}], indirect=True, ids=repr
+    )
+    @pytest.mark.asyncio
+    async def test_request_not_replaced(self, client, url_builder):
+        response = await client.get(url_builder(query='{context}'))
+
+        _json = await response.json()
+        assert response.status == 200
+        assert 'request' in _json['data']['context']
+        assert _json['data']['context'] == str({'request': 'test'})
+
+
 @pytest.mark.asyncio
 async def test_post_multipart_data(client, base_url):
     # pylint: disable=line-too-long