I've added tests to the backend of chats.
Additionally, I've added some authorization checks: At the moment, everybody can send or mark a message as read. After this change, only the participants of the list is allowed to do these tasks.
Merge AFTER #122
Connected Issue: #91
Types of changes
What types of changes does your code introduce to Appium?
Put an x in the boxes that apply
[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.
[x] Please make sure all test have passed and you have not introduced new compiler warnings
[x] Have you added necessary documentation (README) (if appropriate)
[x] Keep pull requests small and please open seperate pull-requests for different issues so they can be easily reviewed
[x] Please avoid making changes to the overall architecture (unless discussed before)
Thank you for your contribution! We appreciate your help!
Proposed changes
I've added tests to the backend of chats. Additionally, I've added some authorization checks: At the moment, everybody can send or mark a message as read. After this change, only the participants of the list is allowed to do these tasks.
Merge AFTER #122 Connected Issue: #91
Types of changes
What types of changes does your code introduce to Appium? Put an
xin the boxes that applyChecklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Thank you for your contribution! We appreciate your help!