Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#21615] fix: hide mercuryo provider #21626

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

mohsen-ghafouri
Copy link
Contributor

@mohsen-ghafouri mohsen-ghafouri commented Nov 14, 2024

fixes #21615

Summary

hide Mercuryo from the list of on-ramp providers
so that we stay compliant with UK law.

Note: it only has status-go changes

Areas that maybe impacted

  • Ways to buy asset modal

Result

status: ready

@status-im-auto
Copy link
Member

status-im-auto commented Nov 14, 2024

Jenkins Builds

Click to see older builds (19)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ f20319b #1 2024-11-14 14:26:56 ~5 min tests 📄log
✔️ f20319b #1 2024-11-14 14:30:28 ~9 min android-e2e 🤖apk 📲
✔️ f20319b #1 2024-11-14 14:31:32 ~10 min ios 📱ipa 📲
✔️ f20319b #1 2024-11-14 14:32:10 ~11 min android 🤖apk 📲
✔️ e5330a5 #2 2024-11-20 12:13:49 ~6 min tests 📄log
e5330a5 #2 2024-11-20 12:13:58 ~7 min android-e2e 📄log
e5330a5 #2 2024-11-20 12:14:34 ~7 min ios 📄log
e5330a5 #2 2024-11-20 12:15:09 ~8 min android 📄log
e5330a5 #3 2024-11-20 13:43:46 ~7 min ios 📄log
e5330a5 #3 2024-11-20 13:44:29 ~3 min android 📄log
e5330a5 #3 2024-11-20 13:44:51 ~3 min android-e2e 📄log
✔️ 115ba82 #3 2024-11-20 14:51:55 ~6 min tests 📄log
✔️ 115ba82 #4 2024-11-20 14:54:38 ~9 min android-e2e 🤖apk 📲
✔️ 115ba82 #4 2024-11-20 14:55:58 ~10 min ios 📱ipa 📲
✔️ 115ba82 #4 2024-11-20 14:56:23 ~10 min android 🤖apk 📲
✔️ dadb10b #4 2024-11-21 14:23:30 ~5 min tests 📄log
✔️ dadb10b #5 2024-11-21 14:26:50 ~9 min ios 📱ipa 📲
✔️ dadb10b #5 2024-11-21 14:27:05 ~9 min android-e2e 🤖apk 📲
✔️ dadb10b #5 2024-11-21 14:28:01 ~10 min android 🤖apk 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 4777713 #5 2024-11-21 15:46:10 ~5 min tests 📄log
✔️ 4777713 #6 2024-11-21 15:49:30 ~8 min android-e2e 🤖apk 📲
✔️ 4777713 #6 2024-11-21 15:51:47 ~10 min android 🤖apk 📲
✔️ 4777713 #6 2024-11-21 15:53:40 ~12 min ios 📱ipa 📲
ad0b7e7 #6 2024-11-22 10:29:28 ~4 min tests 📄log
✔️ ad0b7e7 #7 2024-11-22 10:32:37 ~7 min android-e2e 🤖apk 📲
✔️ ad0b7e7 #7 2024-11-22 10:33:22 ~8 min android 🤖apk 📲
✔️ ad0b7e7 #7 2024-11-22 10:33:45 ~8 min ios 📱ipa 📲
✔️ ad0b7e7 #7 2024-11-22 11:08:57 ~2 min tests 📄log

@status-im-auto
Copy link
Member

62% of end-end tests have passed

Total executed tests: 8
Failed tests: 2
Expected to fail tests: 1
Passed tests: 5
IDs of failed tests: 702745,703133 
IDs of expected to fail tests: 702843 

Failed tests (2)

Click to expand
  • Rerun failed tests

  • Class TestCommunityOneDeviceMerged:

    1. test_restore_multiaccount_with_waku_backup_remove_profile_switch, id: 703133

    Device 1: Find `Button` by `accessibility id`: `show-profiles`
    Device 1: Tap on found: Button

    critical/chats/test_public_chat_browsing.py:245: in test_restore_multiaccount_with_waku_backup_remove_profile_switch
        self.errors.verify_no_errors()
    base_test_case.py:192: in verify_no_errors
        pytest.fail('\n '.join([self.errors.pop(0) for _ in range(len(self.errors))]))
     zQ3...dWXh5 was not restored as a contact from waku backup!
    E    zQ3...Vacac was not restored as a contact from waku backup!
    E    admin_open was not restored from waku-backup!!
    E    member_open was not restored from waku-backup!!
    E    admin_closed was not restored from waku-backup!!
    E    member_closed was not restored from waku-backup!!
    



    Device sessions

    Class TestOneToOneChatMultipleSharedDevicesNewUi:

    1. test_1_1_chat_non_latin_messages_stack_update_profile_photo, id: 702745

    Device 2: Button element not found
    Device 2: Wait for element Button for max 30s and click when it is available

    Test setup failed: critical/chats/test_1_1_public_chats.py:24: in prepare_devices
        self.loop.run_until_complete(run_in_parallel(((self.device_1.create_user, {'enable_notifications': True,
    /usr/lib/python3.10/asyncio/base_events.py:649: in run_until_complete
        return future.result()
    __init__.py:52: in run_in_parallel
        returns.append(await k)
    /usr/lib/python3.10/concurrent/futures/thread.py:58: in run
        result = self.fn(*self.args, **self.kwargs)
    ../views/sign_in_view.py:249: in create_user
        self.not_now_button.wait_and_click()
    ../views/base_element.py:100: in wait_and_click
        self.wait_for_visibility_of_element(sec)
    ../views/base_element.py:147: in wait_for_visibility_of_element
        raise TimeoutException(
     Device 2: Button by xpath:`//*[@text='Not now']` is not found on the screen after wait_for_visibility_of_element
    



    Device sessions

    Expected to fail tests (1)

    Click to expand

    Class TestCommunityMultipleDeviceMerged:

    1. test_community_message_edit, id: 702843

    Test is not run, e2e blocker  
    

    [[reason: [NOTRUN] Skipped due to waku issue on staging fleet]]

    Passed tests (5)

    Click to expand

    Class TestWalletMultipleDevice:

    1. test_wallet_send_asset_from_drawer, id: 727230
    2. test_wallet_send_eth, id: 727229

    Class TestWalletOneDevice:

    1. test_wallet_add_remove_regular_account, id: 727231
    2. test_wallet_balance_mainnet, id: 740490

    Class TestCommunityOneDeviceMerged:

    1. test_community_copy_and_paste_message_in_chat_input, id: 702742
    Device sessions

    @Horupa-Olena
    Copy link

    @mohsen-ghafouri Thanks for your PR!
    Could you please resolve the issues with the failed builds?

    @VolodLytvynenko VolodLytvynenko self-assigned this Nov 20, 2024
    @mohsen-ghafouri
    Copy link
    Contributor Author

    Hi @Horupa-Olena, all good now.

    @Horupa-Olena
    Copy link

    @mohsen-ghafouri Thanks for your work!
    Since Mercuryo has been hidden from the app, the Recurrent tab is now empty, which doesn’t look great. Could you also hide this tab as part of this PR?
    photo_2024-11-21_10-35-32
    photo_2024-11-21_10-35-29

    @status-im-auto
    Copy link
    Member

    75% of end-end tests have passed

    Total executed tests: 8
    Failed tests: 1
    Expected to fail tests: 1
    Passed tests: 6
    
    IDs of failed tests: 703133 
    
    IDs of expected to fail tests: 702843 
    

    Failed tests (1)

    Click to expand
  • Rerun failed tests

  • Class TestCommunityOneDeviceMerged:

    1. test_restore_multiaccount_with_waku_backup_remove_profile_switch, id: 703133

    Device 1: Tap on found: Button
    # STEP: Check that removed user is not shown in the list anymore

    critical/chats/test_public_chat_browsing.py:240: in test_restore_multiaccount_with_waku_backup_remove_profile_switch
        self.errors.verify_no_errors()
    base_test_case.py:192: in verify_no_errors
        pytest.fail('\n '.join([self.errors.pop(0) for _ in range(len(self.errors))]))
     zQ3...dWXh5 was not restored as a contact from waku backup!
    E    zQ3...Vacac was not restored as a contact from waku backup!
    E    admin_open was not restored from waku-backup!!
    E    member_open was not restored from waku-backup!!
    E    admin_closed was not restored from waku-backup!!
    E    member_closed was not restored from waku-backup!!
    



    Device sessions

    Expected to fail tests (1)

    Click to expand

    Class TestCommunityMultipleDeviceMerged:

    1. test_community_message_edit, id: 702843

    Test is not run, e2e blocker  
    

    [[reason: [NOTRUN] Skipped due to waku issue on staging fleet]]

    Passed tests (6)

    Click to expand

    Class TestWalletOneDevice:

    1. test_wallet_add_remove_regular_account, id: 727231
    2. test_wallet_balance_mainnet, id: 740490

    Class TestCommunityOneDeviceMerged:

    1. test_community_copy_and_paste_message_in_chat_input, id: 702742
    Device sessions

    Class TestWalletMultipleDevice:

    1. test_wallet_send_asset_from_drawer, id: 727230
    2. test_wallet_send_eth, id: 727229

    Class TestOneToOneChatMultipleSharedDevicesNewUi:

    1. test_1_1_chat_non_latin_messages_stack_update_profile_photo, id: 702745
    Device sessions

    @mohsen-ghafouri
    Copy link
    Contributor Author

    Hi @Horupa-Olena, please test again. applied changes.

    @Horupa-Olena
    Copy link

    @mohsen-ghafouri Thanks for your work!
    Please resolve the conflicts so I can start testing.

    @mohsen-ghafouri
    Copy link
    Contributor Author

    @Horupa-Olena done

    @Horupa-Olena
    Copy link

    @mohsen-ghafouri Thank you! Everything looks good.
    This PR is ready to be merged.
    photo_2024-11-22_15-41-07

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    Status: MERGE
    Development

    Successfully merging this pull request may close these issues.

    🙈 Hide Mercuryo from the list of on-ramp providers
    6 participants