[solr] branch main updated: SOLR-15839: Don't mock oal.search.Sort class (why the hell was this needed?) (#452)

[uschindler]

This is an automated email from the ASF dual-hosted git repository.

uschindler pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/solr.git


The following commit(s) were added to refs/heads/main by this push:
     new 84a542c  SOLR-15839: Don't mock oal.search.Sort class (why the hell 
was this needed?) (#452)
84a542c is described below

commit 84a542cb4c275038e562384e5c01a2ea39894c07
Author: Uwe Schindler <uschind...@apache.org>
AuthorDate: Wed Dec 8 23:48:47 2021 +0100

    SOLR-15839: Don't mock oal.search.Sort class (why the hell was this 
needed?) (#452)
---
 .../test/org/apache/solr/handler/component/MockSortSpecBuilder.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/solr/core/src/test/org/apache/solr/handler/component/MockSortSpecBuilder.java 
b/solr/core/src/test/org/apache/solr/handler/component/MockSortSpecBuilder.java
index 3c65080..6a8194d 100644
--- 
a/solr/core/src/test/org/apache/solr/handler/component/MockSortSpecBuilder.java
+++ 
b/solr/core/src/test/org/apache/solr/handler/component/MockSortSpecBuilder.java
@@ -34,8 +34,7 @@ public class MockSortSpecBuilder {
     }
 
     public MockSortSpecBuilder withSortFields(SortField[] sortFields) {
-        Sort sort = Mockito.mock(Sort.class);
-        Mockito.when(sort.getSort()).thenReturn(sortFields);
+        Sort sort = new Sort(sortFields);
         Mockito.when(sortSpec.getSort()).thenReturn(sort);
         return this;
     }