Skip to content
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

twister: always add integration platforms to platform scope #68698

Conversation

nashif
Copy link
Member

@nashif nashif commented Feb 7, 2024

Add integration_platforms to the list of the default platforms per test
for improved coverage when not running in integration mode.

Some tests when run without any option do not exercise any tests and
skip the integration platforms completely.

Signed-off-by: Anas Nashif [email protected]

@nashif
Copy link
Member Author

nashif commented Feb 7, 2024

This was noticed for example with:

west twister -s tests/drivers/flash/common/drivers.flash.common.tfm_ns -v

where everything is skipped (filtered) and the designated integration platform does not get built/run at all.

gchwier
gchwier previously approved these changes Feb 7, 2024
Add integration_platforms to the list of the default platforms per test
for improved coverage when not running in integration mode.

Some tests when run without any option do not exercise any tests and
skip the integration platforms completely.

Signed-off-by: Anas Nashif <[email protected]>
@nashif nashif force-pushed the topic/twister/integration_platforms_in_normal_mode branch from 1722a8d to 3d27258 Compare February 8, 2024 03:23
@fabiobaltieri fabiobaltieri added this to the v3.6.0 milestone Feb 16, 2024
@fabiobaltieri fabiobaltieri added the bug The issue is a bug, or the PR is fixing a bug label Feb 16, 2024
@henrikbrixandersen henrikbrixandersen merged commit fc5211f into zephyrproject-rtos:main Feb 16, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Twister Twister bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants