feat: Test cases for Scheduled Message feature #183
Conversation
There was a problem hiding this comment.
Thanks @yasserfaraazkhan! Added some comments on additional cases we want to include.
Also, for the first version I believe we will be supporting moving a draft to a scheduled post. Is this correct @harshilsharma63 ? If so, we need to include test cases for add/remove schedule to drafts
data/test-cases/channels/schedule-messages/1-Create-Schedule-Messages.md
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/4-Failed-Schedule-Message.md
Outdated
Show resolved
Hide resolved
| 1. Log into User's account. | ||
| 2. Type a message in the GM/Channel post text box. | ||
| 3. Click on the Dropdown beside Send button. | ||
| 4. Choose a pre-defined Date from the options. |
There was a problem hiding this comment.
Can we verify the timezone behavior for scheduling? I believe the expected behavior is the date and time dialog is in the users configured timezone, can you confirm @harshilsharma63 ?
There was a problem hiding this comment.
Thats correct, it should be in the user's configured TZ, not the timezone of the browser.
data/test-cases/channels/schedule-messages/1-Create-Schedule-Messages.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Nice, @yasserfaraazkhan! I added a bunch of nitpicky suggestions, plus I've started auditing the priorities set on our test cases, so these should be P3 in our custom priority field, as noted. Approving since the substance looks great; you can decide about my suggestions. :)
data/test-cases/channels/schedule-messages/1-Create-Schedule-Messages.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/1-Create-Schedule-Messages.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/6-Add-Schedule-to-Draft.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/7-Remove-Schedule-and-Move-to-Draft.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/7-Remove-Schedule-and-Move-to-Draft.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/7-Remove-Schedule-and-Move-to-Draft.md
Outdated
Show resolved
Hide resolved
data/test-cases/channels/schedule-messages/7-Remove-Schedule-and-Move-to-Draft.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Nice, @yasserfaraazkhan! I added a bunch of nitpicky suggestions, plus I've started auditing the priorities set on our test cases, so I marked that these should be P3 in our custom priority field. Approving since the substance looks great on the test cases; you can decide about my suggestions. :)
JIRA: https://mattermost.atlassian.net/browse/MM-60375