This repository has been archived by the owner on Sep 11, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 829
Add E2E test of audio player #10441
Merged
Merged
Add E2E test of audio player #10441
Changes from 71 commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
4dcc8e7
Add audio-player.spec.ts
luixxiul 1694921
Download audio file
luixxiul ddc7898
Reply
luixxiul b8facfe
ReplyChain
luixxiul f88ff9c
Audio player on a thread
luixxiul 133cf69
Add 'wrapper' option to takeSnapshots() in mx_ThreadView
luixxiul 2e55e28
Click timestamps to highlight mx_EventTile to hopefully avoid flaky P…
luixxiul f2a6d0a
Assert that the radio button for light theme was checked
luixxiul c6cf476
Typo
luixxiul f23cfc2
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul 617f979
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul b282fff
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul c6b908d
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul f0b120e
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul 495b612
Add comments to const percyCSS
luixxiul 649d20b
Remove double quotation marks inside backticks
luixxiul 445be1d
Rename a test
luixxiul 3742b4f
Rename a test and edit a comment on mx_ReplyChain
luixxiul f6ffc1a
Wait until rendering of the player settled and assert that the play b…
luixxiul b18794d
Extract the test for IRC layout
luixxiul 36b650e
Add and edit comments for takeSnapshots() and the test on IRC layout
luixxiul 399a434
Move comments about snapshot widths
luixxiul fa97ee6
Check audio player's rendering and its button's visibility
luixxiul 725d020
Add checkPlayerVisibility()
luixxiul 82e4804
Take snapshots of the small player
luixxiul bcf8634
Edit snapshot name
luixxiul 376ef1b
Use cy.closeDialog()
luixxiul 6d559f0
Check player visibility on ThreadView
luixxiul 1e0560c
Manage widths with variables
luixxiul 39cdbe3
Merge branch 'develop' into test-audioplayer
luixxiul 8ff182a
Remove 'takeSnapshotTimeline'
luixxiul b148eb8
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul f4a0ee5
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul e8094bd
Update cypress/e2e/audio-player/audio-player.spec.ts
luixxiul 6b496e8
Edit comments
luixxiul 98dd473
Remove an obsolete comment
luixxiul 1a0fde5
Remove a command to set a layout value
luixxiul 1b279fd
Remove a command to enable use_system_theme
luixxiul 698f443
Revert "Remove a command to enable use_system_theme"
luixxiul 437ee52
Edit a comment - disable system theme
luixxiul a59586f
Merge branch 'develop' of https://github.com/matrix-org/matrix-react-…
luixxiul 555f5de
Edit a comment
luixxiul cdc77d6
Remove an assertion about the audio player being rendered
luixxiul 946b330
Edit a test on a thread
luixxiul c581aae
Use Cypress Testing Library
luixxiul 6969e5a
Use existing functions in favor of visitRoom()
luixxiul af07043
Improve the test flow - checking on the high contrast theme
luixxiul 04ad2f8
Simplify and improve the widths setting of snapshots
luixxiul a20648f
Improve takeSnapshots
luixxiul dd1d306
Improve the test - checking a reply and a reply chain
luixxiul 6495655
Remove redundant within()
luixxiul 7774d44
Add audio files to make the test check files more strictly
luixxiul 5dc2463
Simplify the test a little bit by removing within()
luixxiul 3c5bb25
Edit a comment
luixxiul 7488d88
Create uploadAndTakeSnapshots()
luixxiul a9fc678
Improve the test - download
luixxiul 21d0798
lint
luixxiul b11f30d
theme -> detail
luixxiul 7019c86
Merge branch 'develop' of https://github.com/matrix-org/matrix-react-…
luixxiul b176a22
Iterate - remove uploadAndTakeSnapshots
luixxiul 50a0c1a
Iterate - remove within()
luixxiul 0175351
Iterate - remove redundant wrappers
luixxiul e2e08b3
Output log for IRC layout
luixxiul f3fc364
Remove redundant comments
luixxiul b38ee4d
Edit log()
luixxiul df5f2b9
Chain commands
luixxiul bf296a4
Edit comments
luixxiul d95b153
Merge branch 'develop' of https://github.com/matrix-org/matrix-react-…
luixxiul c1baa6d
Remove unused injectAxe()
luixxiul c0d9733
Merge branch 'develop' into test-audioplayer
luixxiul 7d0df71
Add a comment about 'monospace'
luixxiul e1ae92b
Edit a comment
luixxiul febe45b
Merge branch 'develop' into test-audioplayer
richvdh 4c9c613
Merge branch 'develop' into test-audioplayer
luixxiul 450a48f
Merge branch 'develop' into test-audioplayer
t3chguy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test result: https://www.virustotal.com/gui/url/7491184fee27fb2611212c7fff4ed5919b5b7a9d840fad00f5d0e8ea0fef78fa?nocache=1