Test: TC-DA-1-9: Device attestation revocation test cases #38128
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
|
PR #38128: Size comparison from 794cc19 to 9c09712 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #38128: Size comparison from 794cc19 to 786c7e5 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
credentials/test/revoked-attestation-certificates/revocation-sets/revocation-set.json
Show resolved
Hide resolved
Clubbed the different revocation set into single file and adjusted the doc accordingly.
Added the test cases for testing device attestation revocation feature
786c7e5 to
96ea0d9
Compare
|
PR #38128: Size comparison from 86e1daf to 1ae11b3 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
shubhamdp
commented
Apr 4, 2025
cjandhyala
approved these changes
Apr 10, 2025
cecille
reviewed
Apr 14, 2025
log path for storing app logs, also fix the ci by passing --PICS argument
|
PR #38128: Size comparison from a473302 to a8709d3 Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
cecille
approved these changes
Apr 17, 2025
|
Approving it for the merge as it is approved by Cecille and Chaitanya |
dhrishi
approved these changes
Apr 18, 2025
dsavitsky-dsr
pushed a commit
to popovdg/connectedhomeip
that referenced
this pull request
Apr 22, 2025
…ip#38128) * da-revocation: add test data for dac and pai both revoked case Clubbed the different revocation set into single file and adjusted the doc accordingly. * Added certification test cases for TC-DA-1-9 Added the test cases for testing device attestation revocation feature * fix the yaml indentation * Restyled by whitespace * Restyled by prettier-json * Restyled by prettier-markdown * Restyled by prettier-yaml * Add CRL for revoked PAI CA and adjust the unit tests after clubbing data into single file * remove manual test case for TC-DA-1-9 * update comment in dac provider test vectors * merge revocation sets * add python test for tc-da-1-9 * Restyled by autopep8 * Restyled by isort * remove unnecessary if * skip running tc-da-1-9 in ci * Restyled by prettier-yaml * change test case as per python testing requirements * enable tc-da in ci * add a todo * Restyled by autopep8 * fix the app path * Restyled by autopep8 * fix paths for ci * remove unused commnet * address review comments * add asserts when required arguments are not present and use the default log path for storing app logs, also fix the ci by passing --PICS argument --------- Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 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.
Testing
Manually ran all the tests listed in the YAML test on the mac host
Unit tests pass
Also ran the TC_DA_1_9.py locally
Fixes #38129