Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Flaky Cypress test Read receipts > redactions > in the main timeline > Redacting all unread messages makes the room read #26160

Closed
andybalaam opened this issue Sep 13, 2023 · 6 comments
Closed

Flaky Cypress test Read receipts > redactions > in the main timeline > Redacting all unread messages makes the room read #26160

andybalaam opened this issue Sep 13, 2023 · 6 comments
Labels
A-Developer-Experience A-Read-Receipts O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-Flaky-Test A test is raising false alarms

Comments

@andybalaam
Copy link
Member

andybalaam commented Sep 13, 2023 •
edited
Loading

Flaking on develop.

https://github.com/matrix-org/matrix-js-sdk/actions/runs/6173326658/job/16755418663#step:8:794

Might just be too many tests in one file.

            cons:warn ❖  failed to asynchronously prepare wasm: RangeError: WebAssembly.instantiate(): Out of memory: Cannot allocate Wasm memory for new instance
           cons:warn ❖  Aborted(RangeError: WebAssembly.instantiate(): Out of memory: Cannot allocate Wasm memory for new instance)

  1) Read receipts
       redactions
         in the main timeline
           "before each" hook for "Redacting all unread messages makes the room read":
     AssertionError: Timed out retrying after 30000ms: Expected to find element: `.mx_MatrixChat`, but never found it.
@andybalaam andybalaam added S-Major Severely degrades major functionality or product features, with no satisfactory workaround A-Developer-Experience O-Uncommon Most users are unlikely to come across this or unexpected workflow Z-Flaky-Test A test is raising false alarms labels Sep 13, 2023
@richvdh richvdh changed the title Flaky Cypress test Read receipts > redactions > in the main timeline > Redacting all unread messages makes the room read Flaky Cypress test Read receipts > redactions > in the main timeline > Redacting all unread messages makes the room read Sep 22, 2023
@richvdh
Copy link
Member

richvdh commented Oct 3, 2023

Haven't seen this one for a while - maybe it is fixed?

@richvdh richvdh mentioned this issue Oct 3, 2023
Merged
@andybalaam
Copy link
Member Author

Haven't seen this one for a while - maybe it is fixed?

Unlikely since the identical (up to the point where it flakes) one just below it is skipped for being flaky.

@richvdh
Copy link
Member

richvdh commented Oct 4, 2023

Might just be too many tests in one file.

Doesn't this predate the breakup of the receipts tests?

@andybalaam
Copy link
Member Author

Might just be too many tests in one file.

Doesn't this predate the breakup of the receipts tests?

Yes

@andybalaam
Copy link
Member Author

So I guess we don't have evidence that this is flaking

@andybalaam
Copy link
Member Author

Closing until we see it flake again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Developer-Experience A-Read-Receipts O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-Flaky-Test A test is raising false alarms
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@andybalaam @richvdh @robintown