Skip to content

tests: drivers: spi: spi_loopback: fix skip for unsupported #90322

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

Conversation

nordic-piks
Copy link
Collaborator

Use ztest_test_skip to stop further execution.

Use ztest_test_skip to stop further execution.

Signed-off-by: Piotr Kosycarz <[email protected]>
@nordic-piks nordic-piks marked this pull request as ready for review May 22, 2025 09:38
@github-actions github-actions bot added the area: SPI SPI bus label May 22, 2025
@github-actions github-actions bot requested review from tbursztyka and teburd May 22, 2025 09:39
@nordic-piks
Copy link
Collaborator Author

nordic-piks commented May 22, 2025

After #87838
testing fails like this:

START - test_spi_word_size_16
E: Word sizes other than 8 bits are not supported
Spi config invalid for this controller - skip

    Assertion failed at ../../../../../../../../../zephyr/tests/drivers/spi/spi_loopback/src/spi.c:628: spi_loopback_test_word_size: (memcmp(compare_data, rx_buffer, buffer_size) is true)
16-bit word buffer contents are different
 FAIL - test_spi_word_size_16 in 0.029 seconds

This add proper skip, which ends execution of particular test case.

@decsny FYI

Copy link

@nordic-piks
Copy link
Collaborator Author

Closing in favor of #90321

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: SPI SPI bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants