On 10/13/23 10:30 AM, Trevor Gamblin wrote:

On 2023-10-13 13:15, Khem Raj wrote:
On Fri, Oct 13, 2023 at 10:08 AM Trevor Gamblin <[email protected]> wrote:
Two introspection-related ptests are consistently failing. For now,
disable them so that a clean ptest run is available.

For regular runs can we  move it out of PTESTS_PROBLEMS_META_PYTHON into
either PTESTS_FAST_META_PYTHON or PTESTS_SLOW_META_PYTHON
Done, sent a patch to move both recipes.

Thanks, it will start running in my regular CI jobs now.


Signed-off-by: Trevor Gamblin <[email protected]>
---
  ...t_mock-skip-args-introspection-tests.patch | 38 +++++++++++++++++++
  .../python/python3-pytest-mock_3.11.1.bb      |  1 +
  2 files changed, 39 insertions(+)
  create mode 100644 meta-python/recipes-devtools/python/python3-pytest-mock/0001-test_pytest_mock-skip-args-introspection-tests.patch

diff --git a/meta-python/recipes-devtools/python/python3-pytest-mock/0001-test_pytest_mock-skip-args-introspection-tests.patch b/meta-python/recipes-devtools/python/python3-pytest-mock/0001-test_pytest_mock-skip-args-introspection-tests.patch
new file mode 100644
index 0000000000..f024e79d93
--- /dev/null
+++ b/meta-python/recipes-devtools/python/python3-pytest-mock/0001-test_pytest_mock-skip-args-introspection-tests.patch
@@ -0,0 +1,38 @@
+From e184fb950ad3b52c5c16438d39fe4d516c784e30 Mon Sep 17 00:00:00 2001
+From: Trevor Gamblin <[email protected]>
+Date: Thu, 12 Oct 2023 17:32:40 -0400
+Subject: [PATCH] test_pytest_mock: skip args introspection tests
+
+Disable these two tests for now so that python3-pytest-mock can have
+successful ptest runs.
+
+Upstream-Status: Inappropriate [OE-Specific]
+
+Signed-off-by: Trevor Gamblin <[email protected]>
+---
+ tests/test_pytest_mock.py | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/tests/test_pytest_mock.py b/tests/test_pytest_mock.py
+index 3d53241..157b6c3 100644
+--- a/tests/test_pytest_mock.py
++++ b/tests/test_pytest_mock.py
+@@ -581,6 +581,7 @@ def test_assert_called_wrapper(mocker: MockerFixture) -> None:
+     stub.assert_called()
+
+
[email protected]("Async not correctly detected, skip for now")
+ @pytest.mark.usefixtures("needs_assert_rewrite")
+ def test_assert_called_args_with_introspection(mocker: MockerFixture) -> None:
+     stub = mocker.stub()
+@@ -597,6 +598,7 @@ def test_assert_called_args_with_introspection(mocker: MockerFixture) -> None:
+         stub.assert_called_once_with(*wrong_args)
+
+
[email protected]("Async not correctly detected, skip for now")
+ @pytest.mark.usefixtures("needs_assert_rewrite")
+ def test_assert_called_kwargs_with_introspection(mocker: MockerFixture) -> None:
+     stub = mocker.stub()
+--
+2.41.0
+
diff --git a/meta-python/recipes-devtools/python/python3-pytest-mock_3.11.1.bb b/meta-python/recipes-devtools/python/python3-pytest-mock_3.11.1.bb
index 69d74567c3..7fd998edc6 100644
--- a/meta-python/recipes-devtools/python/python3-pytest-mock_3.11.1.bb
+++ b/meta-python/recipes-devtools/python/python3-pytest-mock_3.11.1.bb
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM = " \
  SRC_URI = " \
git://github.com/pytest-dev/pytest-mock;branch=main;protocol=https \
      file://run-ptest \
+    file://0001-test_pytest_mock-skip-args-introspection-tests.patch \
  "

  SRCREV ?= "d3e73f2e93f7b93eba0a36e17e43bafd969da4fe"
--
2.41.0




Attachment: OpenPGP_0xBB053355919D3314.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#105489): 
https://lists.openembedded.org/g/openembedded-devel/message/105489
Mute This Topic: https://lists.openembedded.org/mt/101944758/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to