fix: copilot suggestions

Author: jorwoods

tableauserverclient/models/extensions_item.py

@@ -40,7 +40,7 @@ def from_response(cls: type[Self], response, ns) -> Self:
 
         if (enabled_element := element.find("./t:extensionsGloballyEnabled", namespaces=ns)) is not None:
             obj.enabled = string_to_bool(enabled_element.text)
-        obj.block_list = [e.text for e in element.findall("./t:blockList", namespaces=ns)]
+        obj.block_list = [e.text for e in element.findall("./t:blockList", namespaces=ns) if e.text is not None]
 
         return obj
 

tableauserverclient/models/property_decorators.py

@@ -1,7 +1,7 @@
 import datetime
 import re
 from functools import wraps
-from typing import Any, Optional, Tuple
+from typing import Any
 from collections.abc import Container
 
 from tableauserverclient.datetime_helpers import parse_datetime
@@ -67,7 +67,7 @@ def wrapper(self, value):
     return wrapper
 
 
-def property_is_int(range: tuple[int, int], allowed: Optional[Container[Any]] = None):
+def property_is_int(range: tuple[int, int], allowed: Container[Any] | None = None):
     """Takes a range of ints and a list of exemptions to check against
     when setting a property on a model. The range is a tuple of (min, max) and the
     allowed list (empty by default) allows values outside that range.

tableauserverclient/server/endpoint/extensions_endpoint.py

@@ -53,7 +53,7 @@ def get(self) -> ExtensionsSiteSettings:
 
         Returns
         -------
-        list[ExtensionsSiteSettings]
+        ExtensionsSiteSettings
             The site extensions settings
         """
         response = self.get_request(self.baseurl)

test/assets/extensions_site_settings.xml

@@ -5,7 +5,7 @@
         <useDefaultSetting>false</useDefaultSetting>
         <allowTrusted>true</allowTrusted>
         <includeSandboxed>false</includeSandboxed>
-        <includeTableauBuilt>>false</includeTableauBuilt>
+        <includeTableauBuilt>false</includeTableauBuilt>
         <includePartnerBuilt>false</includePartnerBuilt>
         <safeList>
             <url>http://localhost:9123/Dynamic.html</url>