Remove static functions for testability
Open
@g105b is already working on this.
Since Oct 31, 2025.
enhancement
- Dominant language
- PHP
- Stars
- 1
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
WebEngine tests need to mock the SessionSetup class, but there's a static function on there that needs removing (it's currently only being used in one place, and it's not being called statically anyway).
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.
Assessment
This issue has not been assessed yet.