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

[Framework] Move test_psa_*.py scripts to the framework #94

Closed
mpg opened this issue Dec 5, 2024 · 0 comments · Fixed by Mbed-TLS/mbedtls#9940
Closed

[Framework] Move test_psa_*.py scripts to the framework #94

mpg opened this issue Dec 5, 2024 · 0 comments · Fixed by Mbed-TLS/mbedtls#9940
Assignees
Labels
size-s Estimated task size: small (~2d)

Comments

@mpg
Copy link
Contributor

mpg commented Dec 5, 2024

Move tests/scripts/test_psa_compliance.py and tests/scripts/test_psa_constant_names.py to the framework - they will be used by Mbed TLS 3.6 and tf-psa-crypto.

The versions of test_psa_compliance.py in 3.6 and development are slightly different in that the version in development has been adapted to tf-psa-crypto. The version from development should work for 3.6 too as it includes checks for which repo we're in.

The versions of test_psa_constant_names.py in 3.6 and development differ in some paths. These should be reconciled, probably using is_mbedtls_3_6() from the build_tree module. (Once the script is in the framework repo, from . import build_tree should work. From the mbedtls repo, I think we want import framework_scripts_path # pylint: disable=unused-import then from mbedtls_framework import build_tree.)

@mpg mpg added the size-s Estimated task size: small (~2d) label Dec 5, 2024
@mpg mpg moved this to Framework 2/3 (script for the CI) in Mbed TLS Epics Dec 5, 2024
@ronald-cron-arm ronald-cron-arm moved this from Framework 2/3 (for the CI) to TF-PSA-Crypto all.sh 2 - misc in Mbed TLS Epics Jan 7, 2025
@valeriosetti valeriosetti self-assigned this Jan 29, 2025
@valeriosetti valeriosetti added needs-preceding-pr Requires another PR to be merged first and removed needs-preceding-pr Requires another PR to be merged first labels Jan 29, 2025
@valeriosetti valeriosetti changed the title Move test_psa_*.py scripts to the framework [Framework] Move test_psa_*.py scripts to the framework Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-s Estimated task size: small (~2d)
Projects
Status: TF-PSA-Crypto all.sh 2 - misc
Development

Successfully merging a pull request may close this issue.

2 participants