Skip to content

twister: Convert platform names before printing #90404

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 27, 2025

Conversation

gchwier
Copy link
Collaborator

@gchwier gchwier commented May 23, 2025

Just move a code with workaround to convert platform names to the full name with variants.
It fixes an issue, that not every platforms are printed before running tests.

Before fix:
(when using a platform that has a variant, but not given it with --platform or in hardware map):
$ZEPHYR_BASE/scripts/twister -v -T samples/hello_world -p nrf9160dk/nrf9160 --device-testing --device-serial /dev/ttyACM0

INFO    - Building initial testsuite list...
Device testing on:
| Platform   | ID   | Serial device   |
|------------|------|-----------------|
...
INFO    - 1/1 [email protected]/nrf9160  samples/hello_world/sample.basic.helloworld        PASSED (device 1.609s <zephyr>)
...
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
Hardware distribution summary:
| Board                    | ID   |   Counter |   Failures |
|--------------------------|------|-----------|------------|
| [email protected]/nrf9160 |      |         1 |          0 |

after fix, you can find platform also in initial list:

INFO    - Building initial testsuite list...
Device testing on:
| Platform                 | ID   | Serial device   |
|--------------------------|------|-----------------|
| [email protected]/nrf9160 |      | /dev/ttyACM0    |

Just move a code with workaroud to convert platform names
to the full name with variants.
It fixes an issue, that not every platforms are printed
before running tests.

Signed-off-by: Grzegorz Chwierut <[email protected]>
Copy link

@kartben kartben merged commit 632d006 into zephyrproject-rtos:main May 27, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants