From: Aarni Koskela <akx@iki.fi>
Date: Tue, 1 Nov 2022 10:16:04 +0000 (+0200)
Subject: Quiesce pytest warnings (#916)
X-Git-Tag: v2.11.0~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bc9dc84876bed120c67c73bb6fcd2d8045198090;p=thirdparty%2Fbabel.git

Quiesce pytest warnings (#916)

* Skip doctest of deprecated format_number

* Don't return from test_compatible_classes_in_global_and_localedata

* Renovate conftest (and require pytest 6+)
---

diff --git a/babel/numbers.py b/babel/numbers.py
index 192e3ed6..373a9bd1 100644
--- a/babel/numbers.py
+++ b/babel/numbers.py
@@ -338,9 +338,9 @@ def get_group_symbol(locale=LC_NUMERIC):
 def format_number(number, locale=LC_NUMERIC):
     u"""Return the given number formatted for a specific locale.
 
-    >>> format_number(1099, locale='en_US')
+    >>> format_number(1099, locale='en_US')  # doctest: +SKIP
     u'1,099'
-    >>> format_number(1099, locale='de_DE')
+    >>> format_number(1099, locale='de_DE')  # doctest: +SKIP
     u'1.099'
 
     .. deprecated:: 2.6.0
diff --git a/conftest.py b/conftest.py
index bd9f2d32..3982cef4 100644
--- a/conftest.py
+++ b/conftest.py
@@ -1,14 +1,20 @@
+from pathlib import Path
+
 from _pytest.doctest import DoctestModule
-from py.path import local
 
 collect_ignore = ['tests/messages/data', 'setup.py']
-babel_path = local(__file__).dirpath().join('babel')
+babel_path = Path(__file__).parent / 'babel'
+
+
+# Via the stdlib implementation of Path.is_relative_to in Python 3.9
+def _is_relative(p1: Path, p2: Path) -> bool:
+    try:
+        p1.relative_to(p2)
+        return True
+    except ValueError:
+        return False
 
 
-def pytest_collect_file(path, parent):
-    if babel_path.common(path) == babel_path:
-        if path.ext == ".py":
-            # TODO: remove check when dropping support for old Pytest
-            if hasattr(DoctestModule, "from_parent"):
-                return DoctestModule.from_parent(parent, fspath=path)
-            return DoctestModule(path, parent)
+def pytest_collect_file(file_path: Path, parent):
+    if _is_relative(file_path, babel_path) and file_path.suffix == '.py':
+        return DoctestModule.from_parent(parent, path=file_path)
diff --git a/tests/test_core.py b/tests/test_core.py
index 529a424a..2de79e2d 100644
--- a/tests/test_core.py
+++ b/tests/test_core.py
@@ -313,7 +313,7 @@ def test_compatible_classes_in_global_and_localedata(filename):
                                          (module, name))
 
     with open(filename, 'rb') as f:
-        return Unpickler(f).load()
+        assert Unpickler(f).load()
 
 
 def test_issue_601_no_language_name_but_has_variant():
diff --git a/tox.ini b/tox.ini
index dd0ee670..97b6bc09 100644
--- a/tox.ini
+++ b/tox.ini
@@ -5,7 +5,7 @@ envlist =
 
 [testenv]
 deps =
-    pytest
+    pytest>=6.0
     pytest-cov
     freezegun==0.3.12
     backports.zoneinfo;python_version<"3.9"