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

[Backport 3.6] Move test_psa_*.py scripts to the framework #9941

Merged
merged 4 commits into from
Feb 5, 2025

Conversation

valeriosetti
Copy link
Contributor

@valeriosetti valeriosetti commented Jan 29, 2025

Description

This is the backport of #9940

PR checklist

@valeriosetti valeriosetti changed the base branch from development to mbedtls-3.6 January 29, 2025 16:33
@valeriosetti valeriosetti self-assigned this Jan 29, 2025
@valeriosetti valeriosetti added needs-review Every commit must be reviewed by at least two team members, needs-ci Needs to pass CI tests needs-reviewer This PR needs someone to pick it up for review size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Jan 29, 2025
@valeriosetti valeriosetti force-pushed the issue94-3.6 branch 2 times, most recently from adc6d7c to e11fcb5 Compare January 30, 2025 09:25
@valeriosetti valeriosetti removed the needs-ci Needs to pass CI tests label Jan 30, 2025
Harry-Ramsey
Harry-Ramsey previously approved these changes Jan 31, 2025
Copy link
Contributor

@Harry-Ramsey Harry-Ramsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ronald-cron-arm ronald-cron-arm removed the needs-reviewer This PR needs someone to pick it up for review label Jan 31, 2025
@ronald-cron-arm ronald-cron-arm self-requested a review January 31, 2025 13:43
Harry-Ramsey
Harry-Ramsey previously approved these changes Jan 31, 2025
@valeriosetti
Copy link
Contributor Author

OpenCI is failing in a couple of components, but the failure is related to the fact that CI had issues in setting up the environment, not to the code. I'm going to restart it to ensure it's fine. Meanwhile the PR is ready for reviews

Harry-Ramsey
Harry-Ramsey previously approved these changes Feb 4, 2025
ronald-cron-arm
ronald-cron-arm previously approved these changes Feb 4, 2025
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Harry-Ramsey
Harry-Ramsey previously approved these changes Feb 5, 2025
The following files are moved to the framework repo (deleted here):

tests/scripts/test_psa_compliance.py
tests/scripts/test_psa_constant_names.py

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
…ames.py

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ronald-cron-arm ronald-cron-arm added this pull request to the merge queue Feb 5, 2025
Merged via the queue into Mbed-TLS:mbedtls-3.6 with commit c811fb7 Feb 5, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review Every commit must be reviewed by at least two team members, priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)
Development

Successfully merging this pull request may close these issues.

3 participants