Merge pull request #76 from martinandert/fix-directive-introspection-according-to-spec

fix directive inspection according to spec

Author: dschafer

src/type/__tests__/introspection.js

@@ -651,9 +651,13 @@ describe('Introspection', () => {
                   name: 'onOperation',
                   args: [],
                   type: {
-                    kind: 'SCALAR',
-                    name: 'Boolean',
-                    ofType: null,
+                    kind: 'NON_NULL',
+                    name: null,
+                    ofType: {
+                      kind: 'SCALAR',
+                      name: 'Boolean',
+                      ofType: null,
+                    },
                   },
                   isDeprecated: false,
                   deprecationReason: null
@@ -662,9 +666,13 @@ describe('Introspection', () => {
                   name: 'onFragment',
                   args: [],
                   type: {
-                    kind: 'SCALAR',
-                    name: 'Boolean',
-                    ofType: null,
+                    kind: 'NON_NULL',
+                    name: null,
+                    ofType: {
+                      kind: 'SCALAR',
+                      name: 'Boolean',
+                      ofType: null,
+                    },
                   },
                   isDeprecated: false,
                   deprecationReason: null
@@ -673,9 +681,13 @@ describe('Introspection', () => {
                   name: 'onField',
                   args: [],
                   type: {
-                    kind: 'SCALAR',
-                    name: 'Boolean',
-                    ofType: null,
+                    kind: 'NON_NULL',
+                    name: null,
+                    ofType: {
+                      kind: 'SCALAR',
+                      name: 'Boolean',
+                      ofType: null,
+                    },
                   },
                   isDeprecated: false,
                   deprecationReason: null

src/type/__tests__/printer.js

@@ -508,9 +508,9 @@ type __Directive {
   name: String!
   description: String
   args: [__InputValue!]!
-  onOperation: Boolean
-  onFragment: Boolean
-  onField: Boolean
+  onOperation: Boolean!
+  onFragment: Boolean!
+  onField: Boolean!
 }
 
 type __EnumValue {

src/type/introspection.js

@@ -67,9 +67,9 @@ var __Directive = new GraphQLObjectType({
         new GraphQLNonNull(new GraphQLList(new GraphQLNonNull(__InputValue))),
       resolve: directive => directive.args || []
     },
-    onOperation: { type: GraphQLBoolean },
-    onFragment: { type: GraphQLBoolean },
-    onField: { type: GraphQLBoolean },
+    onOperation: { type: new GraphQLNonNull(GraphQLBoolean) },
+    onFragment: { type: new GraphQLNonNull(GraphQLBoolean) },
+    onField: { type: new GraphQLNonNull(GraphQLBoolean) },
   }),
 });