Catch and assert warnings in tests (#487)

* tests: add checks for warnings

* tests: add checks for warnings read_text write_text

* tests: ignore keyword warnings in NotImplemented tests

* tests: check warnings in extensions tests

* tests: fix warnings in pathlib312 tests

Author: ap--

upath/implementations/local.py

@@ -449,7 +449,7 @@ def from_uri(cls, uri: str, **storage_options: Any) -> Self:
         def is_dir(self, *, follow_symlinks: bool = True) -> bool:
             if not follow_symlinks:
                 warnings.warn(
-                    f"{type(self).__name__}.is_dir() follow_symlinks=False"
+                    f"{type(self).__name__}.is_dir(): follow_symlinks=False"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -459,7 +459,7 @@ def is_dir(self, *, follow_symlinks: bool = True) -> bool:
         def is_file(self, *, follow_symlinks: bool = True) -> bool:
             if not follow_symlinks:
                 warnings.warn(
-                    f"{type(self).__name__}.is_file() follow_symlinks=False"
+                    f"{type(self).__name__}.is_file(): follow_symlinks=False"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -474,7 +474,7 @@ def read_text(
         ) -> str:
             if newline is not None:
                 warnings.warn(
-                    f"{type(self).__name__}.read_text() newline"
+                    f"{type(self).__name__}.read_text(): newline"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -499,7 +499,7 @@ def glob(  # type: ignore[override]
             if case_sensitive is not None:
                 if sys.version_info < (3, 12):
                     warnings.warn(
-                        f"{type(self).__name__}.glob() case_sensitive"
+                        f"{type(self).__name__}.glob(): case_sensitive"
                         " is currently ignored.",
                         UserWarning,
                         stacklevel=2,
@@ -508,7 +508,7 @@ def glob(  # type: ignore[override]
                     kw["case_sensitive"] = case_sensitive
             if recurse_symlinks:
                 warnings.warn(
-                    f"{type(self).__name__}.glob() recurse_symlinks=True"
+                    f"{type(self).__name__}.glob(): recurse_symlinks=True"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -530,7 +530,7 @@ def rglob(  # type: ignore[override]
             if case_sensitive is not None:
                 if sys.version_info < (3, 12):
                     warnings.warn(
-                        f"{type(self).__name__}.rglob() case_sensitive"
+                        f"{type(self).__name__}.rglob(): case_sensitive"
                         " is currently ignored.",
                         UserWarning,
                         stacklevel=2,
@@ -539,7 +539,7 @@ def rglob(  # type: ignore[override]
                     kw["case_sensitive"] = case_sensitive
             if recurse_symlinks:
                 warnings.warn(
-                    f"{type(self).__name__}.rglob() recurse_symlinks=True"
+                    f"{type(self).__name__}.rglob(): recurse_symlinks=True"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -592,7 +592,7 @@ def match(
                 pattern_str = os.fspath(path_pattern)
             if case_sensitive is not None:
                 warnings.warn(
-                    f"{type(self).__name__}.match() case_sensitive"
+                    f"{type(self).__name__}.match(): case_sensitive"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,
@@ -602,7 +602,7 @@ def match(
         def exists(self, *, follow_symlinks: bool = True) -> bool:
             if not follow_symlinks:
                 warnings.warn(
-                    f"{type(self).__name__}.exists() follow_symlinks=False"
+                    f"{type(self).__name__}.exists(): follow_symlinks=False"
                     " is currently ignored.",
                     UserWarning,
                     stacklevel=2,

upath/tests/pathlib/test_pathlib_312.py

@@ -129,12 +129,7 @@ def test_with_segments_common(self):
         class P(_BasePurePathSubclass, self.cls):
             pass
 
-        if self.cls is UPath:
-            cm = temporary_register("", P)
-        else:
-            cm = nullcontext()
-
-        with cm:
+        with temporary_register("", P):
             p = P('foo', 'bar', session_id=42)
             self.assertEqual(42, (p / 'foo').session_id)
             self.assertEqual(42, ('foo' / p).session_id)
@@ -3281,6 +3276,15 @@ class PathSubclassTest(_BasePathTest, unittest.TestCase):
     class cls(WindowsUPath if os.name == 'nt' else PosixUPath):
         pass
 
+    def setUp(self):
+        self._registry_cm = temporary_register("", self.cls)
+        self._registry_cm.__enter__()
+        return super().setUp()
+
+    def tearDown(self):
+        self._registry_cm.__exit__(None, None, None)
+        return super().tearDown()
+
     # repr() roundtripping is not supported in custom subclass.
     test_repr_roundtrips = None
 

upath/tests/test_extensions.py

@@ -65,6 +65,16 @@ def test_chmod(self):
     def test_eq(self):
         super().test_eq()
 
+    if sys.version_info < (3, 12):
+
+        def test_storage_options_dont_affect_hash(self):
+            # On Python < 3.12, storage_options trigger warnings for LocalPath
+            with pytest.warns(
+                UserWarning,
+                match=r".*on python <= \(3, 11\) ignores protocol and storage_options",
+            ):
+                super().test_storage_options_dont_affect_hash()
+
     def test_group(self):
         pytest.importorskip("grp")
         self.path.group()
@@ -91,9 +101,22 @@ def test_as_uri(self):
     def test_rename2(self):
         super().test_rename2()
 
-    def test_lstat(self):
-        st = self.path.lstat()
-        assert st is not None
+    if sys.version_info < (3, 10):
+
+        def test_lstat(self):
+            # On Python < 3.10, stat(follow_symlinks=False) triggers warnings
+            with pytest.warns(
+                UserWarning,
+                match=r".*stat\(\) follow_symlinks=False is currently ignored",
+            ):
+                st = self.path.lstat()
+            assert st is not None
+
+    else:
+
+        def test_lstat(self):
+            st = self.path.lstat()
+            assert st is not None
 
     def test_relative_to(self):
         base = self.path