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

[GH-10035] Added the cypress spec for cancel out leave a team modal #2232

Merged
merged 2 commits into from
Jan 8, 2019

Conversation

pradeepmurugesan
Copy link
Contributor

@pradeepmurugesan pradeepmurugesan commented Dec 25, 2018

Summary

Added the cypress test for the case "cancel out leave a team modal"

Ticket Link

mattermost/mattermost#10035

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Ran make check-style to check for style errors (required for all pull requests)
  • Ran make test to ensure unit and component tests passed

@hanzei hanzei changed the title 10035 added the cypress spec for cancel out leave a team modal [GH-10035] Added the cypress spec for cancel out leave a team modal Dec 26, 2018
@jasonblais jasonblais added the 2: Dev Review Requires review by a core commiter label Dec 28, 2018
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 @pradeepmurugesan! 🎉
Made some comments to satisfy: "Expected: Modal closes, user is still on team, still viewing team, no changes"

cypress/integration/team/teams_spec.js Show resolved Hide resolved
cypress/integration/team/teams_spec.js Show resolved Hide resolved
cypress/integration/team/teams_spec.js Outdated Show resolved Hide resolved
cypress/integration/team/teams_spec.js Show resolved Hide resolved
cypress/integration/team/teams_spec.js Show resolved Hide resolved
@pradeepmurugesan
Copy link
Contributor Author

@saturninoabril made the changes. Kindly review

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.

LGTM, thanks @pradeepmurugesan 🎉

Copy link
Contributor

@sudheerDev sudheerDev left a comment

Choose a reason for hiding this comment

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

Looks Great! Thanks @pradeepmurugesan

@sudheerDev sudheerDev merged commit e21f930 into mattermost:master Jan 8, 2019
@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Jan 8, 2019
@lindy65 lindy65 removed the 4: Reviews Complete All reviewers have approved the pull request label Jan 10, 2019
@JtheBAB JtheBAB mentioned this pull request Jan 15, 2019
9 tasks
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Jan 23, 2019
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Feb 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants