matrix-org / matrix-org/mjolnir
Config shouldn't be so coupled to the file, makes automated testing different configs difficult.
Open
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 445
- Forks
- 75
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 3
Description
The example being reportPolling VS the web proxy, it would be nice to test both setups in isolation of each other so it is obvious one isn't doing the other's job.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing how configuration is coupled to the file and how reportPolling and the web proxy are initialized. Compare the automated test setup for both configurations; done means each setup can be tested in isolation and neither performs the other’s job.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100