fix: use return instead of raise StopIteration in QuerySet.__iter__

Per PEP 479, StopIteration raised inside a generator is converted to
RuntimeError. QuerySet.__iter__ is a generator and was raising
StopIteration to handle 400006 (invalid page number) errors, causing
RuntimeError for callers iterating endpoints that don't support
pagination past the last page.

Fixes #1786

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: jacalata

tableauserverclient/server/query.py

@@ -84,8 +84,9 @@ def __iter__(self: Self) -> Iterator[T]:
                 if e.code == "400006":
                     # If the endpoint does not support pagination, it will end
                     # up overrunning the total number of pages. Catch the
-                    # error and break out of the loop.
-                    raise StopIteration
+                    # error and return to cleanly end the generator.
+                    # (raise StopIteration would cause RuntimeError per PEP 479)
+                    return
             if len(self._result_cache) == 0:
                 return
             yield from self._result_cache

test/test_pager.py

@@ -136,3 +136,28 @@ def test_queryset_no_matches(server: TSC.Server) -> None:
         all_groups = server.groups.all()
         groups = list(all_groups)
     assert len(groups) == 0
+
+
+def test_queryset_400006_returns_cleanly(server: TSC.Server) -> None:
+    """Regression test for PEP 479: 400006 on page 2 must not raise RuntimeError."""
+    page1_xml = b"""<?xml version='1.0' encoding='UTF-8'?>
+<tsResponse xmlns='http://tableau.com/api'>
+  <pagination pageNumber='1' pageSize='1' totalAvailable='2'/>
+  <projects><project id='abc' name='Test' contentPermissions='ManagedByOwner'/></projects>
+</tsResponse>"""
+    error_xml = b"""<?xml version='1.0' encoding='UTF-8'?>
+<tsResponse xmlns='http://tableau.com/api'>
+  <error code='400006'><summary>Bad Request</summary><detail>Invalid page number</detail></error>
+</tsResponse>"""
+
+    with requests_mock.mock() as m:
+        m.get(
+            requests_mock.ANY,
+            [
+                {"content": page1_xml, "status_code": 200},
+                {"content": error_xml, "status_code": 400},
+            ],
+        )
+        # Before the fix this raised RuntimeError: generator raised StopIteration
+        results = list(server.projects.all())
+    assert len(results) == 1