DEV: Disable splash screen in system tests (#21840)

Why is this change required?

We had a system test failed right after visiting a URL because an
element that was supposed to be on screen isn't. In the screenshot
captured, the app was still stuck on the splash screen. We're not sure
if this has been causing flakiness in our system tests but there is no
good reason to enable splash screen which adds some overhead to each
page load.
This commit is contained in:
Alan Guo Xiang Tan 2023-05-31 07:16:22 +09:00 committed by GitHub
parent 906e053f32
commit e3e7b080c6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 0 deletions

View File

@ -27,6 +27,7 @@ module SystemHelpers
SiteSetting.external_system_avatars_enabled = false
SiteSetting.disable_avatar_education_message = true
SiteSetting.enable_user_tips = false
SiteSetting.splash_screen = false
end
def try_until_success(timeout: 2, frequency: 0.01)