broadcast: increase testability of broadcast handling related functionality

Issue #257 resolved
Saxon Milton created an issue

The broadcast related functionality is currently difficult to test. On first inspection this seems to be caused by direct dependency on the global “settingsStore” value, which ends up being hardcoded in places + direct dependency on the broadcast package (youtube broadcast API implementation). Obviously these two things are not available in a gotest setting. It’s therefore recommended we decouple from these things and create some interfaces through which functionality can access implementations. We can then switch these things out with dummy version for testing.

Comments (3)

  1. Log in to comment