Skip to content

Commit d38dde4

Browse files
committedJun 24, 2019
fix for execute-plugin when documents field is missing
1 parent c965517 commit d38dde4

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed
 

‎packages/graphql-codegen-core/src/execute-plugin.ts

+5-4
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,16 @@ export async function executePlugin(options: ExecutePluginOptions, plugin: Codeg
3131
}
3232

3333
const outputSchema: GraphQLSchema = buildASTSchema(options.schema);
34+
const documents = options.documents || [];
3435

35-
if (outputSchema && options.documents.length > 0) {
36-
const errors = validateGraphQlDocuments(outputSchema, options.documents);
36+
if (outputSchema && documents.length > 0) {
37+
const errors = validateGraphQlDocuments(outputSchema, documents);
3738
checkValidationErrors(errors);
3839
}
3940

4041
if (plugin.validate && typeof plugin.validate === 'function') {
4142
try {
42-
await plugin.validate(outputSchema, options.documents, options.config, options.outputFilename, options.allPlugins);
43+
await plugin.validate(outputSchema, documents, options.config, options.outputFilename, options.allPlugins);
4344
} catch (e) {
4445
throw new DetailedError(
4546
`Plugin "${options.name}" validation failed:`,
@@ -50,7 +51,7 @@ export async function executePlugin(options: ExecutePluginOptions, plugin: Codeg
5051
}
5152
}
5253

53-
return plugin.plugin(outputSchema, options.documents, options.config, {
54+
return plugin.plugin(outputSchema, documents, options.config, {
5455
outputFile: options.outputFilename,
5556
allPlugins: options.allPlugins,
5657
});

0 commit comments

Comments
 (0)
Please sign in to comment.