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

Fix failing test in at-mentions #3692

Merged
merged 1 commit into from
Sep 19, 2019
Merged

Fix failing test in at-mentions #3692

merged 1 commit into from
Sep 19, 2019

Conversation

saturninoabril
Copy link
Member

Summary

Quick test for failing E2E on at-mentions

Ticket Link

None, reported on daily Cypress test - https://community.mattermost.com/core/pl/yiwatj6efpybjyso6cuxkihb4c

@saturninoabril saturninoabril added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Sep 18, 2019
@@ -11,7 +11,7 @@ import users from '../../fixtures/users.json';

function setNotificationSettings(desiredSettings = {first: true, username: true, shouts: true, custom: true, customText: '@'}) {
// Navigate to settings modal
cy.toAccountSettingsModal(null, true);
cy.toAccountSettingsModal('user-1');
Copy link
Member Author

Choose a reason for hiding this comment

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

Explicitly login before going into account settings

Copy link
Contributor

@srkgupta srkgupta left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@iomodo iomodo left a comment

Choose a reason for hiding this comment

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

LGTM!

@saturninoabril saturninoabril 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 Sep 19, 2019
@saturninoabril saturninoabril merged commit 7f71bb5 into mattermost:master Sep 19, 2019
@saturninoabril saturninoabril deleted the fix-e2e-atmention branch September 19, 2019 14:05
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Sep 19, 2019
jwilander pushed a commit that referenced this pull request Sep 24, 2019
skheria pushed a commit to uber-archive/mattermost-webapp that referenced this pull request Oct 3, 2019
skheria pushed a commit to uber-archive/mattermost-webapp that referenced this pull request Oct 3, 2019
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
4 participants