Revert "VEND58: Sort assert handles both (in)sensitive order by"

This reverts commit 3755f95.
nuxeo · Jun 25, 2024 · 79c0be9 · 79c0be9
1 parent 3755f95
commit 79c0be9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/...scim2.testsuite.core/src/main/java/org/wso2/scim2/testsuite/core/tests/GroupTestImpl.java b/...scim2.testsuite.core/src/main/java/org/wso2/scim2/testsuite/core/tests/GroupTestImpl.java
@@ -388,9 +388,7 @@ private boolean isGroupListSorted(ArrayList<Group> returnedGroups) throws Charon
 
         boolean sorted = true;
         for (int i = 1; i < returnedGroups.size(); i++) {
-            String previous = returnedGroups.get(i - 1).getId();
-            String next  = returnedGroups.get(i).getId();
-            if (previous.compareTo(next) > 0 && previous.compareToIgnoreCase(next) > 0) {
+            if (returnedGroups.get(i - 1).getId().compareTo(returnedGroups.get(i).getId()) > 0) {
                 sorted = false;
             }
         }

diff --git a/....scim2.testsuite.core/src/main/java/org/wso2/scim2/testsuite/core/tests/UserTestImpl.java b/....scim2.testsuite.core/src/main/java/org/wso2/scim2/testsuite/core/tests/UserTestImpl.java
@@ -231,9 +231,7 @@ private boolean isUserListSorted(ArrayList<User> userList) throws CharonExceptio
 
         boolean sorted = true;
         for (int i = 1; i < userList.size(); i++) {
-            String previous = userList.get(i - 1).getId();
-            String next  = userList.get(i).getId();
-            if (previous.compareTo(next) > 0 && previous.compareToIgnoreCase(next) > 0) {
+            if (userList.get(i - 1).getId().compareTo(userList.get(i).getId()) > 0) {
                 sorted = false;
             }
         }