Skip to content

Add comprehensive C library testsuite #51982

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

Open
27 tasks
Tracked by #45776
stephanosio opened this issue Nov 5, 2022 · 1 comment
Open
27 tasks
Tracked by #45776

Add comprehensive C library testsuite #51982

stephanosio opened this issue Nov 5, 2022 · 1 comment
Assignees
Labels
area: C Library C Standard Library area: Tests Issues related to a particular existing or missing test Enhancement Changes/Updates/Additions to existing features

Comments

@stephanosio
Copy link
Member

stephanosio commented Nov 5, 2022

Summary

Add a comprehensive C library test suite that:

  • tests the functional requirements of all supported C libraries to ensure that the C library meets the Zephyr expectations.
  • has common codebase for testing different C libraries (focus on testing standard C features).

Background

We currently have tests/lib/c_lib; but, this test suite is by no means comprehensive and is missing many tests leading to the issues like #51978.

Resources

Tasks

Test Clean-up

  • Remove tests/lib/c_lib
    • This is a half-baked test for a very limited subset of the libc.
    • tests/lib/libc, aka. "comprehensive C library test," will replace it in its entirety.
  • Remove tests/lib/sprintf
    • There is no reason for a separate sprintf test to exist.
    • This test will be part of the new tests/lib/libc test.
    • Any cbprintf-specific bits, outside the scope of the ISO C standard, ought to be tested by the cbprintf tests.
  • Remove tests/lib/time
    • There is no reason for a separate time test to exist.
    • This test will be part of the new tests/lib/libc test.

Test Implementation

  • assert.h
  • complex.h
  • ctype.h
  • errno.h
  • fenv.h
  • float.h
  • inttypes.h
  • iso646.h
  • limits.h
  • locale.h
  • math.h
  • setjmp.h
  • signal.h
  • stdarg.h
  • stdbool.h
  • stddef.h
  • stdint.h
  • stdio.h
  • stdlib.h
  • string.h
  • tgmath.h
  • time.h
  • wchar.h
  • wctype.h
@stephanosio stephanosio mentioned this issue Nov 5, 2022
6 tasks
@stephanosio stephanosio added area: C Library C Standard Library area: Tests Issues related to a particular existing or missing test labels Nov 5, 2022
@laurenmurphyx64 laurenmurphyx64 added the Enhancement Changes/Updates/Additions to existing features label Nov 8, 2022
@zephyrbot
Copy link
Collaborator

Hi @stephanosio,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. Please confirm the issue is correctly assigned and re-assign it otherwise.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: C Library C Standard Library area: Tests Issues related to a particular existing or missing test Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

3 participants