android icon indicating copy to clipboard operation
android copied to clipboard
verified publisher icon home-assistant

Move handleEvent to WebSocketEventHandler

Open NeimadTL opened this issue 1 month ago • 2 comments

For readability reason the handleEvent function has been moved WebSocketCoreImpl to WebSocketEventHandler. The function could be refactored into smaller functions later on. Tests still need to be added.

Related to #5271

NeimadTL avatar Nov 15 '25 19:11 NeimadTL

@NeimadTL thanks for your contributions, I'm going to put this PR in draft since you have some ktlint issues. I would also like to see unit tests like it is written in the issue you've linked.

TimoPtr avatar Nov 17 '25 11:11 TimoPtr

@NeimadTL , you're welcome. Indeed, I still have some work to do on it (linter + tests, maybe some more refactoring); will continue soon.

NeimadTL avatar Nov 17 '25 11:11 NeimadTL