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.

Sign up for GitHub

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

Jump to bottom

Test/LDG-513--nano-app-implement-credentialdeployment-test #35

Merged
merged 4 commits into from
Dec 10, 2024
Merged

Test/LDG-513--nano-app-implement-credentialdeployment-test #35

merged 4 commits into from
Dec 10, 2024

Conversation

keiff3r
Copy link

@keiff3r keiff3r commented Dec 10, 2024

  • Add test cases for new and existing credential deployment
  • Implement test infrastructure in three parts:
    1. Initial credential deployment with derivation path
    2. Verification key handling with multiple keys
    3. Final deployment with signature threshold, AR identity, and proofs
  • Add test snapshots for nanosp and nanox devices
  • Include test validation for both new and existing credential scenarios
  • Add debug logging for response verification at each step

@keiff3r
test: add credential deployment tests
5cfe894 
- Add test cases for new and existing credential deployment
- Implement test infrastructure in three parts:
  1. Initial credential deployment with derivation path
  2. Verification key handling with multiple keys
  3. Final deployment with signature threshold, AR identity, and proofs
- Add test snapshots for nanosp and nanox devices
- Include test validation for both new and existing credential scenarios
- Add debug logging for response verification at each step
@keiff3r
fix: remove one variable to have 15/15
945ed24
@keiff3r
fix: remove useless todo
6ec8cd3
@keiff3r keiff3r marked this pull request as ready for review December 10, 2024 15:20
@keiff3r keiff3r requested a review from a team December 10, 2024 15:20
@n4l5u0r n4l5u0r requested a review from a team December 10, 2024 15:43
@n4l5u0r n4l5u0r merged commit 0602cd0 into main Dec 10, 2024
29 of 30 checks passed
@n4l5u0r n4l5u0r deleted the test/LDG-513--nano-app-implement-credentialdeployment-test branch December 10, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GuilaneDen GuilaneDen GuilaneDen approved these changes

@n4l5u0r n4l5u0r n4l5u0r approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@keiff3r @n4l5u0r @GuilaneDen