From 91d58f08ab7e49def54419d47b6cf067269e93ba Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 22 May 2024 10:31:29 +0200 Subject: [PATCH] Bug 36916: Do not generate invalid JS or CSS from TestBuilder TestBuilder generates random strings for branches.opacuserjs and branches.opacusercss which produces invalid JS and CSS. Selenium has several warnings related to this: koha-selenium-1 | JavaScript error: http://koha:8080/cgi-bin/koha/opac-user.pl, line 1744: ReferenceError: CLYxPjQ152 is not defined koha-selenium-1 | JavaScript error: http://koha:8080/cgi-bin/koha/opac-search.pl, line 2069: ReferenceError: CLYxPjQ152 is not defined koha-selenium-1 | JavaScript error: http://koha:8080/cgi-bin/koha/opac-reserve.pl, line 1351: ReferenceError: CLYxPjQ152 is not defined Because of the following in the DOM This patch suggests to set to an empty string by default, to prevent random failure or inconsistent behaviours when testing the UI. Test plan: Run t/db_dependent/selenium/authentication.t and watch the selenium output. With this patch applied you will not see the "JavaScript error" lines Signed-off-by: David Nind Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer (cherry picked from commit a51b46d4bab2989486ea0be05e130066c666fa50) Signed-off-by: Fridolin Somers --- t/lib/TestBuilder.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/t/lib/TestBuilder.pm b/t/lib/TestBuilder.pm index 91fb518494..a3eee8f31f 100644 --- a/t/lib/TestBuilder.pm +++ b/t/lib/TestBuilder.pm @@ -615,6 +615,8 @@ sub _gen_default_values { }, Branch => { pickup_location => 0, + opacuserjs => q{}, + opacusercss => q{}, }, Reserve => { non_priority => 0, -- 2.39.5