-
Notifications
You must be signed in to change notification settings - Fork 981
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: new tests are automated #13510
e2e: new tests are automated #13510
Conversation
Jenkins BuildsClick to see older builds (18)
|
100% of end-end tests have passed
Not executed tests (175)Passed tests (5)Click to expand
|
100% of end-end tests have passed
Not executed tests (173)Passed tests (7)Click to expand
|
100% of end-end tests have passed
Not executed tests (175)Passed tests (5)Click to expand
|
@@ -200,6 +200,68 @@ def test_profile_chat_two_minutes_gap_in_public_and_no_gap_in_1_1_chat_and_group | |||
chat_view.back_button.click() | |||
self.errors.verify_no_errors() | |||
|
|||
@marks.testrail_id(702367) | |||
def test_chat_mute_unmute_contact(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add push in test name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanx @churik, renamed
self.errors.verify_no_errors() | ||
|
||
@marks.testrail_id(702368) | ||
def test_chat_group_chat_reopen_app_on_last_viewed_chat(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test is not related to group_chat, better to remove it from name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanx @churik, renamed
test/appium/views/chat_view.py
Outdated
def __init__(self, driver): | ||
super(ProfileUnmuteContactButton, self).__init__(driver, accessibility_id="Unmute-item-button") | ||
|
||
def click(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure if we need to define separate class for re-defining only this method, as scroll_and_click
method exists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanx @churik, refactored
0% of end-end tests have passed
Not executed tests (175)Failed tests (5)Click to expand
|
0% of end-end tests have passed
Not executed tests (2)Failed tests (3)Click to expand
|
100% of end-end tests have passed
Not executed tests (175)Passed tests (5)Click to expand
|
71ec552
to
71313a1
Compare
71313a1
to
ff190c1
Compare
Added following checks:
https://ethstatus.testrail.net/index.php?/cases/view/702132
https://ethstatus.testrail.net/index.php?/cases/view/702135
https://ethstatus.testrail.net/index.php?/cases/view/702136
https://ethstatus.testrail.net/index.php?/cases/view/702344