-
Notifications
You must be signed in to change notification settings - Fork 2.7k
MM-29347: E2E Cypress Tests - System Console > Enterprise > Group Mentions (Part 1) #6652
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. All tests passed locally. Thanks @srkgupta !!
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please consider adding the timeouts after cy.visit
. Cypress server runs usually fail on those.
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Outdated
Show resolved
Hide resolved
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Show resolved
Hide resolved
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Outdated
Show resolved
Hide resolved
Thanks @josephbaylon. Addressed your comments and added additional timeouts after cy.visit. Please recheck. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @srkgupta !!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @srkgupta! Left couple of suggestions.
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Outdated
Show resolved
Hide resolved
e2e/cypress/integration/enterprise/group_mentions/group_mentions_system_messages_spec.js
Outdated
Show resolved
Hide resolved
/update-branch |
Thanks @saturninoabril for your feedback. Committed the requested changes. Please recheck. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@srkgupta Thanks for updating! LGTM, tested and passed.
Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour. |
…tions (Part 1) (#6652) * Added some cypress tests part of MM-29347 * Fixed issue on fresh LDAP setup * Added additional timeouts after cy.visit * Fixed as per PR review comments Co-authored-by: Mattermod <[email protected]>
…tions (Part 1) (#6652) * Added some cypress tests part of MM-29347 * Fixed issue on fresh LDAP setup * Added additional timeouts after cy.visit * Fixed as per PR review comments Co-authored-by: Mattermod <[email protected]>
Summary
MM-29347: E2E Cypress Tests - System Console > Enterprise > Group Mentions (Part 1)
Ticket Link
JIRA Ticket:
MM-29347
TM4J Tests:
MM-T2456
MM-T2457
MM-T2458
MM-T2459
Related Pull Requests
None
Screenshots