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

PSA client-server: run unit tests #8966

Open
gilles-peskine-arm opened this issue Mar 20, 2024 · 2 comments · May be fixed by #9237
Open

PSA client-server: run unit tests #8966

gilles-peskine-arm opened this issue Mar 20, 2024 · 2 comments · May be fixed by #9237
Assignees
Labels
component-test Test framework and CI scripts enhancement size-m Estimated task size: medium (~1w)

Comments

@gilles-peskine-arm
Copy link
Contributor

gilles-peskine-arm commented Mar 20, 2024

Definition of done:

  • We can run make test on a PSA client-only build in the full configuration minus threads, with a PSA crypto server connected through psasim, and everything passes.
  • The test suite has no dependencies on MBEDTLS_PSA_CRYPTO_C, only on MBEDTLS_PSA_CRYPTO_CLIENT, except in places where we have explicitly concluded that the code doesn't make sense on a client-server build.

This issue is very large and should be broken down. Further study is required to break it down: we pretty much need to have the client-server mechanism and see how it goes when running the tests. There will likely be an intermediate stage where we only run certain test suites or even only certain test functions.

Prerequisites: #8965, #8968

@gilles-peskine-arm gilles-peskine-arm added enhancement size-l Estimated task size: large (2w+) component-test Test framework and CI scripts labels Mar 20, 2024
@gilles-peskine-arm gilles-peskine-arm added size-m Estimated task size: medium (~1w) and removed size-l Estimated task size: large (2w+) labels Mar 21, 2024
@minosgalanakis minosgalanakis added this to PSA client-server testing in EPICs for Mbed TLS Apr 17, 2024
@tom-cosgrove-arm
Copy link
Contributor

How does "This issue is very large and should be broken down. Further study is required to break it down" track with size-m?

@gilles-peskine-arm
Copy link
Contributor Author

size-m is my gut feeling, but we need to see test results to have a better idea. It might end up being smaller if driver-only has already solved all the problems except RSA. Or it might end up being larger if there are lots of different problems.

@valeriosetti valeriosetti self-assigned this Jun 28, 2024
@valeriosetti valeriosetti linked a pull request Jun 28, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component-test Test framework and CI scripts enhancement size-m Estimated task size: medium (~1w)
Projects
EPICs for Mbed TLS
PSA client-server testing
Development

Successfully merging a pull request may close this issue.

3 participants