Switch on ExtendedScalars.Object instead of our custom objectGraphQLScalarType

Author: Victor LOIZEL

pom.xml

@@ -25,6 +25,12 @@
 			<artifactId>spring-boot-starter-web</artifactId>
 		</dependency>
 
+		<dependency>
+			<groupId>com.graphql-java</groupId>
+			<artifactId>graphql-java-extended-scalars</artifactId>
+			<version>18.1</version>
+		</dependency>
+
 		<dependency>
 			<groupId>org.springframework.boot</groupId>
 			<artifactId>spring-boot-starter-test</artifactId>

src/main/java/issue/spring/graphql/objectlist/GraphQlConfig.java

@@ -1,5 +1,6 @@
 package issue.spring.graphql.objectlist;
 
+import graphql.scalars.ExtendedScalars;
 import graphql.schema.*;
 import org.springframework.boot.autoconfigure.graphql.GraphQlSourceBuilderCustomizer;
 import org.springframework.context.annotation.Bean;
@@ -37,7 +38,8 @@ public Object parseLiteral(Object input) throws CoercingParseLiteralException {
     public GraphQlSourceBuilderCustomizer sourceBuilderCustomizer(GraphQLScalarType objectGraphQLScalarType) {
         return (graphQlSourceBuilder) -> graphQlSourceBuilder
                 .configureRuntimeWiring((wiringBuilder) -> wiringBuilder
-                        .scalar(objectGraphQLScalarType));
+                        .scalar(ExtendedScalars.Object));
+                        //.scalar(objectGraphQLScalarType));
     }
 }
 

src/test/java/issue/spring/graphql/objectlist/BusinessEntityControllerTests.java

@@ -1,5 +1,6 @@
 package issue.spring.graphql.objectlist;
 
+import graphql.scalars.ExtendedScalars;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.graphql.GraphQlTest;
@@ -37,6 +38,10 @@ void filterValueIsInstanceOfList_stringValue() {
     /**
      * There is no {@link Exception} with an array of 2 {@link String} declared directly in the GraphQL query,
      * we have an {@link graphql.language.ArrayValue} in {@link Object}, shall we have a List instead ?
+     *
+     * UPDATE: after switching on {@link ExtendedScalars} object type we have the exception on both test
+     * (`filterValueIsInstanceOfList_withoutVariables` and `filterValueIsInstanceOfList_withException`),
+     * it is more coherent.
      */
     @Test
     void filterValueIsInstanceOfList_withoutVariables() {