implement requested feedback

issue spring-projects#3501

Author: dmfrey

build.gradle

@@ -529,8 +529,6 @@ project('spring-integration-graphql') {
 	dependencies {
 		api project(':spring-integration-core')
 		api "org.springframework.graphql:spring-graphql:$springGraphqlVersion"
-
-		testImplementation "org.springframework.graphql:spring-graphql-test:$springGraphqlVersion"
 	}
 }
 

spring-integration-graphql/src/main/java/org/springframework/integration/graphql/outbound/GraphQlMessageHandler.java

@@ -20,20 +20,24 @@
 import org.springframework.graphql.RequestInput;
 import org.springframework.integration.handler.AbstractReplyProducingMessageHandler;
 import org.springframework.messaging.Message;
+import org.springframework.util.Assert;
 
 import graphql.ExecutionResult;
 import reactor.core.publisher.Mono;
 
 /**
- * A <code>MessageHandler</code> capable of fielding GraphQL Query, Mutation and Subscription requests.
+ * A {@link org.springframework.messaging.MessageHandler} capable of fielding GraphQL Query, Mutation and Subscription requests.
  *
  * @author Daniel Frey
+ * @since 6.0
  */
 public class GraphQlMessageHandler extends AbstractReplyProducingMessageHandler {
 
 	private final GraphQlService graphQlService;
 
 	public GraphQlMessageHandler(final GraphQlService graphQlService) {
+		Assert.notNull(graphQlService, "'graphQlService' must not be null");
+
 		this.graphQlService = graphQlService;
 		setAsync(true);
 	}

spring-integration-graphql/src/main/java/org/springframework/integration/graphql/outbound/package-info.java

@@ -1 +1 @@
-package org.springframework.integration.graphql.outbound;
+package org.springframework.integration.graphql.outbound;

spring-integration-graphql/src/main/java/org/springframework/integration/graphql/package-info.java

@@ -2,4 +2,4 @@
  * Root package of the Spring Integration GraphQL module, which contains various
  * GraphQL Query, Mutation and Subscription components
  */
-package org.springframework.integration.graphql;
+package org.springframework.integration.graphql;

spring-integration-graphql/src/test/java/org/springframework/integration/graphql/outbound/GraphQlMessageHandlerTests.java

@@ -65,9 +65,9 @@
 /**
  *
  * @author Daniel Frey
- *
+ * @since 6.0
  */
-@SpringJUnitConfig(GraphQlMessageHandlerTests.TestConfig.class)
+@SpringJUnitConfig
 @DirtiesContext
 public class GraphQlMessageHandlerTests {
 

spring-integration-graphql/src/test/resources/graphql/test-schema.graphqls

@@ -16,4 +16,4 @@ type QueryResult {
 
 type Update {
     id: String
-}
+}

spring-integration-graphql/src/test/resources/log4j2-test.xml

@@ -6,7 +6,6 @@
 		</Console>
 	</Appenders>
 	<Loggers>
-		<Logger name="test" level="debug"/>
 		<Logger name="org.springframework" level="warn"/>
 		<Logger name="org.springframework.graphql" level="warn"/>
 		<Logger name="org.springframework.integration.graphql" level="warn"/>

src/reference/asciidoc/graphql.adoc

@@ -0,0 +1,24 @@
+[[graphql]]
+== GraphQL Support
+
+Spring Integration provides support for GraphQL.
+
+You need to include this dependency into your project:
+
+====
+.Maven
+[source, xml, subs="normal"]
+----
+<dependency>
+    <groupId>org.springframework.integration</groupId>
+    <artifactId>spring-integration-graphql</artifactId>
+    <version>{project-version}</version>
+</dependency>
+----
+
+.Gradle
+[source, groovy, subs="normal"]
+----
+compile "org.springframework.integration:spring-integration-graphql:{project-version}"
+----
+====

src/reference/asciidoc/index-single.adoc

@@ -41,6 +41,8 @@ include::./ftp.adoc[]
 
 include::./gemfire.adoc[]
 
+include::./graphql.adoc[]
+
 include::./http.adoc[]
 
 include::./jdbc.adoc[]

src/reference/asciidoc/index.adoc

@@ -30,6 +30,7 @@ This documentation is also available as single searchable link:index-single.html
 <<./file.adoc#files,File Support>> ::
 <<./ftp.adoc#ftp,FTP/FTPS Adapters>> ::
 <<./gemfire.adoc#gemfire,Pivotal GemFire and Apache Geode Support>> ::
+<<./graphql.adoc#graphql,GraphQL Support>> ::
 <<./http.adoc#http,HTTP Support>> ::
 <<./jdbc.adoc#jdbc,JDBC Support>> ::
 <<./jpa.adoc#jpa,JPA Support>> ::