element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
GNU Affero General Public License v3.0
11.23k stars 2k forks source link

Fix Percy tests for `ThreadView` on `threads.spec.ts` #24883

Closed luixxiul closed 7 months ago

luixxiul commented 1 year ago

Steps to reproduce

The PR https://github.com/matrix-org/matrix-react-sdk/pull/9016 has added Percy tests to detect visual regressions of EventTiles inside ThreadView.

It turns out they do not really work how they are supposed to.

3785632928c7fa75893dd1c60c73ccad6d6458f4f2010b08fe832aa0e0dbfe90

All of them seem to have taken snapshots of the right panel without their content. They are supposed to take snapshots ThreadView including EventTiles and a poll.

Outcome

What did you expect?

They should take snapshots of contents inside ThreadView.

What happened instead?

They take snapshots of the ThreadView without its content such as EventTiles, redacted messages placeholder, etc. on modern layout and bubble layout (note that there is not IRC layout on ThreadView).

Operating system

No response

Browser information

No response

URL for webapp

No response

Application version

develop branch

Homeserver

No response

Will you send logs?

No

t3chguy commented 7 months ago

Percy tests are gone