-
Notifications
You must be signed in to change notification settings - Fork 985
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 for mobile data sync #9085
e2e for mobile data sync #9085
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds (18)
|
71db00c
to
9eb71a6
Compare
if not chat.chat_element_by_text(message).is_element_displayed(): | ||
self.errors.append("Message was not sent!") |
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.
I'd check here the message status after it sent rather than message itself
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.
It makes more sense on IOS, on Android you can't even send a message from offline.
But I can add check
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.
also, please add self.verify_no_errors() in the end of the test
Well done, @churik! |
self.errors.append("Mobile data is enabled by default") | ||
if not profile_view.ask_me_when_on_mobile_network.attribute_value("checked"): | ||
self.errors.append("'Ask me when on mobile network' is not enabled by default") | ||
|
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.
please, add self.verify_no_errors()
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.
thx, added.
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.
There will be no verification in a test which does self.errors.append('whatever')
if self.verify_no_errors()
is not added in the end of this test
1399d78
to
8b57a9c
Compare
@antdanchenko thanks for stepping in! |
Signed-off-by: Churikova Tetiana <churikova.tm@gmail.com>
8b57a9c
to
da4fb01
Compare
Added 3 tests about mobile sync after fix #8973