File tree Expand file tree Collapse file tree 2 files changed +17
-2
lines changed
main/java/org/springframework/data/redis/connection
test/java/org/springframework/data/redis/connection Expand file tree Collapse file tree 2 files changed +17
-2
lines changed Original file line number Diff line number Diff line change @@ -73,7 +73,11 @@ public static ReturnType fromJavaType(@Nullable Class<?> javaType) {
73
73
return ReturnType .BOOLEAN ;
74
74
}
75
75
76
- if (ClassUtils .isAssignable (Long .class , javaType )) {
76
+ if (ClassUtils .isAssignable (Double .class , javaType ) || ClassUtils .isAssignable (Float .class , javaType )) {
77
+ return ReturnType .VALUE ;
78
+ }
79
+
80
+ if (ClassUtils .isAssignable (Number .class , javaType )) {
77
81
return ReturnType .INTEGER ;
78
82
}
79
83
Original file line number Diff line number Diff line change @@ -34,8 +34,19 @@ class ReturnTypeUnitTests {
34
34
@ ParameterizedTest // DATAREDIS-1245
35
35
@ ValueSource (classes = { List .class , ArrayList .class , LinkedList .class })
36
36
void shouldConsiderListsAsMultiType (Class <?> listClass ) {
37
-
38
37
assertThat (ReturnType .fromJavaType (listClass )).isEqualTo (ReturnType .MULTI );
39
38
}
40
39
40
+ @ ParameterizedTest // GH-3090
41
+ @ ValueSource (classes = { Integer .class , Long .class , Number .class })
42
+ void shouldConsiderIntegerType (Class <?> listClass ) {
43
+ assertThat (ReturnType .fromJavaType (listClass )).isEqualTo (ReturnType .INTEGER );
44
+ }
45
+
46
+ @ ParameterizedTest // GH-3090
47
+ @ ValueSource (classes = { Double .class , Float .class , String .class })
48
+ void shouldConsiderValueType (Class <?> listClass ) {
49
+ assertThat (ReturnType .fromJavaType (listClass )).isEqualTo (ReturnType .VALUE );
50
+ }
51
+
41
52
}
You can’t perform that action at this time.
0 commit comments