-
Notifications
You must be signed in to change notification settings - Fork 78
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
test(chat): Can do a mention in a chat #6961
Conversation
Jenkins BuildsClick to see older builds (23)
|
d70afec
to
490df62
Compare
490df62
to
a98d69d
Compare
Please @alaibe , @jrainville test them locally and ensure the Indeed, we should need to comment out all |
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.
Great work. I only left message and name suggestions
e6fb96a
to
f4f2f9a
Compare
`tst_chatFlow`: - Commented out bc of `mailserver` issues and then weak. - Added basic test scenario for sending a mention in a public chat with needed validations. - Added basic test scenario to check a mention cannot be done if it is a non existing user. `StatusChatScreen` updates: - Updated join room method to validate the chat is loaded. - Added methods for doing and verifying a mention. `SquishDriver` updates: - Added support in `SquishDriver` to click into a link in a text or label component. - Minor function renames. Closes #6879
f4f2f9a
to
d5b8730
Compare
Closes #6879
IMPORTANT: Weak test because it relys on
mailserver
and if the chat history is not loaded, it will fail!!!What does the PR do
tst_chatFlow
:StatusChatScreen
updates:SquishDriver
updates:SquishDriver
to click into a link in a text or label component.Affected areas
tst_chatFlow
Screenshot of functionality (including design for comparison)
Screen.Recording.2022-08-11.at.09.37.32.mov
Screen.Recording.2022-08-11.at.09.39.00.mov