msgraphfs 0.4__tar.gz → 0.5__tar.gz

{msgraphfs-0.4 → msgraphfs-0.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: msgraphfs
-Version: 0.4
+Version: 0.5
 Dynamic: Summary
 Project-URL: Source, https://github.com/acsone/msgraphfs
 Author-email: Laurent Mignon <laurent.mignon@acsone.eu>
@@ -230,3 +230,7 @@ To use this filesystem, you need to register an Azure AD application:
 6. Note the Application (client) ID, Directory (tenant) ID, and client secret
 
 The filesystem uses the OAuth2 client credentials flow with the default scope (`https://graph.microsoft.com/.default`), which automatically includes all application permissions granted to your Azure AD application.
+
+### Running the tests
+
+See [TESTING.md](TESTING.md) for how to run the test suite, including the live tests against a real SharePoint site.

{msgraphfs-0.4 → msgraphfs-0.5}/README.md

@@ -184,3 +184,7 @@ To use this filesystem, you need to register an Azure AD application:
 6. Note the Application (client) ID, Directory (tenant) ID, and client secret
 
 The filesystem uses the OAuth2 client credentials flow with the default scope (`https://graph.microsoft.com/.default`), which automatically includes all application permissions granted to your Azure AD application.
+
+### Running the tests
+
+See [TESTING.md](TESTING.md) for how to run the test suite, including the live tests against a real SharePoint site.

{msgraphfs-0.4 → msgraphfs-0.5}/src/msgraphfs/core.py

@@ -64,6 +64,15 @@ def parse_range_header(range_header):
         raise ValueError("Invalid Range header format")
 
 
+def split_parent_child(path: str) -> tuple[str, str]:
+    """Split a path into its parent path and its final component.
+
+    If ``path`` has no ``/`` separator, the parent is the empty string
+    (i.e. the item is considered a direct child of the root).
+    """
+    return path.rsplit("/", 1) if "/" in path else ("", path)
+
+
 def parse_msgraph_url(url_path):  # noqa: C901
     """Parse a msgraph URL to extract site_name, drive_name, and path.
 
@@ -655,7 +664,7 @@ class AbstractMSGraphFS(AsyncFileSystem):
             path1, item_id=source_item_id, action="copy"
         )
         path2 = self._strip_protocol(path2)
-        parent_path, _file_name = path2.rsplit("/", 1)
+        parent_path, _file_name = split_parent_child(path2)
         item_reference = await self._get_item_reference(parent_path)
         json = {
             "parentReference": item_reference,
@@ -721,7 +730,6 @@ class AbstractMSGraphFS(AsyncFileSystem):
             For example, if you want to expand the properties to include the thumbnails,
             you can pass "thumbnails" as the value of the expand parameter.
         """
-
         url = await self._path_to_url_async(path, item_id=item_id)
         params = {}
         if expand:
@@ -729,7 +737,7 @@ class AbstractMSGraphFS(AsyncFileSystem):
         response = await self._msgraph_get(url, params=params)
         return self._drive_item_info_to_fsspec_info(response.json())
 
-    async def _ls(
+    async def _ls(  # noqa: C901
         self,
         path: str,
         detail: bool = True,
@@ -768,13 +776,21 @@ class AbstractMSGraphFS(AsyncFileSystem):
             params = {"select": "name,parentReference"}
         if expand:
             params = {"expand": expand}
-        response = await self._msgraph_get(url, params=params)
-        result = response.json()
-        items = result.get("value", [])
-        while "@odata.nextLink" in result:
-            response = await self._msgraph_get(result["@odata.nextLink"])
+        items = []
+        try:
+            response = await self._msgraph_get(url, params=params)
             result = response.json()
-            items.extend(result.get("value", []))
+            items = result.get("value", [])
+            while "@odata.nextLink" in result:
+                response = await self._msgraph_get(result["@odata.nextLink"])
+                result = response.json()
+                items.extend(result.get("value", []))
+        except HTTPStatusError as e:
+            if (
+                not e.response.status_code == 422
+                and "getChildrenOnNonFolder" not in e.response.content
+            ):
+                raise e
         if not items:
             # maybe the path is a file
             try:
@@ -891,7 +907,7 @@ class AbstractMSGraphFS(AsyncFileSystem):
 
     async def _mkdir(self, path, create_parents=True, exist_ok=False, **kwargs) -> str:
         path = self._strip_protocol(path).rstrip("/")
-        parent, child = path.rsplit("/", 1)
+        parent, child = split_parent_child(path)
         parent_id = await self._get_item_id(parent)
         if not parent_id and not create_parents:
             raise FileNotFoundError(f"Parent directory does not exists: {parent}")
@@ -961,7 +977,7 @@ class AbstractMSGraphFS(AsyncFileSystem):
         if destination_item_id:
             item_reference = await self._get_item_reference(path2)
         else:
-            parent_path, name = path2.rsplit("/", 1)
+            parent_path, name = split_parent_child(path2)
             item_reference = await self._get_item_reference(parent_path)
         json = {
             "parentReference": item_reference,
@@ -1071,7 +1087,7 @@ class AbstractMSGraphFS(AsyncFileSystem):
                     url, json={"lastModifiedDateTime": datetime.now().isoformat()}
                 )
         else:
-            parent_path, file_name = path.rsplit("/", 1)
+            parent_path, file_name = split_parent_child(path)
             parent_id = await self._get_item_id(parent_path, throw_on_missing=True)
             item_id = f"{parent_id}:/{file_name}:"
             url = await self._path_to_url_async(path, item_id=item_id, action="content")
@@ -1095,7 +1111,6 @@ class AbstractMSGraphFS(AsyncFileSystem):
         refresh : bool (=False)
             if False, look in local cache for file details first
         """
-
         info = await self._info(path, refresh=refresh)
 
         if info["type"] != "directory":
@@ -1766,12 +1781,13 @@ class MSGDriveFS(AbstractMSGraphFS):
         }
 
     async def _get_recycle_bin_items(self) -> list[dict]:
-        """Get the items in the recycle bin. (Beta!!)
+        """Get the items in the recycle bin.
 
-        Returns:
-            list[dict]: A list of dictionaries with information about the items in the recycle bin.
+        (Beta!!)
+                Returns:
+                    list[dict]: A list of dictionaries with information about the items in the recycle bin.
 
-        see https://docs.microsoft.com/en-us/graph/api/resources/driveitem?view=graph-rest-1.0
+                see https://docs.microsoft.com/en-us/graph/api/resources/driveitem?view=graph-rest-1.0
         """
         site_id = await self._get_site_id()
         url = f"https://graph.microsoft.com/beta/sites/{site_id}/recycleBin/items"
@@ -1904,7 +1920,7 @@ class AsyncStreamedFileMixin:
         """
         item_id = await self.item_id
         if not item_id:
-            parent_path, file_name = self.path.rsplit("/", 1)
+            parent_path, file_name = split_parent_child(self.path)
             parent_id = await self.fs._get_item_id(parent_path)
             item_id = f"{parent_id}:/{file_name}:"
         url = self.fs._path_to_url(
@@ -1944,7 +1960,7 @@ class AsyncStreamedFileMixin:
             headers["content-type"] = self.fs._guess_type(self.path)
         item_id = await self.item_id
         if not item_id:
-            parent_path, file_name = self.path.rsplit("/", 1)
+            parent_path, file_name = split_parent_child(self.path)
             parent_id = await self.fs._get_item_id(parent_path, throw_on_missing=True)
             item_id = f"{parent_id}:/{file_name}:"
         url = self.fs._path_to_url(self.path, item_id=item_id, action="content")