forked from pool/python-SQLAlchemy
- Replace pytest_depr_from_parent.patch with the upstream commit resolving
same issue (gh#sqlalchemy/sqlalchemy#commit40cdb9c0bf4d). OBS-URL: https://build.opensuse.org/package/show/devel:languages:python/python-SQLAlchemy?expand=0&rev=175
This commit is contained in:
@@ -1,6 +1,22 @@
|
|||||||
|
From 40cdb9c0bf4db2af293f9d8669257294d8fc69f9 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Mike Bayer <mike_mp@zzzcomputing.com>
|
||||||
|
Date: Sat, 26 Sep 2020 21:01:01 -0400
|
||||||
|
Subject: [PATCH] Support pytest 6.x
|
||||||
|
|
||||||
|
pytest has removed support for pytest.Class().collect()
|
||||||
|
and we need to use from_parent.
|
||||||
|
|
||||||
|
Change-Id: Ia5fed9b22e76c99f71489283acee207f996f52a4
|
||||||
|
(cherry picked from commit ffafbd4b9657a4ee8bec57c0861414144f37bdc5)
|
||||||
|
---
|
||||||
|
lib/sqlalchemy/testing/plugin/pytestplugin.py | 21 +++++++------------
|
||||||
|
1 file changed, 7 insertions(+), 14 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/lib/sqlalchemy/testing/plugin/pytestplugin.py b/lib/sqlalchemy/testing/plugin/pytestplugin.py
|
||||||
|
index 079066d829..ad4ebb6565 100644
|
||||||
--- a/lib/sqlalchemy/testing/plugin/pytestplugin.py
|
--- a/lib/sqlalchemy/testing/plugin/pytestplugin.py
|
||||||
+++ b/lib/sqlalchemy/testing/plugin/pytestplugin.py
|
+++ b/lib/sqlalchemy/testing/plugin/pytestplugin.py
|
||||||
@@ -162,16 +162,19 @@ def pytest_collection_modifyitems(sessio
|
@@ -162,13 +162,10 @@ def pytest_collection_modifyitems(session, config, items):
|
||||||
if sub_cls is not test_class.cls:
|
if sub_cls is not test_class.cls:
|
||||||
per_cls_dict = rebuilt_items[test_class.cls]
|
per_cls_dict = rebuilt_items[test_class.cls]
|
||||||
|
|
||||||
@@ -11,50 +27,29 @@
|
|||||||
-
|
-
|
||||||
- for inst in pytest.Class(
|
- for inst in pytest.Class(
|
||||||
- sub_cls.__name__, parent=test_class.parent.parent
|
- sub_cls.__name__, parent=test_class.parent.parent
|
||||||
- ).collect():
|
+ # support pytest 5.4.0 and above pytest.Class.from_parent
|
||||||
- for t in inst.collect():
|
+ ctor = getattr(pytest.Class, "from_parent", pytest.Class)
|
||||||
- per_cls_dict[t.name].append(t)
|
+ for inst in ctor(
|
||||||
+ if hasattr(pytest.Class, "from_parent"):
|
+ name=sub_cls.__name__, parent=test_class.parent.parent
|
||||||
+ # in pytest 5.4.0
|
).collect():
|
||||||
+ for inst in pytest.Class.from_parent(
|
for t in inst.collect():
|
||||||
+ test_class.parent.parent, name=sub_cls.__name__
|
per_cls_dict[t.name].append(t)
|
||||||
+ ).collect():
|
@@ -196,15 +193,11 @@ def pytest_pycollect_makeitem(collector, name, obj):
|
||||||
+ for t in inst.collect():
|
|
||||||
+ per_cls_dict[t.name].append(t)
|
|
||||||
+ else:
|
|
||||||
+ for inst in pytest.Class(
|
|
||||||
+ sub_cls.__name__, parent=test_class.parent.parent
|
|
||||||
+ ).collect():
|
|
||||||
+ for t in inst.collect():
|
|
||||||
+ per_cls_dict[t.name].append(t)
|
|
||||||
|
|
||||||
newitems = []
|
|
||||||
for item in items:
|
|
||||||
@@ -197,16 +200,17 @@ def pytest_pycollect_makeitem(collector,
|
|
||||||
if inspect.isclass(obj) and plugin_base.want_class(name, obj):
|
if inspect.isclass(obj) and plugin_base.want_class(name, obj):
|
||||||
|
|
||||||
# in pytest 5.4.0
|
- # in pytest 5.4.0
|
||||||
- # return [
|
- # return [
|
||||||
- # pytest.Class.from_parent(collector,
|
- # pytest.Class.from_parent(collector,
|
||||||
- # name=parametrize_cls.__name__)
|
- # name=parametrize_cls.__name__)
|
||||||
- # for parametrize_cls in _parametrize_cls(collector.module, obj)
|
- # for parametrize_cls in _parametrize_cls(collector.module, obj)
|
||||||
- # ]
|
- # ]
|
||||||
-
|
+ # support pytest 5.4.0 and above pytest.Class.from_parent
|
||||||
- return [
|
+ ctor = getattr(pytest.Class, "from_parent", pytest.Class)
|
||||||
|
|
||||||
|
return [
|
||||||
- pytest.Class(parametrize_cls.__name__, parent=collector)
|
- pytest.Class(parametrize_cls.__name__, parent=collector)
|
||||||
- for parametrize_cls in _parametrize_cls(collector.module, obj)
|
+ ctor(name=parametrize_cls.__name__, parent=collector)
|
||||||
- ]
|
for parametrize_cls in _parametrize_cls(collector.module, obj)
|
||||||
+ if hasattr(pytest.Class, "from_parent"):
|
]
|
||||||
+ return [
|
|
||||||
+ pytest.Class.from_parent(collector,
|
|
||||||
+ name=parametrize_cls.__name__)
|
|
||||||
+ for parametrize_cls in _parametrize_cls(collector.module, obj)
|
|
||||||
+ ]
|
|
||||||
+ else:
|
|
||||||
+ return [
|
|
||||||
+ pytest.Class(parametrize_cls.__name__, parent=collector)
|
|
||||||
+ for parametrize_cls in _parametrize_cls(collector.module, obj)
|
|
||||||
+ ]
|
|
||||||
elif (
|
elif (
|
||||||
inspect.isfunction(obj)
|
|
||||||
and isinstance(collector, pytest.Instance)
|
|
||||||
|
|||||||
@@ -1,3 +1,9 @@
|
|||||||
|
-------------------------------------------------------------------
|
||||||
|
Fri Oct 9 13:02:39 UTC 2020 - Matej Cepl <mcepl@suse.com>
|
||||||
|
|
||||||
|
- Replace pytest_depr_from_parent.patch with the upstream commit resolving
|
||||||
|
same issue (gh#sqlalchemy/sqlalchemy#commit40cdb9c0bf4d).
|
||||||
|
|
||||||
-------------------------------------------------------------------
|
-------------------------------------------------------------------
|
||||||
Thu Oct 8 15:45:04 UTC 2020 - Matej Cepl <mcepl@suse.com>
|
Thu Oct 8 15:45:04 UTC 2020 - Matej Cepl <mcepl@suse.com>
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user