starWarsIntrospection-test __schema.types confusion

[burner]

I'm implementing a graphql backend in D and when looking at the expected types
returned in https://github.com/graphql/graphql-js/blob/master/src/__tests__/starWarsIntrospection-test.js#L22
I'm confused why "Boolean" is part of the returned names.
I get that bool is a standard type but so is Int and Int does not appear in the names array.
That being said, no field or inputvalue has the type bool or int in the star wars schema.

Could somebody please clarify this for me

[IvanGoncharov]

I get that bool is a standard type but so is Int and Int does not appear in the names array.

@burner You absolutely correct types (both standard and user-defined) are included only if they are used in the schema.
Boolean type is used both inside introspection types, e.g.:

graphql-js/src/type/introspection.js

Line 227 in f85d9b6

includeDeprecated: { type: GraphQLBoolean, defaultValue: false },

and in standard directives, e.g.:

graphql-js/src/type/directives.js

Line 133 in f85d9b6

type: GraphQLNonNull(GraphQLBoolean),

[burner]

Thank you for the quick answer.
My bad, sorry for the noise.

[IvanGoncharov]

@burner No problem. The main purpose of graphql-js is to be the reference implementation of GraphQL and assist with implementing interoperable GraphQL servers in various languages.
If something is not clear from source code it should be addressed in comments so you actually helping us to improve source code and save time for future readers.
For example, based on this issue I added this comment:

graphql-js/src/__tests__/starWarsIntrospection-test.js

Lines 29 to 32 in d9f959e

	// Include all types used by StarWars schema, introspection types and
	// standard directives. For example, `Boolean` is used in `@skip`,
	// `@include` and also inside introspection types.
	expect(data).to.deep.equal({

So please feel free to open similar issues if you will have such questions.
Also if you discover something that is not covered by our tests please open an issue or PR.

[burner]

@IvanGoncharov will do, thanks