diff mbox series

[meta-python,kirkstone] python3-pint: fix ptests

Message ID 20251023212901.841481-1-skandigraun@gmail.com
State New
Headers show
Series [meta-python,kirkstone] python3-pint: fix ptests | expand

Commit Message

Gyorgy Sarvari Oct. 23, 2025, 9:29 p.m. UTC
1. Add missing ptest dependency (pytest-subtest)

2. The testsuite is installed in both the site-packages and ${PTEST_PATH}
folders, however some dependencies are only available in the site-packages
folder, so many test cases fail.
At this point of the branch lifecycle I decided not to refactor the recipe, but
rather to just use the installation in the site-packages dir to run the
tests (switch to that folder in the run-ptest script)

3. Fix the run-ptest script to output PASS/FAIL status.

Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
---
 meta-python/recipes-devtools/python/python3-pint/run-ptest | 3 ++-
 meta-python/recipes-devtools/python/python3-pint_0.19.1.bb | 6 +-----
 2 files changed, 3 insertions(+), 6 deletions(-)

Comments

Khem Raj Oct. 24, 2025, 12:12 a.m. UTC | #1
Do we need this on master

On Thu, Oct 23, 2025 at 2:29 PM Gyorgy Sarvari via lists.openembedded.org
<skandigraun=gmail.com@lists.openembedded.org> wrote:

> 1. Add missing ptest dependency (pytest-subtest)
>
> 2. The testsuite is installed in both the site-packages and ${PTEST_PATH}
> folders, however some dependencies are only available in the site-packages
> folder, so many test cases fail.
> At this point of the branch lifecycle I decided not to refactor the
> recipe, but
> rather to just use the installation in the site-packages dir to run the
> tests (switch to that folder in the run-ptest script)
>
> 3. Fix the run-ptest script to output PASS/FAIL status.
>
> Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
> ---
>  meta-python/recipes-devtools/python/python3-pint/run-ptest | 3 ++-
>  meta-python/recipes-devtools/python/python3-pint_0.19.1.bb | 6 +-----
>  2 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/meta-python/recipes-devtools/python/python3-pint/run-ptest
> b/meta-python/recipes-devtools/python/python3-pint/run-ptest
> index 5cec711696..6abbb5de73 100644
> --- a/meta-python/recipes-devtools/python/python3-pint/run-ptest
> +++ b/meta-python/recipes-devtools/python/python3-pint/run-ptest
> @@ -1,3 +1,4 @@
>  #!/bin/sh
>
> -pytest
> +cd /usr/lib/python3*/site-packages/pint
> +pytest -o log_cli=true -o log_cli_level=INFO | sed -e 's/\[...%\]//g'|
> sed -e 's/PASSED/PASS/g'| sed -e 's/FAILED/FAIL/g'|sed -e
> 's/SKIPPED/SKIP/g'| awk '{if ($NF=="PASS" || $NF=="FAIL" || $NF=="SKIP" ||
> $NF=="XFAIL" || $NF=="XPASS"){printf "%s: %s\n", $NF, $0}else{print}}'| awk
> '{if ($NF=="PASS" || $NF=="FAIL" || $NF=="SKIP" || $NF=="XFAIL" ||
> $NF=="XPASS") {$NF="";print $0}else{print}}'
> diff --git a/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
> b/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
> index e1c471e064..4402908a35 100644
> --- a/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
> +++ b/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
> @@ -27,9 +27,5 @@ RDEPENDS:${PN} += " \
>
>  RDEPENDS:${PN}-ptest += " \
>         ${PYTHON_PN}-pytest \
> +        ${PYTHON_PN}-pytest-subtests \
>  "
> -
> -do_install_ptest() {
> -       install -d ${D}${PTEST_PATH}/testsuite
> -       cp -rf ${S}/pint/testsuite/* ${D}${PTEST_PATH}/testsuite/
> -}
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#120940):
> https://lists.openembedded.org/g/openembedded-devel/message/120940
> Mute This Topic: https://lists.openembedded.org/mt/115918828/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
diff mbox series

Patch

diff --git a/meta-python/recipes-devtools/python/python3-pint/run-ptest b/meta-python/recipes-devtools/python/python3-pint/run-ptest
index 5cec711696..6abbb5de73 100644
--- a/meta-python/recipes-devtools/python/python3-pint/run-ptest
+++ b/meta-python/recipes-devtools/python/python3-pint/run-ptest
@@ -1,3 +1,4 @@ 
 #!/bin/sh
 
-pytest
+cd /usr/lib/python3*/site-packages/pint
+pytest -o log_cli=true -o log_cli_level=INFO | sed -e 's/\[...%\]//g'| sed -e 's/PASSED/PASS/g'| sed -e 's/FAILED/FAIL/g'|sed -e 's/SKIPPED/SKIP/g'| awk '{if ($NF=="PASS" || $NF=="FAIL" || $NF=="SKIP" || $NF=="XFAIL" || $NF=="XPASS"){printf "%s: %s\n", $NF, $0}else{print}}'| awk '{if ($NF=="PASS" || $NF=="FAIL" || $NF=="SKIP" || $NF=="XFAIL" || $NF=="XPASS") {$NF="";print $0}else{print}}'
diff --git a/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb b/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
index e1c471e064..4402908a35 100644
--- a/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
+++ b/meta-python/recipes-devtools/python/python3-pint_0.19.1.bb
@@ -27,9 +27,5 @@  RDEPENDS:${PN} += " \
 
 RDEPENDS:${PN}-ptest += " \
 	${PYTHON_PN}-pytest \
+        ${PYTHON_PN}-pytest-subtests \
 "
-
-do_install_ptest() {
-	install -d ${D}${PTEST_PATH}/testsuite
-	cp -rf ${S}/pint/testsuite/* ${D}${PTEST_PATH}/testsuite/
-}