From 5981f4bd13ac1849e513299500548b52c97ed409 Mon Sep 17 00:00:00 2001 From: Aditya Toshniwal Date: Mon, 9 Oct 2023 12:42:52 +0530 Subject: [PATCH] In GUI tests, do not try to set the binary path in preferences if it is blank --- .../feature_tests/pg_utilities_backup_restore_test.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/web/regression/feature_tests/pg_utilities_backup_restore_test.py b/web/regression/feature_tests/pg_utilities_backup_restore_test.py index 077d19d64..cc1479cdb 100644 --- a/web/regression/feature_tests/pg_utilities_backup_restore_test.py +++ b/web/regression/feature_tests/pg_utilities_backup_restore_test.py @@ -291,7 +291,8 @@ class PGUtilitiesBackupFeatureTest(BaseFeatureTest): server_types = default_binary_path.keys() path_already_set = True for serv in server_types: - if serv == 'pg' and server_version is not None: + if serv == 'pg' and server_version is not None and \ + default_binary_path['pg'] != '': path_input = \ self.page.find_by_xpath( "//div[span[text()='PostgreSQL {}']]" @@ -303,7 +304,8 @@ class PGUtilitiesBackupFeatureTest(BaseFeatureTest): self.page.clear_edit_box(path_input) path_input.click() path_input.send_keys(default_binary_path['pg']) - elif serv == 'ppas' and server_version is not None: + elif serv == 'ppas' and server_version is not None and \ + default_binary_path['ppas'] != '': path_input = \ self.page.find_by_xpath( "//div[span[text()='EDB Advanced Server {}']]"