From f8712600437f192ffc7612db5b6ec75792b7aadd Mon Sep 17 00:00:00 2001
From: Alex Carney <alcarneyme@gmail.com>
Date: Sun, 25 Feb 2024 20:08:15 +0000
Subject: [PATCH] lsp: Delete `MockSphinxClient`

The tests now use the real thing and there does not seem to be much
benefit in trying to maintain this.
---
 .../features/sphinx_manager/__init__.py       |   4 -
 .../features/sphinx_manager/client_mock.py    | 100 ------------------
 2 files changed, 104 deletions(-)
 delete mode 100644 lib/esbonio/esbonio/server/features/sphinx_manager/client_mock.py

diff --git a/lib/esbonio/esbonio/server/features/sphinx_manager/__init__.py b/lib/esbonio/esbonio/server/features/sphinx_manager/__init__.py
index c8e138ee6..610959d25 100644
--- a/lib/esbonio/esbonio/server/features/sphinx_manager/__init__.py
+++ b/lib/esbonio/esbonio/server/features/sphinx_manager/__init__.py
@@ -4,8 +4,6 @@
 
 from .client import ClientState
 from .client import SphinxClient
-from .client_mock import MockSphinxClient
-from .client_mock import mock_sphinx_client_factory
 from .client_subprocess import make_subprocess_sphinx_client
 from .config import SphinxConfig
 from .manager import SphinxManager
@@ -19,8 +17,6 @@
     "SphinxClient",
     "SphinxConfig",
     "SphinxManager",
-    "MockSphinxClient",
-    "mock_sphinx_client_factory",
 ]
 
 
diff --git a/lib/esbonio/esbonio/server/features/sphinx_manager/client_mock.py b/lib/esbonio/esbonio/server/features/sphinx_manager/client_mock.py
deleted file mode 100644
index 1836fb207..000000000
--- a/lib/esbonio/esbonio/server/features/sphinx_manager/client_mock.py
+++ /dev/null
@@ -1,100 +0,0 @@
-from __future__ import annotations
-
-import asyncio
-import typing
-from typing import Dict
-from typing import List
-from typing import Optional
-from typing import Union
-
-from esbonio.sphinx_agent import types
-
-if typing.TYPE_CHECKING:
-    from esbonio.server import Uri
-
-    from .client import SphinxClient
-    from .manager import SphinxManager
-
-
-class MockSphinxClient:
-    """A mock SphinxClient implementation, used for testing."""
-
-    def __init__(
-        self,
-        create_result: Union[types.SphinxInfo, Exception],
-        build_result: Optional[Union[types.BuildResult, Exception]] = None,
-        build_file_map: Optional[Dict[Uri, str]] = None,
-        startup_delay: float = 0.5,
-    ):
-        """Create a new instance.
-
-        Parameters
-        ----------
-        create_result
-           The result to return when calling ``create_application``.
-           If an exception is given it will be raised.
-
-        build_file_map
-           The build file map to use.
-
-        build_result
-           The result to return when calling ``build``.
-           If an exception is given it will be raised.
-
-        startup_delay
-           The duration to sleep for when calling ``start``
-        """
-        self._create_result = create_result
-        self._startup_delay = startup_delay
-        self._build_result = build_result or types.BuildResult()
-        self._build_file_map = build_file_map or {}
-        self.building = False
-
-    @property
-    def id(self) -> Optional[str]:
-        """The id of the Sphinx instance."""
-        if isinstance(self._create_result, Exception):
-            return None
-
-        return self._create_result.id
-
-    async def start(self, *args, **kwargs):
-        await asyncio.sleep(self._startup_delay)
-
-    async def stop(self, *args, **kwargs):
-        pass
-
-    async def create_application(self, *args, **kwargs) -> types.SphinxInfo:
-        """Create an application."""
-
-        if isinstance(self._create_result, Exception):
-            raise self._create_result
-
-        return self._create_result
-
-    async def build(self, *args, **kwargs) -> types.BuildResult:
-        """Trigger a build"""
-
-        if isinstance(self._build_result, Exception):
-            raise self._build_result
-
-        return self._build_result
-
-    async def get_src_uris(self) -> List[Uri]:
-        """Return all known source files."""
-        return [s for s in self._build_file_map.keys()]
-
-    async def get_build_path(self, src_uri: Uri) -> Optional[str]:
-        """Get the build path associated with the given ``src_uri``."""
-        return self._build_file_map.get(src_uri)
-
-
-def mock_sphinx_client_factory(client: Optional[SphinxClient] = None):
-    """Return a factory function that can be used with a ``SphinxManager`` instance."""
-
-    def factory(manager: SphinxManager):
-        if client is None:
-            raise RuntimeError("Unexpected client creation")
-        return client
-
-    return factory