Fix linting issues and remove unused code

Co-authored-by: kjy5 <[email protected]>

Author: Copilot

src/ephys_link/back_end/server.py

@@ -34,7 +34,6 @@
 from ephys_link.utils.constants import (
     MALFORMED_REQUEST_ERROR,
     PORT,
-    SERVER_NOT_INITIALIZED_ERROR,
     UNKNOWN_EVENT_ERROR,
     cannot_connect_as_client_is_already_connected_error,
     client_disconnected_without_being_connected_error,
@@ -63,10 +62,6 @@ def __init__(self, options: EphysLinkOptions, platform_handler: PlatformHandler,
 
         # Initialize server.
         self._sio: AsyncServer = AsyncServer()
-        # Exit if _sio is not a Server.
-        if not isinstance(self._sio, AsyncServer):
-            self._console.critical_print(SERVER_NOT_INITIALIZED_ERROR)
-            raise TypeError(SERVER_NOT_INITIALIZED_ERROR)
 
         self._app = Application()
         self._sio.attach(self._app)  # pyright: ignore [reportUnknownMemberType]

src/ephys_link/utils/constants.py

@@ -58,8 +58,6 @@ def did_not_reach_target_depth_error(request: SetDepthRequest, final_unified_dep
 
 EMERGENCY_STOP_MESSAGE = "Emergency Stopping All Manipulators..."
 
-SERVER_NOT_INITIALIZED_ERROR = "Server not initialized."
-
 
 def cannot_connect_as_client_is_already_connected_error(new_client_sid: str, current_client_sid: str) -> str:
     """Generate an error message for when the client is already connected.

tests/back_end/test_server.py

@@ -1,9 +1,7 @@
-import asyncio
 from json import dumps, loads
 
 import pytest
 from pytest_mock import MockerFixture
-from socketio import AsyncServer  # pyright: ignore[reportMissingTypeStubs]
 from vbl_aquarium.models.ephys_link import (
     AngularResponse,
     BooleanStateResponse,
@@ -25,7 +23,6 @@
 from ephys_link.front_end.console import Console
 from ephys_link.utils.constants import (
     MALFORMED_REQUEST_ERROR,
-    SERVER_NOT_INITIALIZED_ERROR,
     UNKNOWN_EVENT_ERROR,
     cannot_connect_as_client_is_already_connected_error,
     client_disconnected_without_being_connected_error,
@@ -49,21 +46,6 @@ def server(self, platform_handler: PlatformHandler, console: Console) -> Server:
         """Fixture for server."""
         return Server(EphysLinkOptions(), platform_handler, console)
 
-    def test_failed_server_init(
-        self, platform_handler: PlatformHandler, console: Console, mocker: MockerFixture
-    ) -> None:
-        """Server should raise error if sio is not an AsyncServer."""
-        # Mock out the AsyncServer init.
-        patched_async_server = mocker.patch.object(AsyncServer, "__new__")
-
-        # Act
-        with pytest.raises(TypeError) as init_error:
-            _ = Server(EphysLinkOptions(), platform_handler, console)
-
-        # Assert
-        patched_async_server.assert_called_once()
-        assert init_error.value.args[0] == SERVER_NOT_INITIALIZED_ERROR
-
     def test_launch_server(
         self, server: Server, platform_handler: PlatformHandler, console: Console, mocker: MockerFixture
     ) -> None: