mattermost-webapp
mattermost-webapp copied to clipboard
MM-T1539: E2E test to verify :man-gesturing-ok: is rendered with the correct gender
Summary
Verify 🙆♂️ is rendered with the correct gender
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/18163
Related Pull Requests
No related PRs.
Screenshots
No UI changes.
Release Note
NONE
Hello @sk409,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.
I will move spec file and change describe's name.
@sk409 Could you, please use ts instead of js, we are currently running a typescript migration campaign, and these will eventually add up to tech debt. Thanks
@AshishDhama Thank you for your review! I renamed .js to .ts.
/e2e-test
Successfully triggered e2e testing! https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/249259
@sk409 Can you please lint errors ?
@yasserfaraazkhan @furqanmlk
I forgot to run make fix-style
.
I fixed style.
/update-branch
This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!
What is the review status of this PR?