mcp-gee-sweet 0.9.2.dev238__tar.gz → 0.9.2.dev243__tar.gz

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: mcp-gee-sweet
-Version: 0.9.2.dev238
+Version: 0.9.2.dev243
 Summary: MCP server for Google Workspace — Sheets, Drive, Docs (HTML or Markdown), and Calendar.
 Project-URL: Homepage, https://github.com/khuisman/mcp-gee-sweet
 Project-URL: Bug Tracker, https://github.com/khuisman/mcp-gee-sweet/issues

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/docs/roadmap.md

@@ -193,7 +193,7 @@ Same shape as v0.8.1 — stabilize on defects that surfaced since v0.9.0 shipped
 - [x] Lock in the isolated-run rendering rule — an isolated bullet run renders at its true depth (`depth N` → `nestingLevel N`) at *every* depth, never clamped (decided 2026-09-09: consistency over avoiding orphaned indent; the markup generator owns the depth it emits). Extend TC-DOC190 with a `min_depth >= 2` case; state the rule in `emitter.py`'s bullet-pass comment and the `docs/emitter.py` paragraph of `CLAUDE.md` (PR #731) ([#714](https://github.com/khuisman/mcp-gee-sweet/issues/714))
 - [x] `create_doc` markdown: a fenced code block whose body is raw HTML tags renders as loose plain paragraphs (fence ignored, indentation flattened) instead of a monospaced code block — python-markdown's raw-HTML-block handling appears to pre-empt `fenced_code`. Surfaced during #714 triage (PR #730) ([#719](https://github.com/khuisman/mcp-gee-sweet/issues/719))
 - [ ] **Bundle:** Docs write side (emitter, html_parser, images, content), one PR closing [#509](https://github.com/khuisman/mcp-gee-sweet/issues/509), [#567](https://github.com/khuisman/mcp-gee-sweet/issues/567), [#581](https://github.com/khuisman/mcp-gee-sweet/issues/581), [#638](https://github.com/khuisman/mcp-gee-sweet/issues/638), [#715](https://github.com/khuisman/mcp-gee-sweet/issues/715) — lane A ([#894](https://github.com/khuisman/mcp-gee-sweet/issues/894))
-- [ ] **Bundle:** Calendar hardening, one PR closing [#459](https://github.com/khuisman/mcp-gee-sweet/issues/459), [#615](https://github.com/khuisman/mcp-gee-sweet/issues/615), [#626](https://github.com/khuisman/mcp-gee-sweet/issues/626), [#627](https://github.com/khuisman/mcp-gee-sweet/issues/627), [#726](https://github.com/khuisman/mcp-gee-sweet/issues/726) — lane B ([#890](https://github.com/khuisman/mcp-gee-sweet/issues/890))
+- [x] **Bundle:** Calendar hardening, one PR closing [#459](https://github.com/khuisman/mcp-gee-sweet/issues/459), [#615](https://github.com/khuisman/mcp-gee-sweet/issues/615), [#626](https://github.com/khuisman/mcp-gee-sweet/issues/626), [#627](https://github.com/khuisman/mcp-gee-sweet/issues/627), [#726](https://github.com/khuisman/mcp-gee-sweet/issues/726) — lane B (PR #912) ([#890](https://github.com/khuisman/mcp-gee-sweet/issues/890))
 - [x] `delete_file(permanent=True)` surfaces a misleading 404 when the caller can trash but not permanently delete (Shared Drive Content manager) — an agent can read it as "already gone" and skip cleanup; check `capabilities.canDelete` first — lane B ([#876](https://github.com/khuisman/mcp-gee-sweet/issues/876)) (PR #888)
 - [x] SSE: SIGTERM with an SSE stream open logs an "Exception in ASGI application" traceback; the server still exits cleanly, pre-existing before #833 — lane A (PR #911) ([#868](https://github.com/khuisman/mcp-gee-sweet/issues/868))
 - [x] `sync_folder`'s `convert_markdown` restamp-failure handling: quota-message gap, duplicated create()+update() try/except, test fixture isolation — companion to #420 (PR #669) ([#650](https://github.com/khuisman/mcp-gee-sweet/issues/650))
@@ -214,6 +214,7 @@ Same shape as v0.8.1 — stabilize on defects that surfaced since v0.9.0 shipped
 - [x] auth: under `--transport sse`, the OAuth consent wait runs synchronously on the event loop, stalling every other session and ignoring SIGTERM until consent or timeout. #811 follow-up, found in PR #828 ([#833](https://github.com/khuisman/mcp-gee-sweet/issues/833)) (PR #867)
 - [x] Tool errors are opaque on PyPI installs: `mcp` 2.3 (what `uvx` resolves; the lock pins 2.0.0) hides any non-`ToolError` exception text, so #811's "run `mcp-gee-sweet auth`" message and uncaught Google API errors reach the client as a bare `Error executing tool <name>`. Observed live. Urgent: lane-a, next after #844 ([#872](https://github.com/khuisman/mcp-gee-sweet/issues/872)) (PR #905)
 - [ ] auth: offer OAuth re-authorization from the failing tool call, via URL-mode elicitation with the consent URL in a `ToolError` as fallback, then rebuild services in place. Covers a refresh token revoked mid-session too. No unprompted browser. Lane-a, after #872 ([#873](https://github.com/khuisman/mcp-gee-sweet/issues/873))
+- [ ] sheets: an A1 range with a dangling colon (`A1:`, `A:`, `5:`, `:5`, `:B`, `:B5`) passes `_parse_a1_notation` and widens through `_grid_range` to the whole sheet or a bigger rectangle. Hits every `_grid_range_or_error` tool; for `sort_range`/`merge_cells` it changes data. Found reviewing community PR #922, which rejects only a bare `":"`. Start after #922 merges (same lines) ([#926](https://github.com/khuisman/mcp-gee-sweet/issues/926))
 
 **Sheets hardening** _(triaged out of `backlog` 2026-09-11 — see the "Backlog triage" note below)_
 - [x] `update_sheet_properties`/dimension-tool hardening + dedup follow-ups from PR #321's review ([#323](https://github.com/khuisman/mcp-gee-sweet/issues/323)) (PR #734)
@@ -272,7 +273,7 @@ Same shape as v0.8.1 — stabilize on defects that surfaced since v0.9.0 shipped
 **Calendar hardening** _(triaged out of `backlog` 2026-09-11)_
 
 **Testing** _(triaged out of `backlog` 2026-09-11)_
-- [ ] **Bundle:** test-suite tightening, one PR closing [#392](https://github.com/khuisman/mcp-gee-sweet/issues/392), [#526](https://github.com/khuisman/mcp-gee-sweet/issues/526), [#551](https://github.com/khuisman/mcp-gee-sweet/issues/551), [#561](https://github.com/khuisman/mcp-gee-sweet/issues/561), [#574](https://github.com/khuisman/mcp-gee-sweet/issues/574) — lane B ([#891](https://github.com/khuisman/mcp-gee-sweet/issues/891))
+- [x] **Bundle:** test-suite tightening, one PR closing [#392](https://github.com/khuisman/mcp-gee-sweet/issues/392), [#526](https://github.com/khuisman/mcp-gee-sweet/issues/526), [#551](https://github.com/khuisman/mcp-gee-sweet/issues/551), [#561](https://github.com/khuisman/mcp-gee-sweet/issues/561), [#574](https://github.com/khuisman/mcp-gee-sweet/issues/574) — lane B (PR #924) ([#891](https://github.com/khuisman/mcp-gee-sweet/issues/891))
 - [ ] `qa`: `search_files` has no unit test coverage for the error-dict path ([#573](https://github.com/khuisman/mcp-gee-sweet/issues/573))
 
 **Infrastructure**

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/src/mcp_gee_sweet/tools/sheets/helpers.py

@@ -65,9 +65,9 @@ def _parse_a1_notation(range_str: str) -> dict[str, int]:
     if not range_str:
         raise ValueError("Invalid A1 notation: empty string")
 
-    match = re.match(r"^([A-Z]+)?(\d+)?(?::([A-Z]+)?(\d+)?)?$", range_str.upper())
+    match = re.fullmatch(r"([A-Z]+)?(\d+)?(?::([A-Z]+)?(\d+)?)?", range_str.upper())
 
-    if not match:
+    if not match or not any(match.groups()):
         raise ValueError(f"Invalid A1 notation: {range_str}")
 
     start_col, start_row, end_col, end_row = match.groups()

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/drive/test_files.py

@@ -34,6 +34,14 @@ _drive_tool, _drive_tools = _make_tool_registry()
 drive_files_module.register(_drive_tool)
 
 
+def _assert_error_list(result, underlying: str) -> None:
+    """The "error dict, not raised" contract: one {"error": ...} item carrying the
+    underlying exception's text. Doesn't pin the message's prefix wording (#574)."""
+    assert len(result) == 1
+    assert set(result[0]) == {"error"}
+    assert underlying in result[0]["error"]
+
+
 class TestSearchSpreadsheets:
     """Bug: single quotes in query were interpolated raw into the Drive query string."""
 
@@ -77,7 +85,7 @@ class TestSearchSpreadsheets:
         )
         ctx = _make_ctx(drive_service=drive_svc)
         result = await _drive_tools["search_spreadsheets"](query="budget 2024", ctx=ctx)
-        assert result == [{"error": "Search failed: simulated API failure"}]
+        _assert_error_list(result, "simulated API failure")
 
     async def test_requests_starred_field(self):
         """#388: search_spreadsheets shares search_files' starred exposure."""
@@ -184,7 +192,7 @@ class TestSearchFiles:
         )
         ctx = _make_ctx(drive_service=svc)
         result = await _drive_tools["search_files"](query="budget", ctx=ctx)
-        assert result == [{"error": "Search failed: simulated API failure"}]
+        _assert_error_list(result, "simulated API failure")
 
 
 class TestFileMutations:
@@ -721,7 +729,7 @@ class TestListSharedWithMe:
         )
         ctx = _make_ctx(drive_service=svc)
         result = await _drive_tools["list_shared_with_me"](ctx=ctx)
-        assert result == [{"error": "List shared with me failed: simulated API failure"}]
+        _assert_error_list(result, "simulated API failure")
 
     async def test_max_results_capped_at_200(self):
         svc = self._drive_service()
@@ -806,7 +814,7 @@ class TestListRecentFiles:
         )
         ctx = _make_ctx(drive_service=svc)
         result = await _drive_tools["list_recent_files"](ctx=ctx)
-        assert result == [{"error": "List recent files failed: simulated API failure"}]
+        _assert_error_list(result, "simulated API failure")
 
 
 class TestListFiles:
@@ -920,7 +928,7 @@ class TestListFiles:
         )
         ctx = self._ctx(svc)
         result = await _drive_tools["list_files"](folder_id="folder1", ctx=ctx)
-        assert result == [{"error": "List files failed: simulated API failure"}]
+        _assert_error_list(result, "simulated API failure")
 
     async def test_requests_next_page_token(self):
         """#698: nextPageToken's absence is what marks a listing exhaustive."""

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/drive/test_transfer.py

@@ -5115,11 +5115,22 @@ class TestDownloadFolder:
         assert result["downloaded"] == ["Doc One.pdf"]
         assert result["failed"] == []
 
-    async def test_skip_if_exists_true_skips_existing_and_does_not_redownload(self, tmp_path):
+    @pytest.mark.parametrize(
+        ("skip_if_exists", "skipped", "downloaded", "export_calls", "final_bytes"),
+        [
+            (True, ["Doc One.pdf"], [], 0, b"already here"),
+            (False, [], ["Doc One.pdf"], 1, b"new content"),
+        ],
+        ids=["skip", "redownload"],
+    )
+    async def test_skip_if_exists_with_existing_destination(
+        self, tmp_path, skip_if_exists, skipped, downloaded, export_calls, final_bytes
+    ):
         """TC-D109 (v0.8.0 QA run, #227): skip_if_exists was live-SKIPped as
-        local-filesystem-dependent and had no unit coverage. A destination file
-        that already exists must be left untouched — the candidate never reaches
-        export()/get_media() at all."""
+        local-filesystem-dependent and had no unit coverage. With the flag on, an
+        existing destination file is left untouched and the candidate never
+        reaches export(); with it off, the file is overwritten by a fresh export.
+        One parametrized test so the shared mock can't drift between cases (#561)."""
         svc = MagicMock()
         svc.files.return_value.list.return_value.execute.return_value = {
             "files": [
@@ -5130,6 +5141,7 @@ class TestDownloadFolder:
                 },
             ]
         }
+        svc.files.return_value.export.return_value.execute.return_value = b"new content"
         dest = tmp_path / "Doc One.pdf"
         dest.write_bytes(b"already here")
 
@@ -5137,44 +5149,14 @@ class TestDownloadFolder:
             folder_id="root",
             local_path=str(tmp_path),
             export_format="pdf",
-            skip_if_exists=True,
+            skip_if_exists=skip_if_exists,
             ctx=self._ctx(svc),
         )
-        assert result["skipped"] == ["Doc One.pdf"]
-        assert result["downloaded"] == []
+        assert result["skipped"] == skipped
+        assert result["downloaded"] == downloaded
         assert result["failed"] == []
-        svc.files.return_value.export.assert_not_called()
-        assert dest.read_bytes() == b"already here"
-
-    async def test_skip_if_exists_false_redownloads_existing_file(self, tmp_path):
-        """Companion to the skip_if_exists=True case above: with the flag off, an
-        existing destination file is not treated as a skip candidate and gets
-        overwritten by a fresh export."""
-        svc = MagicMock()
-        svc.files.return_value.list.return_value.execute.return_value = {
-            "files": [
-                {
-                    "id": "doc1",
-                    "name": "Doc One",
-                    "mimeType": "application/vnd.google-apps.document",
-                },
-            ]
-        }
-        svc.files.return_value.export.return_value.execute.return_value = b"new content"
-        dest = tmp_path / "Doc One.pdf"
-        dest.write_bytes(b"stale content")
-
-        result = await _transfer_tools["download_folder"](
-            folder_id="root",
-            local_path=str(tmp_path),
-            export_format="pdf",
-            skip_if_exists=False,
-            ctx=self._ctx(svc),
-        )
-        assert result["downloaded"] == ["Doc One.pdf"]
-        assert result["skipped"] == []
-        svc.files.return_value.export.assert_called_once()
-        assert dest.read_bytes() == b"new content"
+        assert svc.files.return_value.export.call_count == export_calls
+        assert dest.read_bytes() == final_bytes
 
     async def test_mime_type_filter_appended_to_drive_query(self, tmp_path):
         """TC-D110 (v0.8.0 QA run, #227): mime_type_filter is delegated entirely to
@@ -5227,6 +5209,13 @@ class TestDownloadFile:
     def _metadata(self, name, mime_type):
         return {"name": name, "mimeType": mime_type}
 
+    def _assert_metadata_call(self, svc, file_id):
+        """#551: supportsAllDrives=True is what lets a shared-drive file
+        resolve, and `fields` must include both keys download_file reads."""
+        svc.files.return_value.get.assert_called_once_with(
+            fileId=file_id, fields="name, mimeType", supportsAllDrives=True
+        )
+
     async def test_google_doc_export(self, tmp_path):
         svc = MagicMock()
         svc.files.return_value.get.return_value.execute.return_value = self._metadata(
@@ -5244,6 +5233,7 @@ class TestDownloadFile:
         svc.files.return_value.export.assert_called_once_with(
             fileId="doc1", mimeType="application/pdf"
         )
+        self._assert_metadata_call(svc, "doc1")
         dest = tmp_path / "My Doc.pdf"
         assert result == {
             "local_path": str(dest),
@@ -5274,6 +5264,30 @@ class TestDownloadFile:
         assert result["local_path"] == str(dest)
         assert dest.read_bytes() == b"xlsx bytes"
 
+    async def test_str_export_content_is_utf8_encoded(self, tmp_path):
+        """#551: googleapiclient can hand back a text export (csv/html/txt) as
+        str rather than bytes; download_file encodes it as UTF-8 before writing."""
+        svc = MagicMock()
+        svc.files.return_value.get.return_value.execute.return_value = self._metadata(
+            "My Sheet", "application/vnd.google-apps.spreadsheet"
+        )
+        text = "name,city\nZoë,Zürich\n"
+        svc.files.return_value.export.return_value.execute.return_value = text
+
+        result = await _transfer_tools["download_file"](
+            file_id="sheet1",
+            local_path=str(tmp_path),
+            export_format="csv",
+            ctx=self._ctx(svc),
+        )
+
+        svc.files.return_value.export.assert_called_once_with(fileId="sheet1", mimeType="text/csv")
+        self._assert_metadata_call(svc, "sheet1")
+        dest = tmp_path / "My Sheet.csv"
+        expected = text.encode("utf-8")
+        assert dest.read_bytes() == expected
+        assert result["size_bytes"] == len(expected)
+
     async def test_explicit_local_path_overrides_drive_filename(self, tmp_path):
         """When local_path names an exact file (not a directory), that name is
         used as-is rather than Drive's own name + a guessed extension — the
@@ -5330,6 +5344,7 @@ class TestDownloadFile:
         svc.files.return_value.get_media.assert_called_once_with(
             fileId="bin1", supportsAllDrives=True
         )
+        self._assert_metadata_call(svc, "bin1")
         dest = tmp_path / "photo.png"
         assert result == {"local_path": str(dest), "name": "photo.png", "size_bytes": len(content)}
         assert dest.read_bytes() == content

mcp_gee_sweet-0.9.2.dev243/tests/fakes.py

@@ -0,0 +1,27 @@
+"""Shared fake Google API services for unit tests."""
+
+from types import SimpleNamespace
+
+
+class FakeSheetsService:
+    """A minimal spreadsheets().get(...).execute() stand-in. Give it either
+    `result` (returned by every execute() call) or `exception` (raised
+    instead). Shared so a change to the mocked call shape is made once
+    (#392)."""
+
+    _http = SimpleNamespace(credentials=None)
+
+    def __init__(self, *, result: dict | None = None, exception: Exception | None = None):
+        self._result = result
+        self._exception = exception
+
+    def spreadsheets(self):
+        return self
+
+    def get(self, spreadsheetId, fields):
+        return self
+
+    def execute(self, **kwargs):
+        if self._exception is not None:
+            raise self._exception
+        return self._result

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/sheets/test_helpers.py

@@ -1,5 +1,3 @@
-from types import SimpleNamespace
-
 import pytest
 
 from mcp_gee_sweet.cache import SheetInfo, SheetStructureCache
@@ -11,6 +9,8 @@ from mcp_gee_sweet.tools.sheets.helpers import (
     _parse_a1_notation,
 )
 
+from ..fakes import FakeSheetsService
+
 
 class TestColumnIndexToLetter:
     def test_single_letters(self):
@@ -48,6 +48,11 @@ class TestLetterToColumnIndex:
 
 
 class TestParseA1Notation:
+    @pytest.mark.parametrize("range_str", [":", "A1\n", "1:2\n", "B:D\n"])
+    def test_incomplete_or_trailing_newline_range_raises(self, range_str):
+        with pytest.raises(ValueError, match="Invalid A1 notation"):
+            _parse_a1_notation(range_str)
+
     def test_single_cell(self):
         result = _parse_a1_notation("A1")
         assert result["startColumnIndex"] == 0
@@ -133,31 +138,6 @@ class TestParseA1Notation:
             _parse_a1_notation("B1:A1")
 
 
-class _FakeSheetsService:
-    """A minimal spreadsheets().get(...).execute() stand-in. Give it either
-    `result` (returned by every execute() call) or `exception` (raised
-    instead) — covers the transient-failure, genuine-not-found, and
-    real-match fixture shapes that used to be three separate, near-identical
-    hand-rolled classes (PR #754 review)."""
-
-    _http = SimpleNamespace(credentials=None)
-
-    def __init__(self, *, result: dict | None = None, exception: Exception | None = None):
-        self._result = result
-        self._exception = exception
-
-    def spreadsheets(self):
-        return self
-
-    def get(self, spreadsheetId, fields):
-        return self
-
-    def execute(self, **kwargs):
-        if self._exception is not None:
-            raise self._exception
-        return self._result
-
-
 _ONE_OTHER_SHEET = {"sheets": [{"properties": {"title": "Other", "sheetId": 0}}]}
 _TWO_SHEETS = {
     "sheets": [
@@ -167,21 +147,21 @@ _TWO_SHEETS = {
 }
 
 
-def _raising_service() -> _FakeSheetsService:
+def _raising_service() -> FakeSheetsService:
     """Simulates a transient API failure (rate limit, timeout, auth hiccup)."""
-    return _FakeSheetsService(exception=TimeoutError("simulated transient API failure"))
+    return FakeSheetsService(exception=TimeoutError("simulated transient API failure"))
 
 
-def _empty_service() -> _FakeSheetsService:
+def _empty_service() -> FakeSheetsService:
     """A real API response where the sheet genuinely doesn't exist."""
-    return _FakeSheetsService(result=_ONE_OTHER_SHEET)
+    return FakeSheetsService(result=_ONE_OTHER_SHEET)
 
 
-def _matching_service() -> _FakeSheetsService:
+def _matching_service() -> FakeSheetsService:
     """A real API response with more than one sheet, so a happy-path lookup
     exercises picking the right match rather than trivially returning the
     only entry present."""
-    return _FakeSheetsService(result=_TWO_SHEETS)
+    return FakeSheetsService(result=_TWO_SHEETS)
 
 
 class TestGetSheetIdExceptionPropagation:

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/sheets/test_structure.py

@@ -3,6 +3,8 @@
 from itertools import groupby
 from unittest.mock import AsyncMock, MagicMock
 
+import pytest
+
 from mcp_gee_sweet.cache import SheetInfo
 from mcp_gee_sweet.tools.sheets import structure as sheets_structure_module
 
@@ -1260,6 +1262,19 @@ class TestResizeColumns:
 
 
 class TestFormatCells:
+    @pytest.mark.parametrize("range_str", [":", "A1\n"])
+    async def test_malformed_range_does_not_format_any_cells(self, range_str):
+        svc = self._sheets_service()
+        result = await _structure_tools["format_cells"](
+            spreadsheet_id="ss1",
+            sheet="Sheet1",
+            range=range_str,
+            bold=True,
+            ctx=_make_ctx(sheets_service=svc, cache=None),
+        )
+        assert "error" in result
+        svc.spreadsheets.return_value.batchUpdate.assert_not_called()
+
     def _sheets_service(self, sheet_id=0):
         mock = MagicMock()
         mock.spreadsheets.return_value.get.return_value.execute.return_value = {

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/test_cache.py

@@ -19,6 +19,8 @@ from mcp_gee_sweet.cache import (
 )
 from mcp_gee_sweet.tools.sheets.helpers import _get_sheet_id
 
+from .fakes import FakeSheetsService
+
 # All tests use an in-memory SQLite database.
 DB = ":memory:"
 
@@ -458,24 +460,10 @@ class TestCalendarCacheSetTtl:
         assert cache.get_ttl() == 5
 
 
-class _FakeSheetsService:
-    """Returns a spreadsheet with a single sheet named 'New', id 1."""
-
-    _http = SimpleNamespace(credentials=None)
-
-    class _Spreadsheets:
-        class _Request:
-            def execute(self, **kwargs):
-                return {
-                    "properties": {"title": "New Title"},
-                    "sheets": [{"properties": {"title": "New", "sheetId": 1}}],
-                }
-
-        def get(self, spreadsheetId, fields):
-            return self._Request()
-
-    def spreadsheets(self):
-        return self._Spreadsheets()
+_NEW_SHEET = {
+    "properties": {"title": "New Title"},
+    "sheets": [{"properties": {"title": "New", "sheetId": 1}}],
+}
 
 
 class _FakeDriveService:
@@ -500,7 +488,9 @@ class TestFetchSheetsModifiedTimeValidation:
         cache = SheetStructureCache(db_path=DB, ttl=1000)
         cache.store("sid", [SheetInfo(title="Old", sheet_id=0)], modified_time="v1")
 
-        result = await fetch_sheets(_FakeSheetsService(), "sid", cache, _FakeDriveService())
+        result = await fetch_sheets(
+            FakeSheetsService(result=_NEW_SHEET), "sid", cache, _FakeDriveService()
+        )
 
         assert len(result) == 1
         assert result[0].title == "New"
@@ -510,8 +500,8 @@ class TestFetchSheetsModifiedTimeValidation:
         cache.store("sid", [SheetInfo(title="Old", sheet_id=0)], modified_time="v1")
 
         # No drive_service passed → no modifiedTime check → TTL-valid cache hit,
-        # even though _FakeSheetsService would return a different sheet.
-        result = await fetch_sheets(_FakeSheetsService(), "sid", cache)
+        # even though the Sheets fake would return a different sheet.
+        result = await fetch_sheets(FakeSheetsService(result=_NEW_SHEET), "sid", cache)
 
         assert result[0].title == "Old"
 
@@ -533,7 +523,7 @@ class TestGetSheetIdModifiedTimePropagation:
         cache.mark_dirty("sid")  # force the next lookup to be a genuine miss
 
         sheet_id = await _get_sheet_id(
-            _FakeSheetsService(), "sid", "New", cache, _FakeDriveService()
+            FakeSheetsService(result=_NEW_SHEET), "sid", "New", cache, _FakeDriveService()
         )
 
         assert sheet_id == 1

{mcp_gee_sweet-0.9.2.dev238 → mcp_gee_sweet-0.9.2.dev243}/tests/test_server.py

@@ -166,6 +166,11 @@ class TestAuthStatusResource:
         assert "create_spreadsheet" in quota["tools"]
         assert "transfer_ownership" not in quota["tools"]
         assert "storage quota" in quota["reason"].lower()
+        # sync_folder is listed by exact name, with its direction caveat in the
+        # reason text (#516); guard the caveat against a reword dropping it (#526).
+        assert "sync_folder" in quota["tools"]
+        assert "sync_folder" in quota["reason"]
+        assert "upload and bidirectional" in quota["reason"]
         assert quota["alternatives"] is not None
 
     def test_service_account_transfer_ownership_limitation(self):