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

tests/sys/psa_crypto: test interleaved hash operations #20606

Merged

Conversation

mguetschow
Copy link
Contributor

Contribution description

Wrote this test while debugging some code, just to make sure the PSA Crypto API behaves as expected when using different hash operations in an interleaved form. It does, but I think it doesn't harm to expand the test coverage.

Testing procedure

CI should confirm it's working.

@mguetschow mguetschow requested review from Einhornhool and removed request for aabadie, leandrolanzieri and MichelRottleuthner April 22, 2024 10:32
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 22, 2024
@github-actions github-actions bot added Area: tests Area: tests and testing framework and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Apr 22, 2024
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 22, 2024
@riot-ci
Copy link

riot-ci commented Apr 22, 2024

Murdock results

✔️ PASSED

07f841d tests/sys/psa_crypto: test interleaved hash operations

Success Failures Total Runtime
196 0 196 02m:59s

Artifacts

Copy link
Contributor

@Einhornhool Einhornhool left a comment

Choose a reason for hiding this comment

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

Looks good to me and runs and passes on my computer and the nRF52840dk.

@mguetschow mguetschow requested a review from benpicco May 16, 2024 08:34
@mguetschow mguetschow added this pull request to the merge queue May 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 16, 2024
@mguetschow mguetschow added this pull request to the merge queue May 16, 2024
Merged via the queue into RIOT-OS:master with commit de64a1c May 16, 2024
29 checks passed
@mguetschow mguetschow deleted the test-psa-crypto-hash-interleaved branch May 17, 2024 08:24
@mguetschow mguetschow added this to the Release 2024.07 milestone Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants