tests: crypto: update HASH test and add AES testcase #98099
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
zephyrbot added platform: ITE zephyrbot requested review from Chenhongren, Dino-Li, FRASTM, GTLin08, GeorgeCGV, RuibinChang, ceolin, djiatsaf-st, erwango, fabiobaltieri, gautierg-st, keith-zephyr, marwaiehm-st, mathieuchopstm and valeriosetti 
sylvioalves force-pushed the feature/crypto-tests-aes-sha branch from 641ce39 to 320be9b Compare zephyrbot requested review from ChiHuaL, JasonLin-RealTek, ioannis-karachalios and jvasanth1 zephyrbot requested review from MulinChao, nashif, vaishnavachath and ydamigos sylvioalves force-pushed the feature/crypto-tests-aes-sha branch from 320be9b to 70d1f45 Compare 
sylvioalves commented Oct 24, 2025
sylvioalves force-pushed the feature/crypto-tests-aes-sha branch from 70d1f45 to e552fca Compare 
Make sure all crypto driver returns proper error when feature is not supported. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Select SHA24, SHA256 and SHA384 as part of the shim driver. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Refactor hash tests to support SHA-224, SHA-256, SHA-384, and SHA-512 algorithms with a unified test vector framework. This improves test coverage and makes it easier to verify hardware crypto drivers that support multiple SHA variants. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
sylvioalves force-pushed the feature/crypto-tests-aes-sha branch from e552fca to 170402d Compare Add AES test cases for ECB, CBC, CCM, and GCM modes with standard test vectors from FIPS-197 and NIST specifications. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Add ESP32 devkits in testcases. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
sylvioalves force-pushed the feature/crypto-tests-aes-sha branch from 170402d to 92f3f4d Compare 
| Please retry analysis of this Pull-Request directly on SonarQube Cloud |
ceolin approved these changes Oct 28, 2025
nashif approved these changes Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
This PR updates the crypto test cases:
crypto_hashtest scenarioscrypto_aestest caseSplit from #97991