Fix javadoc of GraphQLTest

Author: oliemansm

example-graphql-tools/src/test/java/com/graphql/sample/boot/SpringBootTestWithoutWebEnvironmentTest.java

@@ -1,6 +1,7 @@
 package graphql.servlet.examples.dataloader.requestscope;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 import com.fasterxml.jackson.databind.JsonNode;
 import org.junit.jupiter.api.Test;
@@ -15,7 +16,7 @@
 
 @ExtendWith(SpringExtension.class)
 @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
-public class ApplicationTest {
+class ApplicationTest {
 
   @Autowired
   private TestRestTemplate restTemplate;
@@ -24,12 +25,13 @@ public class ApplicationTest {
   private CustomerRepository repository;
 
   @Test
-  public void testSanity() {
-    this.restTemplate.getForObject("/graphql/schema.json", String.class);
+  void testSanity() {
+    String schema = this.restTemplate.getForObject("/graphql/schema.json", String.class);
+    assertNotNull(schema);
   }
 
   @Test
-  public void testRequestScope() {
+  void testRequestScope() {
 
     String requestGraphQL = "query {\n" +
         "  walmartCustomers(storeNumber:4177){\n" +
@@ -42,15 +44,15 @@ public void testRequestScope() {
     ResponseEntity<JsonNode> response = this.restTemplate
         .postForEntity("/graphql", new HttpEntity<>(requestGraphQL, headers), JsonNode.class);
 
-    assertThat(response.getBody().toString()).isEqualTo(
+    assertThat(response.getBody()).hasToString(
         "{\"data\":{\"walmartCustomers\":[{\"customerId\":101,\"name\":\"Customer Name 1\"},{\"customerId\":102,\"name\":\"Customer Name 2\"},{\"customerId\":103,\"name\":\"Customer Name 3\"},{\"customerId\":104,\"name\":\"Customer Name 4\"}]}}");
 
     repository.updateUsernameForId(101, "New Name 1");
 
     response = this.restTemplate
         .postForEntity("/graphql", new HttpEntity<>(requestGraphQL, headers), JsonNode.class);
 
-    assertThat(response.getBody().toString()).isEqualTo(
+    assertThat(response.getBody()).hasToString(
         "{\"data\":{\"walmartCustomers\":[{\"customerId\":101,\"name\":\"New Name 1\"},{\"customerId\":102,\"name\":\"Customer Name 2\"},{\"customerId\":103,\"name\":\"Customer Name 3\"},{\"customerId\":104,\"name\":\"Customer Name 4\"}]}}");
   }
 

example-request-scoped-dataloader/src/test/java/graphql/servlet/examples/dataloader/requestscope/ApplicationTest.java

@@ -1,7 +1,10 @@
 package graphql.kickstart.tools.boot;
 
+import static org.assertj.core.api.Assertions.assertThat;
+
 import graphql.kickstart.tools.GraphQLQueryResolver;
 import graphql.schema.GraphQLObjectType;
+import graphql.schema.GraphQLSchema;
 import graphql.schema.idl.SchemaDirectiveWiring;
 import graphql.schema.idl.SchemaDirectiveWiringEnvironment;
 import org.junit.jupiter.api.AfterEach;
@@ -10,7 +13,7 @@
 import org.springframework.context.annotation.Configuration;
 import org.springframework.stereotype.Component;
 
-public class GraphQLToolsDirectiveTest extends AbstractAutoConfigurationTest {
+class GraphQLToolsDirectiveTest extends AbstractAutoConfigurationTest {
 
   public GraphQLToolsDirectiveTest() {
     super(GraphQLJavaToolsAutoConfiguration.class);
@@ -22,10 +25,11 @@ public void clear() {
   }
 
   @Test
-  public void directiveIsLoaded() {
+  void directiveIsLoaded() {
     System.setProperty("graphql.tools.schemaLocationPattern",
         "graphql/schema-directive-test.graphql");
     load(BaseConfiguration.class);
+    assertThat(this.getContext().getBean(GraphQLSchema.class)).isNotNull();
   }
 
   @Configuration
@@ -43,7 +47,7 @@ public GraphQLObjectType onObject(
     }
 
     @Component
-    public class Query implements GraphQLQueryResolver {
+    public static class Query implements GraphQLQueryResolver {
 
       String schemaLocationTest(String id) {
         return id;

graphql-kickstart-spring-boot-autoconfigure-tools/src/test/java/graphql/kickstart/tools/boot/GraphQLToolsDirectiveTest.java

@@ -48,9 +48,9 @@
 import org.springframework.web.socket.server.standard.ServerEndpointExporter;
 
 /**
- * Annotation that can be specified on a test class in combination with {@code
- * @RunWith(SpringRunner.class)} for running tests that focus <strong>only</strong> on GraphQL
- * components.
+ * Annotation that can be specified on a test class in combination with
+ * <code>@RunWith(SpringRunner.class)</code> for running tests that focus <strong>only</strong> on
+ * GraphQL components.
  * <p>
  * Provides the following features over the regular <strong>Spring TestContext Framework</strong>:
  * <ul>