diff mbox series

[kirkstone,1/1] python3-jinja2: fix for the ptest result format

Message ID 20230929082419.3767306-1-narpat.mali@windriver.com
State Accepted, archived
Commit edfed23716e6240a65f53630bbaf8b7319f0d1ce
Headers show
Series [kirkstone,1/1] python3-jinja2: fix for the ptest result format | expand

Commit Message

nmali Sept. 29, 2023, 8:24 a.m. UTC
From: Narpat Mali <narpat.mali@windriver.com>

The output of python3-jinja2 ptest should follow a unified format as below
result: testname

Reference:
https://wiki.yoctoproject.org/wiki/Ptest

Signed-off-by: Narpat Mali <narpat.mali@windriver.com>
---
 meta/recipes-devtools/python/python3-jinja2/run-ptest | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Steve Sakoman Oct. 2, 2023, 2:34 p.m. UTC | #1
On Thu, Sep 28, 2023 at 10:24 PM Narpat Mali via
lists.openembedded.org
<narpat.mali=windriver.com@lists.openembedded.org> wrote:
>
> From: Narpat Mali <narpat.mali@windriver.com>
>
> The output of python3-jinja2 ptest should follow a unified format as below
> result: testname
>
> Reference:
> https://wiki.yoctoproject.org/wiki/Ptest

Is this also an issue with the version in the master branch?

If so, please submit a patch for master branch too.  We don't want to
fix this in a stable branch without also fixing it in master.

Thanks!

Steve

> Signed-off-by: Narpat Mali <narpat.mali@windriver.com>
> ---
>  meta/recipes-devtools/python/python3-jinja2/run-ptest | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/python/python3-jinja2/run-ptest b/meta/recipes-devtools/python/python3-jinja2/run-ptest
> index 5cec711696..3004024619 100644
> --- a/meta/recipes-devtools/python/python3-jinja2/run-ptest
> +++ b/meta/recipes-devtools/python/python3-jinja2/run-ptest
> @@ -1,3 +1,3 @@
>  #!/bin/sh
>
> -pytest
> +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}}'
> --
> 2.40.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#188394): https://lists.openembedded.org/g/openembedded-core/message/188394
> Mute This Topic: https://lists.openembedded.org/mt/101653794/3620601
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [steve@sakoman.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
nmali Oct. 3, 2023, 4:50 a.m. UTC | #2
On 02-10-2023 20:04, Steve Sakoman wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> On Thu, Sep 28, 2023 at 10:24 PM Narpat Mali via
> lists.openembedded.org
> <narpat.mali=windriver.com@lists.openembedded.org> wrote:
>> From: Narpat Mali <narpat.mali@windriver.com>
>>
>> The output of python3-jinja2 ptest should follow a unified format as below
>> result: testname
>>
>> Reference:
>> https://wiki.yoctoproject.org/wiki/Ptest
> Is this also an issue with the version in the master branch?
>
> If so, please submit a patch for master branch too.  We don't want to
> fix this in a stable branch without also fixing it in master.

Hi Steve,

In master branch, it has been already fixed using 
"python3-unittest-automake-output" instead of sed to reformat the output.

https://git.openembedded.org/openembedded-core/commit/?id=def25193748fac40c4120e9a6e8861ae4c7ae760

../Narpat

>
> Thanks!
>
> Steve
>
>> Signed-off-by: Narpat Mali <narpat.mali@windriver.com>
>> ---
>>   meta/recipes-devtools/python/python3-jinja2/run-ptest | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-devtools/python/python3-jinja2/run-ptest b/meta/recipes-devtools/python/python3-jinja2/run-ptest
>> index 5cec711696..3004024619 100644
>> --- a/meta/recipes-devtools/python/python3-jinja2/run-ptest
>> +++ b/meta/recipes-devtools/python/python3-jinja2/run-ptest
>> @@ -1,3 +1,3 @@
>>   #!/bin/sh
>>
>> -pytest
>> +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}}'
>> --
>> 2.40.0
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#188394): https://lists.openembedded.org/g/openembedded-core/message/188394
>> Mute This Topic: https://lists.openembedded.org/mt/101653794/3620601
>> Group Owner: openembedded-core+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [steve@sakoman.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
diff mbox series

Patch

diff --git a/meta/recipes-devtools/python/python3-jinja2/run-ptest b/meta/recipes-devtools/python/python3-jinja2/run-ptest
index 5cec711696..3004024619 100644
--- a/meta/recipes-devtools/python/python3-jinja2/run-ptest
+++ b/meta/recipes-devtools/python/python3-jinja2/run-ptest
@@ -1,3 +1,3 @@ 
 #!/bin/sh
 
-pytest
+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}}'