feat: add vizWidth/vizHeight to PDFRequestOptions

Fixes #1102

- PDFRequestOptions already inherited viz_width/viz_height from
  _ImagePDFCommonExportOptions (shared with ImageRequestOptions), which
  serialises them as vizWidth/vizHeight query params
- Added version guard in views.populate_pdf: raises UnsupportedAttributeError
  when viz_height or viz_width are used below API 3.26
- Added tests for the new version guard, the happy path (API 3.26+), unit-
  level query-param serialisation, and the existing XOR validation
- Removed stray bare literal '44' from test_request_option.py

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

Author: jacalata

tableauserverclient/server/endpoint/views_endpoint.py

@@ -204,7 +204,8 @@ def populate_pdf(self, view_item: ViewItem, req_options: "PDFRequestOptions | No
 
         req_options: PDFRequestOptions | None, default None
             Optional request options for the request. These options can include
-            parameters such as orientation and paper size.
+            parameters such as orientation, paper size, and viz dimensions
+            (viz_width and viz_height, which require API version 3.26+).
 
         Returns
         -------
@@ -214,6 +215,11 @@ def populate_pdf(self, view_item: ViewItem, req_options: "PDFRequestOptions | No
             error = "View item missing ID."
             raise MissingRequiredFieldError(error)
 
+        if req_options is not None:
+            if not self.parent_srv.check_at_least_version("3.26"):
+                if req_options.viz_height or req_options.viz_width:
+                    raise UnsupportedAttributeError("viz_height and viz_width are only supported in API 3.26+")
+
         def pdf_fetcher():
             return self._get_view_pdf(view_item, req_options)
 

tableauserverclient/server/endpoint/workbooks_endpoint.py

@@ -604,12 +604,15 @@ def populate_pdf(self, workbook_item: WorkbookItem, req_options: "PDFRequestOpti
         def pdf_fetcher() -> bytes:
             return self._get_wb_pdf(workbook_item, req_options)
 
-        if not self.parent_srv.check_at_least_version("3.23") and req_options is not None:
-            if req_options.view_filters or req_options.view_parameters:
-                raise UnsupportedAttributeError("view_filters and view_parameters are only supported in 3.23+")
-
-            if req_options.viz_height or req_options.viz_width:
-                raise UnsupportedAttributeError("viz_height and viz_width are only supported in 3.23+")
+        if req_options is not None:
+            if not self.parent_srv.check_at_least_version("3.23"):
+                if req_options.view_filters or req_options.view_parameters:
+                    raise UnsupportedAttributeError("view_filters and view_parameters are only supported in 3.23+")
+            # vizWidth/vizHeight were added to the PDF endpoint in API 3.26
+            # (same version as views.populate_pdf; confirmed in REST API reference).
+            if not self.parent_srv.check_at_least_version("3.26"):
+                if req_options.viz_height or req_options.viz_width:
+                    raise UnsupportedAttributeError("viz_height and viz_width are only supported in 3.26+")
 
         workbook_item._set_pdf(pdf_fetcher)
         logger.info(f"Populated pdf for workbook (ID: {workbook_item.id})")

test/test_request_option.py

@@ -380,9 +380,6 @@ def test_queryset_endpoint_pagesize_filter(server: TSC.Server, page_size: int) -
         _ = list(queryset)
 
 
-44
-
-
 @pytest.mark.parametrize("page_size", [1, 10, 100, 1_000])
 def test_queryset_pagesize_filter(server: TSC.Server, page_size: int) -> None:
     with requests_mock.mock() as m:
@@ -441,3 +438,41 @@ def test_queryset_field_all(server: TSC.Server) -> None:
     fields = history.qs.get("fields", [""])[0]
 
     assert fields == "_all_"
+
+
+def test_pdf_viz_dimensions_query_params() -> None:
+    opts = TSC.PDFRequestOptions(viz_width=1920, viz_height=1080)
+    params = opts.get_query_params()
+    assert params["vizWidth"] == 1920
+    assert params["vizHeight"] == 1080
+
+
+def test_pdf_viz_dimensions_only_one_raises() -> None:
+    opts = TSC.PDFRequestOptions(viz_width=1920)
+    with pytest.raises(ValueError):
+        opts.get_query_params()
+
+    opts2 = TSC.PDFRequestOptions(viz_height=1080)
+    with pytest.raises(ValueError):
+        opts2.get_query_params()
+
+
+def test_pdf_viz_dimensions_none_by_default() -> None:
+    opts = TSC.PDFRequestOptions()
+    params = opts.get_query_params()
+    assert "vizWidth" not in params
+    assert "vizHeight" not in params
+
+
+def test_pdf_viz_dimensions_via_request(server: TSC.Server) -> None:
+    with requests_mock.mock() as m:
+        m.get(requests_mock.ANY)
+        url = server.views.baseurl + "/abc/pdf"
+        opts = TSC.PDFRequestOptions(viz_width=800, viz_height=600)
+
+        resp = server.views.get_request(url, request_object=opts)
+        query_string = parse_qs(resp.request.query)
+    assert "vizwidth" in query_string
+    assert ["800"] == query_string["vizwidth"]
+    assert "vizheight" in query_string
+    assert ["600"] == query_string["vizheight"]

test/test_view.py

@@ -427,7 +427,7 @@ def test_filter_excel(server: TSC.Server) -> None:
         assert response == excel_file
 
 
-def test_pdf_height(server: TSC.Server) -> None:
+def test_pdf_viz_dimensions_unsupported(server: TSC.Server) -> None:
     server.version = "3.8"
     response = POPULATE_PDF.read_bytes()
     with requests_mock.mock() as m:
@@ -438,6 +438,23 @@ def test_pdf_height(server: TSC.Server) -> None:
         single_view = TSC.ViewItem()
         single_view._id = "d79634e1-6063-4ec9-95ff-50acbf609ff5"
 
+        req_option = TSC.PDFRequestOptions(viz_height=1080, viz_width=1920)
+
+        with pytest.raises(UnsupportedAttributeError):
+            server.views.populate_pdf(single_view, req_option)
+
+
+def test_pdf_viz_dimensions(server: TSC.Server) -> None:
+    server.version = "3.26"
+    response = POPULATE_PDF.read_bytes()
+    with requests_mock.mock() as m:
+        m.get(
+            server.views.baseurl + "/d79634e1-6063-4ec9-95ff-50acbf609ff5/pdf?vizHeight=1080&vizWidth=1920",
+            content=response,
+        )
+        single_view = TSC.ViewItem()
+        single_view._id = "d79634e1-6063-4ec9-95ff-50acbf609ff5"
+
         req_option = TSC.PDFRequestOptions(
             viz_height=1080,
             viz_width=1920,

test/test_workbook.py

@@ -506,7 +506,8 @@ def test_populate_pdf_unsupported(server: TSC.Server) -> None:
 
 
 def test_populate_pdf_vf_dims(server: TSC.Server) -> None:
-    server.version = "3.23"
+    # vizWidth/vizHeight require API 3.26 on both workbooks and views populate_pdf
+    server.version = "3.26"
     server.workbooks.baseurl
     response = POPULATE_PDF.read_bytes()
     with requests_mock.mock() as m: