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

E2e: backup of nicknames, blocked users, ENS #13238

Merged
merged 1 commit into from
Apr 1, 2022

Conversation

qoqobolo
Copy link
Contributor

This test adds checks for backup of blocked and ENS users, nicknames for contact and non-contact users.

@qoqobolo qoqobolo added the tests label Mar 31, 2022
@qoqobolo qoqobolo self-assigned this Mar 31, 2022
@qoqobolo qoqobolo requested a review from churik as a code owner March 31, 2022 08:30
@status-github-bot status-github-bot bot added this to REVIEW in Pipeline for QA Mar 31, 2022
@status-im-auto
Copy link
Member

status-im-auto commented Mar 31, 2022

Jenkins Builds

Click to see older builds (6)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 645109b #1 2022-03-31 08:39:46 ~8 min android 📦apk 📲
✔️ 645109b #1 2022-03-31 08:40:33 ~9 min android-e2e 📦apk 📲
✔️ 645109b #1 2022-03-31 08:46:07 ~15 min ios 📦ipa 📲
✔️ df47ffd #2 2022-04-01 10:13:51 ~10 min android 📦apk 📲
✔️ df47ffd #2 2022-04-01 10:14:02 ~10 min android-e2e 📦apk 📲
✔️ df47ffd #2 2022-04-01 10:21:24 ~17 min ios 📦ipa 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 335ea9e #3 2022-04-01 10:22:33 ~12 min android 📦apk 📲
✔️ 335ea9e #3 2022-04-01 10:22:54 ~12 min android-e2e 📦apk 📲
✔️ 335ea9e #3 2022-04-01 10:23:49 ~13 min ios 📦ipa 📲
✔️ 78864c0 #4 2022-04-01 10:31:58 ~13 min ios 📦ipa 📲

@status-im-auto
Copy link
Member

100% of end-end tests have passed

Total executed tests: 1
Failed tests: 0
Passed tests: 1

Passed tests (1)

Click to expand

1. test_backup_of_contacts, id: 700702
Device sessions

Copy link
Member

@churik churik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, squash your commits after fixes.
Nice job! 👍

home.back_button.click()

home.just_fyi('Block user')
from tests.users import chat_users
Copy link
Member

@churik churik Mar 31, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better to move it to the top of the file, we mostly import modules there

profile.element_by_text(basic_user["username"])
if not profile.element_by_text(basic_user['username']).is_element_displayed():
self.driver.fail("Contact was not backed up!")
profile.element_by_text(nickname)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this row does nothing; can be removed

self.errors.append('Nickname of contact was not backed up')

sign_in.just_fyi('Check backup of ENS contact')
profile.element_by_text('@%s' % ens_user['ens'])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this row does nothing; can be removed

@qoqobolo qoqobolo force-pushed the tests/test-extended-contacts-backup branch from df47ffd to 335ea9e Compare April 1, 2022 10:10
@qoqobolo qoqobolo force-pushed the tests/test-extended-contacts-backup branch from 335ea9e to 78864c0 Compare April 1, 2022 10:18
@qoqobolo qoqobolo merged commit 78864c0 into develop Apr 1, 2022
Pipeline for QA automation moved this from REVIEW to DONE Apr 1, 2022
@qoqobolo qoqobolo deleted the tests/test-extended-contacts-backup branch April 1, 2022 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants