diff --git a/graphql-kickstart-spring-support/src/main/java/graphql/kickstart/spring/error/GraphQLErrorHandlerFactory.java b/graphql-kickstart-spring-support/src/main/java/graphql/kickstart/spring/error/GraphQLErrorHandlerFactory.java index 1b044e1b..d31e44fb 100644 --- a/graphql-kickstart-spring-support/src/main/java/graphql/kickstart/spring/error/GraphQLErrorHandlerFactory.java +++ b/graphql-kickstart-spring-support/src/main/java/graphql/kickstart/spring/error/GraphQLErrorHandlerFactory.java @@ -27,7 +27,7 @@ public GraphQLErrorHandler create( .flatMap(List::stream) .collect(toList()); - if (!factories.isEmpty() || exceptionHandlersEnabled) { + if (exceptionHandlersEnabled) { log.debug( "Handle GraphQL errors using exception handlers defined in {} custom factories", factories.size()); @@ -43,7 +43,7 @@ private List scanForExceptionHandlers( try { Class objClz = context.getType(name); if (objClz == null) { - log.info("Cannot load class " + name); + log.info("Cannot load class {}", name); return emptyList(); } return Arrays.stream(objClz.getDeclaredMethods()) diff --git a/graphql-spring-boot-autoconfigure/src/test/java/graphql/kickstart/spring/web/boot/GraphQLErrorHandlerTest.java b/graphql-spring-boot-autoconfigure/src/test/java/graphql/kickstart/spring/web/boot/GraphQLErrorHandlerTest.java index 1ea2c668..e729434a 100644 --- a/graphql-spring-boot-autoconfigure/src/test/java/graphql/kickstart/spring/web/boot/GraphQLErrorHandlerTest.java +++ b/graphql-spring-boot-autoconfigure/src/test/java/graphql/kickstart/spring/web/boot/GraphQLErrorHandlerTest.java @@ -29,6 +29,7 @@ public GraphQLErrorHandlerTest() { @BeforeEach public void setUp() { System.setProperty("graphql.tools.schemaLocationPattern", "graphql/error-handler-test.graphql"); + System.setProperty("graphql.servlet.exception-handlers-enabled", "true"); load(BaseConfiguration.class); GraphQLSchema schema = getContext().getBean(GraphQLSchema.class);