Fix Copilot review round 2: update docstring example to match parens, add assert_called_once to tests

- Update _list_tables docstring example to show parenthesized filter format
- Add assert_called_once() to 5 test methods that read call_args without
  first verifying the mock was called exactly once

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: tpellissier

src/PowerPlatform/Dataverse/data/_odata.py

@@ -1433,7 +1433,7 @@ def _list_tables(
         :param filter: Optional additional OData ``$filter`` expression that is
             combined with the default ``IsPrivate eq false`` clause using
             ``and``.  For example, ``"SchemaName eq 'Account'"`` becomes
-            ``"IsPrivate eq false and SchemaName eq 'Account'"``.
+            ``"IsPrivate eq false and (SchemaName eq 'Account')"``.
             When ``None`` (the default), only the ``IsPrivate eq false`` filter
             is applied.
         :type filter: ``str`` or ``None``

tests/unit/data/test_odata_internal.py

@@ -165,6 +165,7 @@ def test_filter_none_same_as_no_filter(self):
         self._setup_response([])
         self.od._list_tables(filter=None)
 
+        self.od._request.assert_called_once()
         call_kwargs = self.od._request.call_args
         params = call_kwargs.kwargs.get("params") or call_kwargs[1].get("params", {})
         self.assertEqual(params["$filter"], "IsPrivate eq false")
@@ -194,6 +195,7 @@ def test_select_none_omits_query_param(self):
         self._setup_response([])
         self.od._list_tables(select=None)
 
+        self.od._request.assert_called_once()
         call_kwargs = self.od._request.call_args
         params = call_kwargs.kwargs.get("params") or call_kwargs[1].get("params", {})
         self.assertNotIn("$select", params)
@@ -203,6 +205,7 @@ def test_select_empty_list_omits_query_param(self):
         self._setup_response([])
         self.od._list_tables(select=[])
 
+        self.od._request.assert_called_once()
         call_kwargs = self.od._request.call_args
         params = call_kwargs.kwargs.get("params") or call_kwargs[1].get("params", {})
         self.assertNotIn("$select", params)
@@ -212,6 +215,7 @@ def test_select_preserves_case(self):
         self._setup_response([])
         self.od._list_tables(select=["EntitySetName", "LogicalName"])
 
+        self.od._request.assert_called_once()
         call_kwargs = self.od._request.call_args
         params = call_kwargs.kwargs.get("params") or call_kwargs[1].get("params", {})
         self.assertEqual(params["$select"], "EntitySetName,LogicalName")
@@ -238,6 +242,7 @@ def test_select_single_property(self):
         self._setup_response([])
         self.od._list_tables(select=["LogicalName"])
 
+        self.od._request.assert_called_once()
         call_kwargs = self.od._request.call_args
         params = call_kwargs.kwargs.get("params") or call_kwargs[1].get("params", {})
         self.assertEqual(params["$select"], "LogicalName")