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

MM-28266: Cypress/E2E: Automate backlogs - Messaging > Edit (11 test cases) #6838

Merged
merged 7 commits into from
Oct 28, 2020

Conversation

catalintomai
Copy link
Contributor

Summary

Cypress/E2E: Automate backlogs - Messaging > Edit (11 test cases)

Ticket Link

https://mattermost.atlassian.net/browse/MM-28266

@catalintomai catalintomai changed the title MM-28266: MM-28266: Cypress/E2E: Automate backlogs - Messaging > Edit (11 test cases) Oct 19, 2020
@hanzei hanzei added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Oct 21, 2020
Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @catalintomai! The tests look good. Tested and passed. Left small comments.

Note: Non-blocking to this PR but I'll update MM-T712 to reduce and match the test steps into 3, or I might break it into 3 separate test cases.

Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Catalin! Tested and passed. I've modified MM-T712 test case and separated into 3 separate test cases. See comments.

@DHaussermann DHaussermann added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Oct 27, 2020
Copy link

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and passed

  • Reviewed for accuracy with TM4J tests
  • All tests running Green
    LGTM!

@catalintomai catalintomai merged commit 511b891 into mattermost:master Oct 28, 2020
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation
Projects
None yet
5 participants