Fixes issue #477.

Unit tests for `FS.setinfo` now function correctly, whereas they basically did nothing before. Also, a bug in OSFS was fixed which caused timestamps to lose precision with `setinfo`.

Author: atollk

fs/info.py

@@ -135,6 +135,9 @@ def is_writeable(self, namespace, key):
         When creating an `Info` object, you can add a ``_write`` key to
         each raw namespace that lists which keys are writable or not.
 
+        In general, this means they are compatible with the `setinfo`
+        function of filesystem objects.
+
         Arguments:
             namespace (str): A namespace identifier.
             key (str): A key within the namespace.

fs/osfs.py

@@ -665,10 +665,10 @@ def setinfo(self, path, info):
         if "details" in info:
             details = info["details"]
             if "accessed" in details or "modified" in details:
-                _accessed = typing.cast(int, details.get("accessed"))
-                _modified = typing.cast(int, details.get("modified", _accessed))
-                accessed = int(_modified if _accessed is None else _accessed)
-                modified = int(_modified)
+                _accessed = typing.cast(float, details.get("accessed"))
+                _modified = typing.cast(float, details.get("modified", _accessed))
+                accessed = float(_modified if _accessed is None else _accessed)
+                modified = float(_modified)
                 if accessed is not None or modified is not None:
                     with convert_os_errors("setinfo", path):
                         os.utime(sys_path, (accessed, modified))

fs/test.py

@@ -12,7 +12,6 @@
 import io
 import itertools
 import json
-import math
 import os
 import time
 import unittest
@@ -1171,15 +1170,21 @@ def test_removetree_root(self):
 
     def test_setinfo(self):
         self.fs.create("birthday.txt")
-        now = math.floor(time.time())
+        now = time.time()
 
         change_info = {"details": {"accessed": now + 60, "modified": now + 60 * 60}}
         self.fs.setinfo("birthday.txt", change_info)
-        new_info = self.fs.getinfo("birthday.txt", namespaces=["details"]).raw
-        if "accessed" in new_info.get("_write", []):
-            self.assertEqual(new_info["details"]["accessed"], now + 60)
-        if "modified" in new_info.get("_write", []):
-            self.assertEqual(new_info["details"]["modified"], now + 60 * 60)
+        new_info = self.fs.getinfo("birthday.txt", namespaces=["details"])
+        can_write_acccess = new_info.is_writeable("details", "accessed")
+        can_write_modified = new_info.is_writeable("details", "modified")
+        if can_write_acccess:
+            self.assertAlmostEqual(
+                new_info.get("details", "accessed"), now + 60, places=4
+            )
+        if can_write_modified:
+            self.assertAlmostEqual(
+                new_info.get("details", "modified"), now + 60 * 60, places=4
+            )
 
         with self.assertRaises(errors.ResourceNotFound):
             self.fs.setinfo("nothing", {})
@@ -1188,10 +1193,11 @@ def test_settimes(self):
         self.fs.create("birthday.txt")
         self.fs.settimes("birthday.txt", accessed=datetime(2016, 7, 5))
         info = self.fs.getinfo("birthday.txt", namespaces=["details"])
-        writeable = info.get("details", "_write", [])
-        if "accessed" in writeable:
+        can_write_acccess = info.is_writeable("details", "accessed")
+        can_write_modified = info.is_writeable("details", "modified")
+        if can_write_acccess:
             self.assertEqual(info.accessed, datetime(2016, 7, 5, tzinfo=pytz.UTC))
-        if "modified" in writeable:
+        if can_write_modified:
             self.assertEqual(info.modified, datetime(2016, 7, 5, tzinfo=pytz.UTC))
 
     def test_touch(self):