From fbd2326841005c1a288c65502dc038f26923ed65 Mon Sep 17 00:00:00 2001
From: Tom Clegg <tom@curoverse.com>
Date: Fri, 7 Nov 2014 15:55:15 -0500
Subject: [PATCH] 4434: Assume only that if a<b are lowercase ascii, aX<bY for
 any X, Y, collation.

---
 .../arvados/v1/groups_controller_test.rb      | 21 ++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/services/api/test/functional/arvados/v1/groups_controller_test.rb b/services/api/test/functional/arvados/v1/groups_controller_test.rb
index f83bf53c63..294afc292f 100644
--- a/services/api/test/functional/arvados/v1/groups_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/groups_controller_test.rb
@@ -141,15 +141,30 @@ class Arvados::V1::GroupsControllerTest < ActionController::TestCase
         filters: [['uuid', 'is_a', "arvados#collection"]],
         order: "collections.name #{order}"
       }
-      sorted_entries = json_response['items'].collect { |item| item["name"] }
+      sorted_names = json_response['items'].collect { |item| item["name"] }
+      # Here we avoid assuming too much about the database
+      # collation. Both "alice"<"Bob" and "alice">"Bob" can be
+      # correct. Hopefully it _is_ safe to assume that if "a" comes
+      # before "b" in the ascii alphabet, "aX">"bY" is never true for
+      # any strings X and Y.
+      reliably_sortable_names = sorted_names.select do |name|
+        name[0] >= 'a' and name[0] <= 'z'
+      end.uniq do |name|
+        name[0]
+      end
+      # Array#& is documented to preserve order of sorted_names.
+      sorted_names &= reliably_sortable_names
+      actually_checked_anything = false
       previous = nil
-      sorted_entries.each do |entry|
+      sorted_names.each do |entry|
         if previous
           assert_operator(previous, operator, entry,
-                          "Entries sorted incorrectly. Perhaps the application and database have mismatched locale settings?")
+                          "Entries sorted incorrectly.")
+          actually_checked_anything = true
         end
         previous = entry
       end
+      assert actually_checked_anything, "Didn't even find two names to compare."
     end
   end
 
-- 
2.30.2