Simplify SDLs used in 'buidSchema'/`extendSchema' tests (#1964)

Author: IvanGoncharov

src/utilities/__tests__/buildASTSchema-test.js

@@ -110,11 +110,7 @@ describe('Schema Builder', () => {
   });
 
   it('include standard type only if it is used', () => {
-    const schema = buildSchema(`
-      type Query {
-        str: String
-      }
-    `);
+    const schema = buildSchema('type Query');
 
     // String and Boolean are always included through introspection types
     expect(schema.getType('Int')).to.equal(undefined);
@@ -125,10 +121,6 @@ describe('Schema Builder', () => {
   it('With directives', () => {
     const sdl = dedent`
       directive @foo(arg: Int) on FIELD
-
-      type Query {
-        str: String
-      }
     `;
     expect(cycleSDL(sdl)).to.equal(sdl);
   });
@@ -186,11 +178,7 @@ describe('Schema Builder', () => {
   });
 
   it('Maintains @skip & @include', () => {
-    const schema = buildSchema(`
-      type Query {
-        str: String
-      }
-    `);
+    const schema = buildSchema('type Query');
 
     expect(schema.getDirectives()).to.have.lengthOf(3);
     expect(schema.getDirective('skip')).to.equal(GraphQLSkipDirective);
@@ -205,10 +193,6 @@ describe('Schema Builder', () => {
       directive @skip on FIELD
       directive @include on FIELD
       directive @deprecated on FIELD_DEFINITION
-
-      type Query {
-        str: String
-      }
     `);
 
     expect(schema.getDirectives()).to.have.lengthOf(3);
@@ -224,10 +208,6 @@ describe('Schema Builder', () => {
   it('Adding directives maintains @skip & @include', () => {
     const schema = buildSchema(`
       directive @foo(arg: Int) on FIELD
-
-      type Query {
-        str: String
-      }
     `);
 
     expect(schema.getDirectives()).to.have.lengthOf(4);
@@ -261,10 +241,6 @@ describe('Schema Builder', () => {
 
   it('Two types circular', () => {
     const sdl = dedent`
-      schema {
-        query: TypeOne
-      }
-
       type TypeOne {
         str: String
         typeTwo: TypeTwo
@@ -802,9 +778,9 @@ describe('Schema Builder', () => {
         mutation: SomeMutation
         subscription: SomeSubscription
       }
-      type SomeQuery { str: String }
-      type SomeMutation { str: String }
-      type SomeSubscription { str: String }
+      type SomeQuery
+      type SomeMutation
+      type SomeSubscription
     `);
 
     expect(schema.getQueryType()).to.include({ name: 'SomeQuery' });
@@ -816,9 +792,9 @@ describe('Schema Builder', () => {
 
   it('Default root operation type names', () => {
     const schema = buildSchema(`
-      type Query { str: String }
-      type Mutation { str: String }
-      type Subscription { str: String }
+      type Query
+      type Mutation
+      type Subscription
     `);
 
     expect(schema.getQueryType()).to.include({ name: 'Query' });
@@ -827,12 +803,8 @@ describe('Schema Builder', () => {
   });
 
   it('can build invalid schema', () => {
-    const schema = buildSchema(`
-      # Invalid schema, because it is missing query root type
-      type Mutation {
-        str: String
-      }
-    `);
+    // Invalid schema, because it is missing query root type
+    const schema = buildSchema('type Mutation');
     const errors = validateSchema(schema);
     expect(errors).to.have.lengthOf.above(0);
   });

src/utilities/__tests__/extendSchema-test.js

@@ -268,11 +268,7 @@ describe('extendSchema', () => {
   });
 
   it('extends objects with standard type fields', () => {
-    const schema = buildSchema(`
-      type Query {
-        str: String
-      }
-    `);
+    const schema = buildSchema('type Query');
 
     // String and Boolean are always included through introspection types
     expect(schema.getType('Int')).to.equal(undefined);
@@ -1196,9 +1192,7 @@ describe('extendSchema', () => {
   describe('can add additional root operation types', () => {
     it('does not automatically include common root type names', () => {
       const schema = extendTestSchema(`
-        type Mutation {
-          doSomething: String
-        }
+        type Mutation
       `);
       expect(schema.getMutationType()).to.equal(null);
     });
@@ -1227,9 +1221,7 @@ describe('extendSchema', () => {
           mutation: Mutation
         }
 
-        type Mutation {
-          doSomething: String
-        }
+        type Mutation
       `);
       const mutationType = schema.getMutationType();
       expect(mutationType).to.include({ name: 'Mutation' });
@@ -1242,13 +1234,8 @@ describe('extendSchema', () => {
           subscription: Subscription
         }
 
-        type Mutation {
-          doSomething: String
-        }
-
-        type Subscription {
-          hearSomething: String
-        }
+        type Mutation
+        type Subscription
       `);
       const mutationType = schema.getMutationType();
       const subscriptionType = schema.getSubscriptionType();
@@ -1261,18 +1248,12 @@ describe('extendSchema', () => {
         extend schema {
           mutation: Mutation
         }
+        type Mutation
 
         extend schema {
           subscription: Subscription
         }
-
-        type Mutation {
-          doSomething: String
-        }
-
-        type Subscription {
-          hearSomething: String
-        }
+        type Subscription
       `);
       const mutationType = schema.getMutationType();
       const subscriptionType = schema.getSubscriptionType();
@@ -1285,18 +1266,12 @@ describe('extendSchema', () => {
         extend schema {
           mutation: Mutation
         }
+        type Mutation
 
         extend schema {
           subscription: Subscription
         }
-
-        type Mutation {
-          doSomething: String
-        }
-
-        type Subscription {
-          hearSomething: String
-        }
+        type Subscription
       `);
 
       const ast = parse(`