obs-websocket-community-projects/obs-websocket-java

ObsCommunicator - Message Send Facade

TinaTiel opened this issue · 1 comments

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.

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.