From 3945404059d5bb480f64e9be71e6dadd62f37476 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Date: Wed, 9 Jul 2025 10:23:17 +0100
Subject: [PATCH] Bug 36947: (QA follow-up) Add FacetSortingLocale to Search.t
 mock preferences

The test was failing because the new FacetSortingLocale system preference
added was not included in the mock preference handler in t/db_dependent/Search.t.
This caused warnings about the unknown preference when the _sort_facets_zebra
function was called.

This commit adds the FacetSortingLocale preference to the mock, returning
'default' as the value, which matches the expected fallback behavior of
the new unicode collation sorting feature.

Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 21a217d5bfc61eebd4109f7aa3f0a4af82cad7b7)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
---
 t/db_dependent/Search.t | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/t/db_dependent/Search.t b/t/db_dependent/Search.t
index 9249bca11d..beaf6cb43a 100755
--- a/t/db_dependent/Search.t
+++ b/t/db_dependent/Search.t
@@ -170,6 +170,8 @@ $contextmodule->mock(
             return q{};
         } elsif ( $pref eq 'COinSinOPACResults' ) {
             return q{};
+        } elsif ( $pref eq 'FacetSortingLocale' ) {
+            return 'default';
         } else {
             warn
                 "The syspref $pref was requested but I don't know what to say; this indicates that the test requires updating"