Skip to content

Reorder DocClaims by issuer Metadata in toClaimsModel function#326

Merged
phisakel merged 1 commit intoeu-digital-identity-wallet:mainfrom
niscy-eudiw:update_openid4vci
Apr 17, 2026
Merged

Reorder DocClaims by issuer Metadata in toClaimsModel function#326
phisakel merged 1 commit intoeu-digital-identity-wallet:mainfrom
niscy-eudiw:update_openid4vci

Conversation

@phisakel
Copy link
Copy Markdown
Collaborator

Add tests for CBOR docClaims order and include necessary resources

  • Implemented a new test testCborDocClaimsRespectMetadataOrder to verify that the order of document claims follows the metadata claim order.
  • Added pid-demo-openid-credential-issuer.json to provide issuer metadata for testing.
  • Created pid_demo_current_doc_claims_order.json to represent the current order of document claims for comparison in tests.

Add tests for CBOR docClaims order and include necessary resources
- Implemented a new test `testCborDocClaimsRespectMetadataOrder` to verify that the order of document claims follows the metadata claim order.
- Added `pid-demo-openid-credential-issuer.json` to provide issuer metadata for testing.
- Created `pid_demo_current_doc_claims_order.json` to represent the current order of document claims for comparison in tests.
Copy link
Copy Markdown
Collaborator Author

@phisakel phisakel left a comment

Choose a reason for hiding this comment

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

Fixes issue #323

@phisakel phisakel merged commit 93678ec into eu-digital-identity-wallet:main Apr 17, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants