-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Adjust how message notification test works #354
Comments
🤖 meep morp! This Issue is now marked as stale because there has been no activity for a while. 🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days. 💡 To unstale this Issue please push any commits that are ready or provide an update in the comments. |
🤖 meep morp! This issue has been stale for 7 days so it's being automaticaly unassigned and will return to the backlog. 💡 Once in the Backlog this issue will be available for anyone to take up - you can request it again if you have contributions ready to submit. |
🤖 meep morp! This Issue is now marked as stale because there has been no activity for a while. 🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days. 💡 To unstale this Issue please push any commits that are ready or provide an update in the comments. |
🤖 meep morp! This issue has been stale for 7 days so it's being automaticaly unassigned and will return to the backlog. 💡 Once in the Backlog this issue will be available for anyone to take up - you can request it again if you have contributions ready to submit. |
🤖 meep morp! This Issue is now marked as stale because there has been no activity for a while. 🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days. 💡 To unstale this Issue please push any commits that are ready or provide an update in the comments. |
🤖 meep morp! This issue has been stale for 7 days so it's being automaticaly unassigned and will return to the backlog. 💡 Once in the Backlog this issue will be available for anyone to take up - you can request it again if you have contributions ready to submit. |
🤖 meep morp! This Issue is now marked as stale because there has been no activity for a while. 🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days. 💡 To unstale this Issue please push any commits that are ready or provide an update in the comments. |
🤖 meep morp! This issue has been stale for 7 days so it's being automaticaly unassigned and will return to the backlog. 💡 Once in the Backlog this issue will be available for anyone to take up - you can request it again if you have contributions ready to submit. |
Expected Behavior
if notifications are working then the test should always pass.
Current Behavior
It runs properly if the the database has just been reset but if the site has been used a bit and the conversations it initially looks at have been read or had new messages sent or had their items reserved then it will break.
Any other notes
ideally a new test would find a conversation without looking for text that might not be the most recent message anymore and will mark things as unread to test the feature works rather than just rely on them being unread by default
Assignment
This issue is free for anyone to take
The text was updated successfully, but these errors were encountered: