test: add test to cover new changes from jxmorris12#210 jxmorris12#211 jxmorris12#212 jxmorris12#213

Author: mdevolde

tests/unit/test_match.py

@@ -106,6 +106,21 @@ def test_four_byte_char_adjustment(self) -> None:
         adjusted_offset = 2
         assert m.offset == adjusted_offset
 
+    def test_four_byte_char_after_match_is_not_counted(self) -> None:
+        """A 4-byte emoji after the match does not shift the offset."""
+        # "🌅" at position 6 is after the match at offset 0, so the adjustment
+        # loop must break on its first iteration instead of counting it.
+        text = "hello 🌅"
+        attrib = _make_attrib(
+            offset=0,
+            length=5,
+            context_text=text,
+            context_offset=0,
+            sentence=text,
+        )
+        m = Match(attrib, four_byte_char_positions(text))
+        assert m.offset == 0
+
     def test_no_adjustment_without_four_byte_chars(self) -> None:
         """Offsets are unchanged when no 4-byte characters precede the match."""
         text = "Hello world today"

tests/unit/test_server_unit.py

@@ -882,6 +882,16 @@ def test_raises_rate_limit_error_on_http_426(self) -> None:
         with pytest.raises(RateLimitError):
             lt._query_server("http://fake/", num_tries=1)
 
+    def test_raises_rate_limit_error_on_http_426_with_valid_json(self) -> None:
+        """RateLimitError is raised on HTTP 426 even when the body is valid JSON."""
+        r = _make_json_response(
+            b'{"error": "rate limited"}', status_code=_HTTP_RATE_LIMIT_STATUS
+        )
+        lt = _bare_lt(_remote=True)
+        lt._session = _MockSession(get_response=r)
+        with pytest.raises(RateLimitError):
+            lt._query_server("http://fake/", num_tries=1)
+
     def test_returns_none_with_zero_tries(self) -> None:
         """Returns None immediately when num_tries=0 (loop never executes)."""
         lt = _bare_lt()
@@ -939,6 +949,18 @@ def close(self) -> None:
             lt._terminate_server()
         assert mock_stdin.closed
 
+    def test_does_not_raise_when_process_not_in_running_list(self) -> None:
+        """_terminate_server() does not raise if the process is absent from the list."""
+        mock_proc = _MockProcess(poll_return=None)
+        lt = _bare_lt()
+        lt._server = mock_proc  # type: ignore[assignment]
+        with (
+            patch("language_tool_python.server._RUNNING_SERVER_PROCESSES", []),
+            patch("language_tool_python.server._kill_processes"),
+        ):
+            lt._terminate_server()
+        assert lt._server is None
+
 
 class TestStartLocalServer:
     """Tests for _start_local_server() PathError branch."""