fix: override ServiceOperationPath.delimiter to prevent double slash when parent is UrlPath

Author: vgrem

office365/onedrive/driveitems/uploadable_properties.py

@@ -1,11 +1,13 @@
 from __future__ import annotations
 
 from dataclasses import dataclass, field
+from typing import Optional, Dict, Any
 
 from office365.onedrive.driveitems.source import DriveItemSource
 from office365.onedrive.files.system_info import FileSystemInfo
 from office365.onedrive.media_source import MediaSource
 from office365.runtime.client_value import ClientValue
+from office365.runtime.odata.json_format import ODataJsonFormat
 
 
 @dataclass
@@ -25,6 +27,12 @@ def file_size(self):
         """Provides an expected file size to perform a quota check prior to upload. Only on OneDrive Personal."""
         return self._fileSize
 
+    def to_json(self, json_format: Optional[ODataJsonFormat] = None) -> Dict[str, Any]:
+        payload =  super().to_json(json_format)
+        payload.pop("driveItemSource", None)
+        payload.pop("mediaSource", None)
+        return payload
+
     @property
     def entity_type_name(self) -> str:
-        return "microsoft.graph.DriveItemUploadableProperties"
+        return None # type: ignore

office365/runtime/client_value.py

@@ -76,6 +76,9 @@ def _is_valid_value(val):
                 return False
             elif isinstance(val, ClientValueCollection) and len(val) == 0:
                 return False
+            elif isinstance(val, ClientValue):
+                if not any(v is not None for v in vars(val).values()):
+                    return False
             return True
 
         result = {k: v for k, v in self if _is_valid_value(v)}

office365/runtime/paths/service_operation.py

@@ -29,6 +29,14 @@ def __init__(
         super().__init__(name, parent)
         self._parameters = parameters
 
+    @property
+    def delimiter(self):
+        from office365.onedrive.internal.paths.url import UrlPath
+
+        if isinstance(self.parent, UrlPath):
+            return ""
+        return "/"
+
     @property
     def segment(self) -> str:
         """Gets the OData path segment for this service operation."""