fix: rewrite upload_large.py to use resumable_upload with progress and real file

Author: vgrem

examples/onedrive/files/sharing.py

@@ -31,7 +31,7 @@
 
 # 1. Create an anonymous view link
 link = item.create_link("view", "anonymous").execute_query()
-print(f"Anonymous link: {link.link.web_url}")
+print(f"Anonymous link: {link.link.webUrl}")
 
 # 2. Send a sharing invitation to a user
 invite = item.invite(
@@ -45,5 +45,4 @@
 permissions = item.permissions.get().execute_query()
 print(f"Permissions ({len(permissions)}):")
 for p in permissions:
-    roles = p.roles or []
-    print(f"  ID: {p.id}  roles: {', '.join(roles)}")
+    print(f"  ID: {p.id}  roles: {', '.join(p.roles)}")

examples/onedrive/files/upload_large.py

@@ -1,21 +1,33 @@
 """
 Upload a large file using a resumable upload session.
 
-Files larger than ~4 MB should use the resumable upload session.
-The SDK handles chunked upload automatically.
+Reads from disk in chunks with progress reporting.
+No full file loaded into memory.
 
-Requires delegated permission ``Files.ReadWrite``.
-
-https://learn.microsoft.com/en-us/graph/api/driveitem-createuploadsession
+Requires delegated permission Files.ReadWrite.
 """
 
+import os
+from pathlib import Path
+
 from office365.graph_client import GraphClient
 from tests import test_client_id, test_password, test_tenant, test_username
 
-client = GraphClient(tenant=test_tenant).with_username_and_password(test_client_id, test_username, test_password)
 
-# Create a 5 MB file in memory
-content = b"A" * 5_000_000
-uploaded = client.me.drive.root.upload("large_file.bin", content).execute_query()
-print(f"Uploaded: {uploaded.name}  ({len(content)} bytes)")
-print(f"Web URL: {uploaded.web_url}")
+def main():
+    client = GraphClient(tenant=test_tenant).with_username_and_password(test_client_id, test_username, test_password)
+
+    local_path = Path("../../../tests/data/big_buck_bunny.mp4")
+    file_size = os.path.getsize(local_path)
+
+    def print_progress(uploaded_bytes: int) -> None:
+        pct = uploaded_bytes / file_size * 100
+        print(f"  Uploaded {uploaded_bytes:,} / {file_size:,} bytes ({pct:.1f}%)")
+
+    print(f"Uploading {local_path.name} ({file_size:,} bytes)...")
+    uploaded = client.me.drive.root.resumable_upload(str(local_path), chunk_uploaded=print_progress).execute_query()
+    print(f"Uploaded: {uploaded.web_url}")
+
+
+if __name__ == "__main__":
+    main()

generator/.checkpoints/SharePoint.xml.json

@@ -1,6 +1,6 @@
 [sharepoint]
 metadata_path = ./metadata/SharePoint.xml
-output_path = ../office365/sharepoint
+output_path = ../office365
 template_path = ./templates/sharepoint
 include_base_types = ComplexType
 

generator/settings.sharepoint.cfg

@@ -11,6 +11,7 @@
 from office365.runtime.client_value_collection import ClientValueCollection
 from office365.runtime.paths.resource_path import ResourcePath
 from office365.runtime.types.collections import StringCollection
+from office365.runtime.types.odata_property import odata
 
 
 class Organization(DirectoryObject):
@@ -19,6 +20,7 @@ class Organization(DirectoryObject):
     which operate and are provisioned at the tenant-level.
     """
 
+    @odata(name="assignedPlans")
     @property
     def assigned_plans(self) -> ClientValueCollection[AssignedPlan]:
         """The plans that are assigned to the organization."""
@@ -31,6 +33,7 @@ def branding(self) -> OrganizationalBranding:
             OrganizationalBranding(self.context, ResourcePath("branding", self.resource_path)),
         )
 
+    @odata(name="businessPhones")
     @property
     def business_phones(self) -> StringCollection:
         """
@@ -47,6 +50,7 @@ def extensions(self) -> EntityCollection[Extension]:
             EntityCollection(self.context, Extension, ResourcePath("extensions", self.resource_path)),
         )
 
+    @odata(name="certificateBasedAuthConfiguration")
     @property
     def certificate_based_auth_configuration(self) -> EntityCollection[CertificateBasedAuthConfiguration]:
         """Navigation property to manage certificate-based authentication configuration.
@@ -61,23 +65,13 @@ def certificate_based_auth_configuration(self) -> EntityCollection[CertificateBa
             ),
         )
 
+    @odata(name="provisionedPlans")
     @property
     def provisioned_plans(self) -> ClientValueCollection[ProvisionedPlan]:
         return self.properties.get("provisionedPlans", ClientValueCollection(ProvisionedPlan))
 
+    @odata(name="verifiedDomains")
     @property
     def verified_domains(self) -> ClientValueCollection[VerifiedDomain]:
         """The collection of domains associated with this tenant."""
         return self.properties.get("verifiedDomains", ClientValueCollection(VerifiedDomain))
-
-    def get_property(self, name, default_value=None):
-        if default_value is None:
-            property_mapping = {
-                "assignedPlans": self.assigned_plans,
-                "certificateBasedAuthConfiguration": self.certificate_based_auth_configuration,
-                "businessPhones": self.business_phones,
-                "provisionedPlans": self.provisioned_plans,
-                "verifiedDomains": self.verified_domains,
-            }
-            default_value = property_mapping.get(name, None)
-        return super().get_property(name, default_value)

office365/intune/organizations/organization.py

@@ -1,5 +1,14 @@
 from office365.entity import Entity
+from office365.runtime.paths.builder import ODataPathBuilder
 
 
 class OnenoteEntityBaseModel(Entity):
     """This is the base type for OneNote entities."""
+
+    def set_property(self, name, value, persist_changes=True):
+        super().set_property(name, value, persist_changes)
+        if name == "self" and self.resource_url != value:
+            # Fallback to canonical path
+            path_str = value.replace(self.context.service_root_url, "")
+            self._resource_path = ODataPathBuilder.parse_url(path_str)
+        return self

office365/onenote/entity_base_model.py

@@ -41,5 +41,5 @@ def section_groups(self):
         )
 
     @property
-    def entity_type_name(self):
+    def entity_type_name(self) -> str:
         return None  # type: ignore

office365/onenote/notebooks/notebook.py

@@ -101,5 +101,5 @@ def parent_section_group(self) -> SectionGroup:
         )
 
     @property
-    def entity_type_name(self):
+    def entity_type_name(self) -> str:
         return None  # type: ignore

office365/onenote/sections/section.py

@@ -0,0 +1,18 @@
+from __future__ import annotations
+
+from dataclasses import field
+
+from office365.runtime.client_value import ClientValue
+from office365.runtime.types.collections import StringCollection
+
+
+class TenantServiceInfoValue(ClientValue):
+    deletedDateTime: str | None = None
+    objectId: str | None = None
+    serviceElements: StringCollection = field(default_factory=StringCollection)
+    serviceInstance: str | None = None
+    version: int | None = None
+
+    @property
+    def entity_type_name(self) -> str:
+        return "SP.Directory.TenantServiceInfoValue"