Skip to content

Commit f8962f8

Browse files
Shredder121odrotbohm
authored andcommitted
DATACMNS-753 - Added test case to see parsing of Sort is not considering defaults for empty parameter.
Related ticket: DATACMNS-408. Original pull request: #138.
1 parent 79b15df commit f8962f8

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

src/test/java/org/springframework/data/web/SortHandlerMethodArgumentResolverUnitTests.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,21 @@ public void parsesEmptySortToNull() throws Exception {
149149
assertThat(resolveSort(request, PARAMETER), is(nullValue()));
150150
}
151151

152+
/**
153+
*
154+
* @see DATACMNS-753
155+
* see also DATACMNS-408
156+
*/
157+
@Test
158+
public void doesNotReturnNullWhenAnnotatedWithSortDefault() throws Exception {
159+
160+
MockHttpServletRequest request = new MockHttpServletRequest();
161+
request.addParameter("sort", ""); //valid input
162+
163+
assertNotNull(resolveSort(request, getParameterOfMethod("simpleDefault")));
164+
assertNotNull(resolveSort(request, getParameterOfMethod("containeredDefault")));
165+
}
166+
152167
/**
153168
* @see DATACMNS-408
154169
*/

0 commit comments

Comments
 (0)