ObsCommunicator - Message Send Facade
TinaTiel opened this issue · 1 comments
TinaTiel commented
When sending messages at the moment, it depends directly on a full Jetty WebsocketClient Session.
This can be deep-mocked, but a better practice would be to put this behind a facade so we have control over that interface and can verify calls in-general to send messages. This would especially enable unit-testing individual lifecycle methods.
TinaTiel commented
After digging into this more, I'm seeing more complexity for little benefit.
So, I'm going to abandon this for now and may reconsider later.