Skip to content

Conversation

@cfriedt
Copy link
Member

@cfriedt cfriedt commented Apr 5, 2025

Previously, these tests were part of tests/posix/common. In an effort to tie feature tests to specific POSIX features (Option Groups), these tests were moved to a separate suite in #81441.

However, the memory requirements for the legacy tests/posix/common testsuite were significantly higher while the memory requirements for tests in tests/posix/c_lib_ext are significantly lower.

Remove the minimum flash and ram requirements to run the test so that it passes tests on the integration platform qemu_cortex_m0.

Fixes #87561

Testing Done:

west twister --integration -T tests/posix/c_lib_ext
...
INFO    - Total complete:    4/   4  100%  built (not run):    0, filtered:    2, failed:    0, error:    0
INFO    - 6 test scenarios (6 configurations) selected, 2 configurations filtered (2 by static filter, 0 at runtime).
INFO    - 4 of 4 executed test configurations passed (100.00%), 0 built (not run), 0 failed, 0 errored, with no warnings in 13.88 seconds.
INFO    - 36 of 36 executed test cases passed (100.00%) on 1 out of total 991 platforms (0.10%).
INFO    - 4 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

Previously, these tests were part of tests/posix/common.
In an effort to tie feature tests to specific POSIX features
(Option Groups), these tests were moved to a separate suite in a
previous commit.

However, the memory requirements for the legacy
tests/posix/common testsuite were significantly higher while the
memory requirements for tests in tests/posix/c_lib_ext are
significantly lower.

Remove the minimum flash and ram requirements to run the test so
that it passes tests on the integration platform qemu_cortex_m0.

Signed-off-by: Chris Friedt <[email protected]>
@cfriedt cfriedt marked this pull request as ready for review April 5, 2025 15:03
@github-actions github-actions bot added the area: POSIX POSIX API Library label Apr 5, 2025
@github-actions github-actions bot requested a review from ycsin April 5, 2025 15:04
@cfriedt cfriedt requested a review from a team April 5, 2025 15:29
@cfriedt cfriedt added the area: Tests Issues related to a particular existing or missing test label Apr 5, 2025
@kartben kartben merged commit 2ec7af3 into zephyrproject-rtos:main Apr 11, 2025
30 checks passed
@cfriedt cfriedt deleted the issues/87561/tests-tests-posix-c-lib-ext-integration-tests-fail branch April 12, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: POSIX POSIX API Library area: Tests Issues related to a particular existing or missing test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests: tests/posix/c_lib_ext: Integration tests fail

3 participants