feat: revisit user tracking

[JurgenR]

Closes #52

[github-actions]

Test Results

607 tests  +15   607 ✔️ +15   3s ⏱️ -1s
    1 suites ±  0       0 💤 ±  0 
    1 files   ±  0       0 ❌ ±  0 

Results for commit f78435f. ± Comparison against base commit b6a1697.

This pull request removes 3 and adds 18 tests. Note that renamed tests count towards both.

tests.unit.test_server_manager.TestServerManager ‑ test_untrackUser_userTracked_shouldSendRemoveUser
tests.unit.test_state.TestState ‑ test_whenExistingUser_shouldReturnUser
tests.unit.test_state.TestState ‑ test_whenMissingUser_shouldCreateAndReturnUser

tests.unit.test_model.TestModelChatMessage ‑ test_isServerMessage[server-False-False]
tests.unit.test_model.TestModelChatMessage ‑ test_isServerMessage[server-True-True]
tests.unit.test_model.TestModelChatMessage ‑ test_isServerMessage[user-False-False]
tests.unit.test_model.TestModelChatMessage ‑ test_isServerMessage[user-True-False]
tests.unit.test_server_manager.TestServerManager ‑ test_onChatPrivateMessage_shouldSendAckAndEmitEvent
tests.unit.test_server_manager.TestServerManager ‑ test_onJoinRoom_shouldUpdateRoomAndUsers
tests.unit.test_server_manager.TestServerManager ‑ test_onLeaveRoom_shouldUpdateRoomAndUsers
tests.unit.test_server_manager.TestServerManager ‑ test_untrackUser_addUserFlagRemains_shouldNotSendRemoveUser
tests.unit.test_server_manager.TestServerManager ‑ test_untrackUser_lastAddUserFlag_shouldSendRemoveUser
tests.unit.test_server_manager.TestServerManager ‑ test_untrackUser_lastAddUserFlag_shouldSendRemoveUserAndNotSetStatusUnknown
…

♻️ This comment has been updated with latest results.