API: ls support last_modified property (#149)

* API: ls support last_modified property * API: ls support last_modified property
fsspec · Sep 26, 2024 · 5226daa · 5226daa
1 parent bf6b0f6
commit 5226daa
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/tosfs/core.py b/tosfs/core.py
@@ -2141,6 +2141,7 @@ def _fill_file_info(obj: ListedObject, bucket: str, versions: bool = False) -> d
             "size": obj.size,
             "name": f"{bucket}/{obj.key}",
             "type": "file",
+            "LastModified": obj.last_modified,
         }
         if (
             isinstance(obj, ListedObjectVersion)

diff --git a/tosfs/tests/test_fsspec.py b/tosfs/tests/test_fsspec.py
@@ -423,8 +423,8 @@ def test_find(fsspecfs: Any, bucket: str, temporary_workspace: str):
     def remove_last_modification_time_ms(data):
         if isinstance(data, dict):
             for key in data:
-                if "last_modification_time_ms" in data[key]:
-                    del data[key]["last_modification_time_ms"]
+                if "LastModified" in data[key]:
+                    del data[key]["LastModified"]
         return data
 
     file1_path = f"{bucket}/{temporary_workspace}/file1"

diff --git a/tosfs/tests/test_tosfs.py b/tosfs/tests/test_tosfs.py
@@ -48,7 +48,11 @@ def test_ls_dir(tosfs: TosFileSystem, bucket: str, temporary_workspace: str) ->
             f"Directory {temporary_workspace} not found in {detailed_list}"
         )
 
-    assert tosfs.ls(f"{bucket}/{temporary_workspace}", detail=False) == []
+    tosfs.touch(f"{bucket}/{temporary_workspace}/file")
+    for item in tosfs.ls(f"{bucket}/{temporary_workspace}", detail=True):
+        assert item["name"] in [f"{bucket}/{temporary_workspace}/file"]
+        assert item["LastModified"] is not None
+
     assert tosfs.ls(f"{bucket}/{temporary_workspace}/nonexistent", detail=False) == []
 
 
@@ -123,6 +127,12 @@ def test_info(tosfs: TosFileSystem, bucket: str, temporary_workspace: str) -> No
         "size": 0,
         "StorageClass": "DIRECTORY",
     }
+    tosfs.touch(f"{bucket}/{temporary_workspace}/file")
+    file_info = tosfs.info(f"{bucket}/{temporary_workspace}/file")
+    assert file_info["name"] == f"{bucket}/{temporary_workspace}/file"
+    assert file_info["type"] == "file"
+    assert file_info["size"] == 0
+    assert file_info["LastModified"] is not None
 
     with pytest.raises(FileNotFoundError):
         tosfs.info(f"{bucket}/nonexistent")