Avoid runtime checks for missing "os.getxattr" / "os.setxattr"

loguru/_ctime_functions.py

@@ -20,16 +20,25 @@ def set_ctime(filepath, timestamp):
         pass
 
 
-else:
+elif hasattr(os, "getxattr") and hasattr(os, "setxattr"):
 
     def get_ctime(filepath):
         try:
             return float(os.getxattr(filepath, b"user.loguru_crtime"))
-        except (OSError, AttributeError):
+        except OSError:
             return os.stat(filepath).st_mtime
 
     def set_ctime(filepath, timestamp):
         try:
             os.setxattr(filepath, b"user.loguru_crtime", str(timestamp).encode("ascii"))
-        except (OSError, AttributeError):
+        except OSError:
             pass
+
+
+else:
+
+    def get_ctime(filepath):
+        return os.stat(filepath).st_mtime
+
+    def set_ctime(filepath, timestamp):
+        pass

tests/test_filesink_rotation.py

@@ -110,20 +110,17 @@ def raising(*args, **kwargs):
         raise OSError
 
     monkeypatch.setattr(os, "name", "posix")
-    monkeypatch_filesystem(raising="st_birthtime", crtime="st_mtime")
     monkeypatch.setattr(os, "setxattr", raising, raising=False)
     monkeypatch.setattr(os, "getxattr", raising, raising=False)
+    monkeypatch_filesystem(raising="st_birthtime", crtime="st_mtime")
 
 
 @pytest.fixture
 def linux_xattr_attributeerror_filesystem(monkeypatch, monkeypatch_filesystem):
-    def raising(*args, **kwargs):
-        raise OSError
-
     monkeypatch.setattr(os, "name", "posix")
-    monkeypatch_filesystem(raising="st_birthtime", crtime="st_mtime")
     monkeypatch.delattr(os, "setxattr", raising=False)
     monkeypatch.delattr(os, "getxattr", raising=False)
+    monkeypatch_filesystem(raising="st_birthtime", crtime="st_mtime")
 
 
 def test_renaming(tmpdir):