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

test(chat): Can do a mention in a chat #6961

Merged
merged 1 commit into from
Aug 16, 2022
Merged

test(chat): Can do a mention in a chat #6961

merged 1 commit into from
Aug 16, 2022

Conversation

noeliaSD
Copy link
Contributor

@noeliaSD noeliaSD commented Aug 10, 2022

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:

  • 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.

Affected areas

  • tst_chatFlow

Screenshot of functionality (including design for comparison)

  • Can do mention:
Screen.Recording.2022-08-11.at.09.37.32.mov
  • Cannot do mention:
Screen.Recording.2022-08-11.at.09.39.00.mov

@status-im-auto
Copy link
Member

status-im-auto commented Aug 10, 2022

Jenkins Builds

Click to see older builds (23)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ d70afec #1 2022-08-10 16:02:05 ~8 min macos 📦dmg
✔️ d70afec #1 2022-08-10 16:21:50 ~28 min linux 📦tgz
d70afec #1 2022-08-10 16:35:25 ~41 min e2e 📄log
✔️ d70afec #1 2022-08-10 16:46:25 ~52 min windows 📦exe
d70afec #1 2022-08-10 16:53:48 ~4 min macos 📄log
✔️ d70afec #1 2022-08-10 17:01:44 ~13 min linux 📦tgz
✔️ d70afec #2 2022-08-10 17:13:45 ~23 min windows 📦exe
490df62 #2 2022-08-11 07:51:42 ~4 min macos 📄log
✔️ 490df62 #2 2022-08-11 07:58:09 ~10 min linux 📦tgz
490df62 #2 2022-08-11 08:13:28 ~25 min e2e 📄log
✔️ 490df62 #3 2022-08-11 08:13:49 ~26 min windows 📦exe
✔️ a98d69d #3 2022-08-11 08:14:03 ~7 min macos 📦dmg
✔️ a98d69d #3 2022-08-11 08:17:27 ~11 min linux 📦tgz
✔️ a98d69d #4 2022-08-11 08:27:11 ~21 min windows 📦exe
a98d69d #3 2022-08-11 08:37:11 ~31 min e2e 📄log
✔️ bc16758 #4 2022-08-16 09:08:04 ~8 min macos 📦dmg
✔️ bc16758 #4 2022-08-16 09:10:07 ~10 min linux 📦tgz
✔️ bc16758 #5 2022-08-16 09:22:54 ~23 min windows 📦exe
bc16758 #4 2022-08-16 09:23:22 ~23 min e2e 📄log
✔️ e6fb96a #5 2022-08-16 09:12:53 ~9 min macos 📦dmg
✔️ e6fb96a #5 2022-08-16 09:14:31 ~10 min linux 📦tgz
✔️ e6fb96a #6 2022-08-16 09:25:31 ~21 min windows 📦exe
e6fb96a #5 2022-08-16 09:28:32 ~24 min e2e 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ f4f2f9a #6 2022-08-16 09:48:25 ~8 min macos 📦dmg
✔️ f4f2f9a #6 2022-08-16 09:51:38 ~11 min linux 📦tgz
✔️ f4f2f9a #7 2022-08-16 10:05:39 ~25 min windows 📦exe
f4f2f9a #6 2022-08-16 10:14:32 ~34 min e2e 📄log
✔️ d5b8730 #7 2022-08-16 10:21:57 ~8 min macos 📦dmg
✔️ d5b8730 #7 2022-08-16 10:24:28 ~10 min linux 📦tgz
✔️ d5b8730 #8 2022-08-16 10:37:25 ~23 min windows 📦exe
d5b8730 #7 2022-08-16 10:37:54 ~24 min e2e 📄log
d5b8730 #8 2022-08-16 11:33:26 ~40 min e2e 📄log

@noeliaSD
Copy link
Contributor Author

Please @alaibe , @jrainville test them locally and ensure the mailserver is working properly (chat history is loaded), do the review and before integrating I will comment them out bc it is weak. WDYT?

Indeed, we should need to comment out all tst_chatFlow bc I've added a validation to ensure that when we are joining a chat room, the chat is loaded by validating the history is loaded (before, we were only entering into the room but no validation on it).

@noeliaSD noeliaSD changed the title WIP - test(chat): Can do a mention in a chat test(chat): Can do a mention in a chat Aug 11, 2022
@noeliaSD noeliaSD marked this pull request as ready for review August 11, 2022 08:11
Copy link
Member

@jrainville jrainville left a 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

test/ui-test/src/drivers/SquishDriver.py Outdated Show resolved Hide resolved
test/ui-test/src/drivers/SquishDriver.py Outdated Show resolved Hide resolved
test/ui-test/src/screens/StatusChatScreen.py Outdated Show resolved Hide resolved
test/ui-test/src/screens/StatusChatScreen.py Show resolved Hide resolved
@noeliaSD noeliaSD force-pushed the feat/issue-6879 branch 3 times, most recently from e6fb96a to f4f2f9a Compare August 16, 2022 09:39
`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
@noeliaSD noeliaSD merged commit 1334fbb into master Aug 16, 2022
@noeliaSD noeliaSD deleted the feat/issue-6879 branch August 16, 2022 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

can do a mention in a chat
4 participants