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

Add tests that notify Application Services about ephemeral events such as presence #10950

Open
reivilibre opened this issue Sep 30, 2021 · 1 comment
Open

Add tests that notify Application Services about ephemeral events such as presence #10950

reivilibre opened this issue Sep 30, 2021 · 1 comment
Labels
A-Application-Service Related to AS support A-Testing Issues related to testing in complement, synapse, etc T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@reivilibre
Copy link
Contributor

There don't appear to be any tests that exercise the area of the code in which #10941 failed. It would be nice to have some tests to cover this area (it seems like on_new_event is not very tested in general, though it may just be that this kind of testing is better suited to SyTest/Complement?).
@clokep clokep added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Sep 30, 2021
@babolivier babolivier reopened this Oct 4, 2021
@babolivier
Copy link
Contributor

Ugh, closed the wrong one on accident.

@MadLittleMods MadLittleMods added A-Application-Service Related to AS support A-Testing Issues related to testing in complement, synapse, etc labels Dec 23, 2022
@realtyem realtyem mentioned this issue Jul 5, 2023
Open
@matrixbot matrixbot mentioned this issue Dec 21, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Application-Service Related to AS support A-Testing Issues related to testing in complement, synapse, etc T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@clokep @MadLittleMods @babolivier @reivilibre