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

Move threads e2e tests over to cypress #8501

Merged
merged 27 commits into from
May 10, 2022
Merged

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented May 4, 2022

Requires matrix-org/matrix-js-sdk#2352


This change is marked as an internal change (Task), so will not be included in the changelog.

@t3chguy t3chguy added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label May 4, 2022
@cypress
Copy link

cypress bot commented May 9, 2022



Test summary

6 1 0 0


Run details

Project matrix-react-sdk
Status Failed
Commit c6bf77a ℹ️
Started May 9, 2022 6:00 PM
Ended May 9, 2022 6:02 PM
Duration 02:23 💡
OS Linux Ubuntu - 20.04
Browser Chrome 101

View run in Cypress Dashboard ➡️


Failures

cypress/integration/5-threads/threads.spec.ts Failed
1 Threads > should reload when disabling threads beta

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@t3chguy t3chguy marked this pull request as ready for review May 10, 2022 13:37
@t3chguy t3chguy requested a review from a team as a code owner May 10, 2022 13:37
Copy link
Contributor

@kerryarchibald kerryarchibald left a comment

Choose a reason for hiding this comment

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

LGTM! (and yay for removing the flaky puppeteer threads test, thanks!)

cypress/global.d.ts Outdated Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants