Improve test descriptions

Author: kyarik

src/__tests__/http-test.ts

@@ -222,7 +222,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Reports validation errors', async () => {
+    it('reports validation errors', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -248,7 +248,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Errors when missing operation name', async () => {
+    it('errors when missing operation name', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -273,7 +273,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Errors when sending a mutation via GET', async () => {
+    it('errors when sending a mutation via GET', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -295,7 +295,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Errors when selecting a mutation within a GET', async () => {
+    it('errors when selecting a mutation within a GET', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -321,7 +321,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows a mutation to exist within a GET', async () => {
+    it('allows a mutation to exist within a GET', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -344,7 +344,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows async resolvers', async () => {
+    it('allows async resolvers', async () => {
       const schema = new GraphQLSchema({
         query: new GraphQLObjectType({
           name: 'Query',
@@ -372,7 +372,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows passing in a context', async () => {
+    it('allows passing in a context', async () => {
       const schema = new GraphQLSchema({
         query: new GraphQLObjectType({
           name: 'Query',
@@ -408,7 +408,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows passing in a fieldResolver', async () => {
+    it('allows passing in a fieldResolver', async () => {
       const schema = buildSchema(`
         type Query {
           test: String
@@ -438,7 +438,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows passing in a typeResolver', async () => {
+    it('allows passing in a typeResolver', async () => {
       const schema = buildSchema(`
         type Foo {
           foo: String
@@ -479,7 +479,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Uses request as context by default', async () => {
+    it('uses request as context by default', async () => {
       const schema = new GraphQLSchema({
         query: new GraphQLObjectType({
           name: 'Query',
@@ -515,7 +515,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Allows returning an options Promise', async () => {
+    it('allows returning an options Promise', async () => {
       const app = server();
 
       app.get(
@@ -536,7 +536,7 @@ function runTests(server: Server) {
       expect(response.text).to.equal('{"data":{"test":"Hello World"}}');
     });
 
-    it('Provides an options function with arguments', async () => {
+    it('provides an options function with arguments', async () => {
       const app = server();
 
       let seenRequest;
@@ -571,7 +571,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('Catches errors thrown from options function', async () => {
+    it('catches errors thrown from options function', async () => {
       const app = server();
 
       app.get(
@@ -613,7 +613,7 @@ function runTests(server: Server) {
       expect(response.text).to.equal('{"data":{"test":"Hello World"}}');
     });
 
-    it('Allows sending a mutation via POST', async () => {
+    it('allows sending a mutation via POST', async () => {
       const app = server();
 
       app.post(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -1658,7 +1658,7 @@ function runTests(server: Server) {
   });
 
   describe('Built-in GraphiQL support', () => {
-    it('does not renders GraphiQL if no opt-in', async () => {
+    it('does not render GraphiQL if no opt-in', async () => {
       const app = server();
 
       app.get(urlString(), graphqlHTTP({ schema: TestSchema }));
@@ -1829,7 +1829,7 @@ function runTests(server: Server) {
       );
     });
 
-    it('GraphiQL renders provided variables', async () => {
+    it('renders provided variables within GraphiQL', async () => {
       const app = server();
 
       app.get(
@@ -1858,7 +1858,7 @@ function runTests(server: Server) {
       );
     });
 
-    it('GraphiQL accepts an empty query', async () => {
+    it('accepts an empty query when rendering GraphiQL', async () => {
       const app = server();
 
       app.get(
@@ -1879,7 +1879,7 @@ function runTests(server: Server) {
       expect(response.text).to.include('response: undefined');
     });
 
-    it('GraphiQL accepts a mutation query - does not execute it', async () => {
+    it('accepts a mutation query when rendering GraphiQL - does not execute it', async () => {
       const app = server();
 
       app.get(
@@ -2115,7 +2115,7 @@ function runTests(server: Server) {
       };
     };
 
-    it('Do not execute a query if it do not pass the custom validation.', async () => {
+    it('does not execute a query if the custom validation does not pass', async () => {
       const app = server();
 
       app.get(
@@ -2144,8 +2144,8 @@ function runTests(server: Server) {
     });
   });
 
-  describe('Custom execute', () => {
-    it('allow to replace default execute', async () => {
+  describe('Custom execute function', () => {
+    it('allows to replace the default execute function', async () => {
       const app = server();
 
       let seenExecuteArgs;
@@ -2247,7 +2247,7 @@ function runTests(server: Server) {
   });
 
   describe('Custom result extensions', () => {
-    it('allows for adding extensions', async () => {
+    it('allows adding extensions', async () => {
       const app = server();
 
       app.get(
@@ -2273,7 +2273,7 @@ function runTests(server: Server) {
       );
     });
 
-    it('extensions have access to initial GraphQL result', async () => {
+    it('gives extensions access to the initial GraphQL result', async () => {
       const app = server();
 
       app.get(
@@ -2314,7 +2314,7 @@ function runTests(server: Server) {
       });
     });
 
-    it('extension function may be async', async () => {
+    it('allows the extensions function to be async', async () => {
       const app = server();
 
       app.get(