Skip to content
This repository was archived by the owner on Aug 13, 2024. It is now read-only.

Conversation

@Valentina1133
Copy link
Contributor

@Valentina1133 Valentina1133 commented Apr 8, 2024

Community admin can delete member's message and kick a member test automated
CI run (all tests):
https://ci.status.im/job/status-desktop/job/e2e/job/manual/1748/allure/

Screenshot 2024-04-08 at 15 07 18

@Valentina1133
Copy link
Contributor Author

Valentina1133 commented Apr 8, 2024

@anastasiyaig I placed this test in test_communities.py file, but now I think maybe it's better to create a separate file?
Also - we can add steps from this scenario to test_join_community_and_pin_unpin_message - it will reduce time of tests but the scenario will be much longer. What do you think?

@anastasiyaig
Copy link
Collaborator

file is fine i think! Lets keep it as is for now
pls resolve 1 conflict . Btw regarding it, i dont know how test is passing having wrong string for permission deletion (i fixed the string itself but before i changed that - tests were not failing :D

DELETE_PERMISSION_TOAST = 'Community permission deleted'

we might want to check on the verification for the toasts in permission tests (separate task)

@Valentina1133
Copy link
Contributor Author

file is fine i think! Lets keep it as is for now pls resolve 1 conflict . Btw regarding it, i dont know how test is passing having wrong string for permission deletion (i fixed the string itself but before i changed that - tests were not failing :D

DELETE_PERMISSION_TOAST = 'Community permission deleted'

we might want to check on the verification for the toasts in permission tests (separate task)

Yes! You are right!

I think I know why. Because in permissions test we check that toast message from constants is in list of toast messages appeared in test, because there are several toast messages at the same time. So I believe we need to check it element by element using indexes to check every toast message separately.

@Valentina1133 Valentina1133 force-pushed the test/community_kick_member branch from 608f9a2 to a89bd0d Compare April 9, 2024 03:34
@anastasiyaig anastasiyaig force-pushed the test/community_kick_member branch from a89bd0d to 79e0647 Compare April 9, 2024 06:51
@anastasiyaig anastasiyaig merged commit 9df47c9 into master Apr 9, 2024
@anastasiyaig anastasiyaig deleted the test/community_kick_member branch April 9, 2024 07:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Community: can kick a member and delete another member message if admin

2 participants