-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[GH-12291] UI Automation: Write an automated test using Cypress for "Markdown quotation paragraphs" #4158
[GH-12291] UI Automation: Write an automated test using Cypress for "Markdown quotation paragraphs" #4158
Conversation
Updating Fork
Merge fork with Master
Merge Fork into Master
Merge Fork
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 @larkox 👍
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.
@larkox Looks good to me, just minor comments only.
e2e/cypress/integration/messaging/markdown_quotation_paragraphs_spec.js
Outdated
Show resolved
Hide resolved
e2e/cypress/integration/messaging/markdown_quotation_paragraphs_spec.js
Outdated
Show resolved
Hide resolved
e2e/cypress/integration/messaging/markdown_quotation_paragraphs_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.
Looks good to me. Thanks @larkox! 🎉
cy.get('#post_textbox').clear().type('>' + messageParts[0]); | ||
cy.get('#post_textbox').type('{shift}{enter}{enter}'); | ||
|
||
cy.get('#post_textbox').type('>' + messageParts[1]); | ||
cy.get('#post_textbox').type('{shift}{enter}{enter}'); | ||
|
||
cy.get('#post_textbox').type('>' + messageParts[2]); | ||
cy.get('#post_textbox').type('{enter}'); |
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.
Last request and I'm all good to merge. Please change to
cy.get('#post_textbox').clear().
type('>' + messageParts[0]).type('{shift}{enter}{enter}').
type('>' + messageParts[1]).type('{shift}{enter}{enter}').
type('>' + messageParts[2]).type('{enter}');
// # Post message to use | ||
cy.get('#post_textbox').clear().type('>' + messageParts[0]).type('{shift}{enter}{enter}'); | ||
cy.get('#post_textbox').type('>' + messageParts[1]).type('{shift}{enter}{enter}'); | ||
cy.get('#post_textbox').type('>' + messageParts[2]).type('{enter}'); |
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.
I'm good with this as is but would be nice to access cy.get('#post_textbox')
once, and chain others as necessary.
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.
@larkox LGTM, thanks!
Summary
UI Automation: Write an automated test using Cypress for "Markdown quotation paragraphs"
Ticket Link
Fixes mattermost/mattermost#12291
Related Pull Requests
None
Screenshots
None