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

Cypress tests for event shields #11525

Merged
merged 5 commits into from
Sep 5, 2023
Merged

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Sep 4, 2023

I've got some refactoring on the way for this stuff, so time to make sure we have some tests.

If the diff is a bit impenetrable, try the individual commits.


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

@richvdh richvdh requested a review from a team as a code owner September 4, 2023 16:51
@richvdh richvdh added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Sep 4, 2023
@richvdh
Copy link
Member Author

richvdh commented Sep 4, 2023

This is failing due to element-hq/element-web#26115

@richvdh richvdh enabled auto-merge September 5, 2023 10:22
@richvdh richvdh added this pull request to the merge queue Sep 5, 2023
Merged via the queue into develop with commit 3818c1d Sep 5, 2023
19 checks passed
@richvdh richvdh deleted the rav/element-r/shields/tests branch September 5, 2023 12:08
nurjinjafar pushed a commit to nordeck/matrix-react-sdk that referenced this pull request Sep 6, 2023
* Factor downloadKey out to `utils.ts`

* Add a new `describe` block for event shields

* create a beforeEach block

* Cypress tests for event shields
github-merge-queue bot pushed a commit that referenced this pull request Sep 7, 2023
* Implement denied request mask and logic

Signed-off-by: AHMAD KADRI <[email protected]>

* refactor / fix deny requests isues

* fix tests create denied message test

Signed-off-by: AHMAD KADRI <[email protected]>

* add another test for the primary action for denied request

Signed-off-by: AHMAD KADRI <[email protected]>

* fix linter issues

Signed-off-by: nurjinn jafar <[email protected]>

* regenerate translation

Signed-off-by: nurjinn jafar <[email protected]>

* fix translation and minor refactoring
Signed-off-by: nurjinn jafar <[email protected]>

* segment into 4

* Remove parallel from Cypress command to avoid talking to Cypress Cloud

* Re-add --parallel flag for Percy

* Prevent event propagation when clicking icon buttons (#11515)

* Prevent event propagation when clicking icon buttons

* Inhibit view user on click behaviour for room header face pile

* Update snapshot

* Add a 'm.relates_to' to edits in receipt tests and disable failing tests (#11501)

* Add a 'm.relates_to' to edits in receipt tests

* Disable a test that fails with real edits

* Wait for the room to be read after we mark it as read

* Skip tests that are failing because of inconsistencies between local and CI behaviour

* Allow creating public knock rooms (#11481)

* Allow creating public knock rooms

Signed-off-by: Charly Nguyen <[email protected]>

* Apply PR feedback

Signed-off-by: Charly Nguyen <[email protected]>

* Apply PR feedback

Signed-off-by: Charly Nguyen <[email protected]>

---------

Signed-off-by: Charly Nguyen <[email protected]>

* Collect `console.debug` logs during cypress tests (#11478)

In order for the logs collected by cypress to actually be useful, we really
need `cons:debug`.

* Migrate more strings to translation keys (#11522)

* Only show Search button in RoomSummaryCard if new room UI enabled (#11524)

* Only show Search button in RoomSummaryCard if new room UI enabled

* Update snapshot

* Update vector-im (#11526)

* Update vector-im

* Update snapshots of Compound Avatars

* Update snapshots of Compound Avatars

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>

* Migrate more strings to translation keys (#11530)

* Fix regression around FacePile with overflow (#11527)

* Work around compound-web AvatarStack not applying overlap to non-Avatars

* Fix FacePile overflow tile not being layed out correctly

* Use RoomStateEvent.Update for knocks (#11516)

Signed-off-by: Charly Nguyen <[email protected]>

* Cypress tests for event shields (#11525)

* Factor downloadKey out to `utils.ts`

* Add a new `describe` block for event shields

* create a beforeEach block

* Cypress tests for event shields

* Document how to match the CI config for Cypress (#11531)

* Document how to match the CI config for Cypress

* Clarify language about needing Chrome

* Move Cypress info into the Cypress-specific docs

* Migrate more strings to translation keys (#11532)

---------

Signed-off-by: AHMAD KADRI <[email protected]>
Signed-off-by: nurjinn jafar <[email protected]>
Signed-off-by: Charly Nguyen <[email protected]>
Co-authored-by: AHMAD KADRI <[email protected]>
Co-authored-by: Kerry Archibald <[email protected]>
Co-authored-by: Andy Balaam <[email protected]>
Co-authored-by: Johannes Marbach <[email protected]>
Co-authored-by: Germain <[email protected]>
Co-authored-by: Charly Nguyen <[email protected]>
Co-authored-by: Richard van der Hoff <[email protected]>
Co-authored-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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