From 273a96047158a7c2e465226bfec3767493a4cba4 Mon Sep 17 00:00:00 2001 From: Anastasiya Semenkevich Date: Mon, 7 Oct 2024 14:45:43 +0300 Subject: [PATCH] chore: update timeouts --- test/e2e/gui/components/authenticate_popup.py | 2 +- .../gui/components/community/community_channel_popups.py | 3 ++- .../e2e/gui/components/onboarding/before_started_popup.py | 3 ++- test/e2e/gui/components/onboarding/beta_consent_popup.py | 3 ++- test/e2e/gui/components/settings/block_user_popup.py | 3 ++- .../components/settings/confirm_switch_waku_mode_popup.py | 1 + test/e2e/gui/components/settings/sign_out_popup.py | 3 ++- test/e2e/gui/components/signing_phrase_popup.py | 3 ++- .../gui/components/wallet/remove_wallet_account_popup.py | 3 ++- test/e2e/gui/components/wallet/wallet_account_popups.py | 8 +++++--- test/e2e/gui/main_window.py | 3 ++- test/e2e/gui/screens/community_settings.py | 3 ++- test/e2e/gui/screens/onboarding.py | 4 ++-- test/e2e/scripts/utils/local_system.py | 2 ++ 14 files changed, 29 insertions(+), 15 deletions(-) diff --git a/test/e2e/gui/components/authenticate_popup.py b/test/e2e/gui/components/authenticate_popup.py index 13c69c7750..dbd5325020 100644 --- a/test/e2e/gui/components/authenticate_popup.py +++ b/test/e2e/gui/components/authenticate_popup.py @@ -27,7 +27,7 @@ class AuthenticatePopup(BasePopup): def authenticate(self, password: str): self._password_text_edit.type_text(password) # TODO https://github.com/status-im/status-desktop/issues/15345 - self._primary_button.click(timeout=10) + self._primary_button.click(timeout=60) self._authenticate_button.wait_until_hidden(10000) @allure.step('Check if authenticate button is present') diff --git a/test/e2e/gui/components/community/community_channel_popups.py b/test/e2e/gui/components/community/community_channel_popups.py index eae5e3581a..b0ab618fa6 100644 --- a/test/e2e/gui/components/community/community_channel_popups.py +++ b/test/e2e/gui/components/community/community_channel_popups.py @@ -48,7 +48,8 @@ class NewChannelPopup(ChannelPopup): raise err def save(self): - self._save_create_button.click(timeout=5) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._save_create_button.click(timeout=30) class EditChannelPopup(ChannelPopup): diff --git a/test/e2e/gui/components/onboarding/before_started_popup.py b/test/e2e/gui/components/onboarding/before_started_popup.py index d38f75b446..aec1c1cc9f 100644 --- a/test/e2e/gui/components/onboarding/before_started_popup.py +++ b/test/e2e/gui/components/onboarding/before_started_popup.py @@ -32,5 +32,6 @@ class BeforeStartedPopUp(BasePopup): assert self._terms_of_use_checkBox.checkState != 0, f"ToU checkbox is not checked" assert self._terms_of_use_link.is_visible, f"Terms of use link is missing" assert self._privacy_policy_link.is_visible, f"Privacy Policy link is missing" - self._get_started_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._get_started_button.click(timeout=60) self.wait_until_hidden() diff --git a/test/e2e/gui/components/onboarding/beta_consent_popup.py b/test/e2e/gui/components/onboarding/beta_consent_popup.py index 4796969aea..f3f715206b 100644 --- a/test/e2e/gui/components/onboarding/beta_consent_popup.py +++ b/test/e2e/gui/components/onboarding/beta_consent_popup.py @@ -19,5 +19,6 @@ class BetaConsentPopup(QObject): def confirm(self): self._agree_to_use_checkbox.set(True) self._ready_to_use_checkbox.set(True) - self._ready_to_use_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._ready_to_use_button.click(timeout=60) self.wait_until_hidden() diff --git a/test/e2e/gui/components/settings/block_user_popup.py b/test/e2e/gui/components/settings/block_user_popup.py index 36720cc866..3d0db74360 100644 --- a/test/e2e/gui/components/settings/block_user_popup.py +++ b/test/e2e/gui/components/settings/block_user_popup.py @@ -24,7 +24,8 @@ class BlockUserPopup(BasePopup): @allure.step('Block user') def block(self): - self._block_user_button.click(timeout=5) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._block_user_button.click(timeout=60) @allure.step('Get warning text') def get_warning_text(self) -> str: diff --git a/test/e2e/gui/components/settings/confirm_switch_waku_mode_popup.py b/test/e2e/gui/components/settings/confirm_switch_waku_mode_popup.py index c81dcc3de4..a5bcfc7af4 100644 --- a/test/e2e/gui/components/settings/confirm_switch_waku_mode_popup.py +++ b/test/e2e/gui/components/settings/confirm_switch_waku_mode_popup.py @@ -19,4 +19,5 @@ class SwitchWakuModePopup(BasePopup): @allure.step('Click i understand button') def confirm(self): + # TODO https://github.com/status-im/status-desktop/issues/15345 self._i_understand_button.click(timeout=None) diff --git a/test/e2e/gui/components/settings/sign_out_popup.py b/test/e2e/gui/components/settings/sign_out_popup.py index 473e78eb72..c548abd07a 100644 --- a/test/e2e/gui/components/settings/sign_out_popup.py +++ b/test/e2e/gui/components/settings/sign_out_popup.py @@ -14,7 +14,8 @@ class SignOutPopup(BasePopup): @allure.step('Click sign out and quit button') def sign_out_and_quit(self, attempts: int = 2): try: - self._sign_out_and_quit_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._sign_out_and_quit_button.click(timeout=60) except Exception as ec: if attempts: self.sign_out_and_quit(attempts-1) diff --git a/test/e2e/gui/components/signing_phrase_popup.py b/test/e2e/gui/components/signing_phrase_popup.py index 3434e9bcc0..a18caf3833 100644 --- a/test/e2e/gui/components/signing_phrase_popup.py +++ b/test/e2e/gui/components/signing_phrase_popup.py @@ -13,7 +13,8 @@ class SigningPhrasePopup(QObject): @allure.step('Confirm signing phrase in popup') def confirm_phrase(self): - self._ok_got_it_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._ok_got_it_button.click(timeout=60) SigningPhrasePopup().wait_until_hidden() @allure.step('Verify if the signing phrase popup is visible') diff --git a/test/e2e/gui/components/wallet/remove_wallet_account_popup.py b/test/e2e/gui/components/wallet/remove_wallet_account_popup.py index e956b93092..04c6fa996c 100644 --- a/test/e2e/gui/components/wallet/remove_wallet_account_popup.py +++ b/test/e2e/gui/components/wallet/remove_wallet_account_popup.py @@ -22,7 +22,8 @@ class RemoveWalletAccountPopup(BasePopup): @allure.step('Confirm removing account') def confirm(self): - self._confirm_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._confirm_button.click(timeout=60) self._confirm_button.wait_until_hidden() @allure.step('Agree and confirm removing account') diff --git a/test/e2e/gui/components/wallet/wallet_account_popups.py b/test/e2e/gui/components/wallet/wallet_account_popups.py index a1e3a5c4fd..763a57bc9d 100644 --- a/test/e2e/gui/components/wallet/wallet_account_popups.py +++ b/test/e2e/gui/components/wallet/wallet_account_popups.py @@ -174,7 +174,7 @@ class AccountPopup(BasePopup): assert driver.waitFor(lambda: self.is_save_changes_button_enabled(), configs.timeouts.UI_LOAD_TIMEOUT_MSEC) # TODO https://github.com/status-im/status-desktop/issues/15345 - self._add_save_account_confirmation_button.click(timeout=10) + self._add_save_account_confirmation_button.click(timeout=60) return self @allure.step('Get enabled state of (add account / save changes) button') @@ -275,7 +275,8 @@ class AddNewAccountPopup(BasePopup): @allure.step('Click continue') def click_continue(self): - self._continue_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._continue_button.click(timeout=60) return self @allure.step('Import new seed phrase and continue') @@ -288,7 +289,8 @@ class AddNewAccountPopup(BasePopup): @allure.step('Enter new seed phrase') def enter_new_seed_phrase(self, seed_phrase_words: list): - self._import_seed_phrase_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._import_seed_phrase_button.click(timeout=60) if len(seed_phrase_words) == 12: self._seed_phrase_12_words_button.click() elif len(seed_phrase_words) == 18: diff --git a/test/e2e/gui/main_window.py b/test/e2e/gui/main_window.py index 3cea080bbe..525ba63274 100644 --- a/test/e2e/gui/main_window.py +++ b/test/e2e/gui/main_window.py @@ -154,7 +154,8 @@ class LeftPanel(QObject): @allure.step('Open Wallet section') def open_wallet(self, attempts: int = 3) -> WalletScreen: - self._wallet_button.click(timeout=15) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._wallet_button.click(timeout=30) try: return WalletScreen() except Exception as ex: diff --git a/test/e2e/gui/screens/community_settings.py b/test/e2e/gui/screens/community_settings.py index c53c7f055f..b4a30d6a69 100644 --- a/test/e2e/gui/screens/community_settings.py +++ b/test/e2e/gui/screens/community_settings.py @@ -580,7 +580,8 @@ class PermissionsSettingsView(QObject): @allure.step('Click create permission') def create_permission(self): - self._create_permission_button.click(timeout=10) + # TODO https://github.com/status-im/status-desktop/issues/15345 + self._create_permission_button.click(timeout=30) self._create_permission_button.wait_until_hidden() @allure.step('Open Who holds context menu') diff --git a/test/e2e/gui/screens/onboarding.py b/test/e2e/gui/screens/onboarding.py index 906b20e5db..57f0b47c1c 100755 --- a/test/e2e/gui/screens/onboarding.py +++ b/test/e2e/gui/screens/onboarding.py @@ -37,7 +37,7 @@ class AllowNotificationsView(QObject): @allure.step("Start using Status") def start_using_status(self): # TODO https://github.com/status-im/status-desktop/issues/15345 - self._start_using_status_button.click(timeout=15) + self._start_using_status_button.click(timeout=30) self.wait_until_hidden() @@ -411,7 +411,7 @@ class YourEmojihashAndIdenticonRingView(OnboardingView): @allure.step('Click next in your emojihash and identicon ring view') def next(self): # TODO https://github.com/status-im/status-desktop/issues/15345 - self._next_button.click(timeout=15) + self._next_button.click(timeout=30) time.sleep(1) if configs.system.get_platform() == "Darwin": return AllowNotificationsView().wait_until_appears() diff --git a/test/e2e/scripts/utils/local_system.py b/test/e2e/scripts/utils/local_system.py index 41328d0bb3..d659bc1055 100644 --- a/test/e2e/scripts/utils/local_system.py +++ b/test/e2e/scripts/utils/local_system.py @@ -45,6 +45,7 @@ def kill_process(pid): except Exception as e: print(f"Failed to terminate process {pid}: {e}") + @allure.step('System execute command') def execute( command: list, @@ -75,6 +76,7 @@ def run( check=True ) + @allure.step('Get pid by process name') def get_pid_by_process_name(name): pid_list = []