Spring/spring-graphql
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Allow variables with null values in GraphQlTester

Browse Source 
See gh-176
This commit is contained in:
Koen Punt
2021-10-31 20:40:53 +01:00
committed by Rossen Stoyanchev
parent 5aaf5b1df4
commit a06ebdcd9c
5 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
spring-graphql-test/src/main/java/org/springframework/graphql/test/tester/DefaultGraphQlTester.java
View File

@@ -109,7 +109,7 @@ class DefaultGraphQlTester implements GraphQlTester {
}
@Override
public DefaultRequestSpec variable(String name, Object value) {
public DefaultRequestSpec variable(String name, @Nullable Object value) {
addVariable(name, value);
return this;
}

2
spring-graphql-test/src/main/java/org/springframework/graphql/test/tester/DefaultWebGraphQlTester.java
View File

@@ -119,7 +119,7 @@ class DefaultWebGraphQlTester implements WebGraphQlTester {
}
@Override
public WebRequestSpec variable(String name, Object value) {
public WebRequestSpec variable(String name, @Nullable Object value) {
addVariable(name, value);
return this;
}

2
spring-graphql-test/src/main/java/org/springframework/graphql/test/tester/GraphQlTester.java
View File

@@ -173,7 +173,7 @@ public interface GraphQlTester {
* @param value the variable value
* @return this request spec
*/
T variable(String name, Object value);
T variable(String name, @Nullable Object value);
/**
* Set the locale to associate with the request.

2
spring-graphql-test/src/main/java/org/springframework/graphql/test/tester/GraphQlTesterRequestSpecSupport.java
View File

@@ -53,7 +53,7 @@ class GraphQlTesterRequestSpecSupport {
this.operationName = name;
}
protected void addVariable(String name, Object value) {
protected void addVariable(String name, @Nullable Object value) {
this.variables.put(name, value);
}

4
spring-graphql-test/src/test/java/org/springframework/graphql/test/tester/GraphQlTesterTests.java
View File

@@ -179,6 +179,7 @@ public class GraphQlTesterTests {
.operationName("HeroNameAndFriends")
.variable("episode", "JEDI")
.variable("foo", "bar")
.variable("optional", null)
.execute();
spec.path("hero").entity(MovieCharacter.class).isEqualTo(MovieCharacter.create("R2-D2"));
@@ -186,9 +187,10 @@ public class GraphQlTesterTests {
RequestInput input = this.inputCaptor.getValue();
assertThat(input.getQuery()).contains(query);
assertThat(input.getOperationName()).isEqualTo("HeroNameAndFriends");
assertThat(input.getVariables()).hasSize(2);
assertThat(input.getVariables()).hasSize(3);
assertThat(input.getVariables()).containsEntry("episode", "JEDI");
assertThat(input.getVariables()).containsEntry("foo", "bar");
assertThat(input.getVariables()).containsEntry("optional", null);
}
@Test