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

Add E2E test for FilePanel #10400

Merged
merged 7 commits into from
Mar 22, 2023
Merged

Add E2E test for FilePanel #10400

merged 7 commits into from
Mar 22, 2023

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Mar 17, 2023

This PR adds a basic E2E test for FilePanel. It also add an empty one second OGG file to check the audio player on the panel. The main intention of this PR is to make sure that EventTiles on the panel keep being rendered in the same way after mx_EventTile:not([data-layout=bubble]) is removed from _EventTile.pcss.

Closes element-hq/element-web#24861
For #9033 (checking EventTiles on FilePanel)

type: task

Signed-off-by: Suguru Hirahara luixxiul@users.noreply.github.com

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

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

- Add an empty one second OGG audio file

Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
…panel

Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
@luixxiul luixxiul marked this pull request as ready for review March 18, 2023 05:08
@luixxiul luixxiul requested a review from a team as a code owner March 18, 2023 05:08
@luixxiul
Copy link
Contributor Author

@florianduros Would you please review when you have some time? Thanks in advance.

Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
@luixxiul
Copy link
Contributor Author

Copy link
Contributor

@florianduros florianduros left a comment

Choose a reason for hiding this comment

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

LGFM, thanks!

@florianduros florianduros merged commit a5fcdd7 into matrix-org:develop Mar 22, 2023
@luixxiul
Copy link
Contributor Author

Thanks for the review!

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 Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create E2E test of FilePanel
2 participants