Skip to content

Comments

Refactor tests and add utils #34

Merged
Matthew-Meeco merged 1 commit intomainfrom
task/MPD-2090-sd-jwt-refactor-test-and-add-more-test-cases-for-invalid-inputs
May 29, 2025
Merged

Refactor tests and add utils #34
Matthew-Meeco merged 1 commit intomainfrom
task/MPD-2090-sd-jwt-refactor-test-and-add-more-test-cases-for-invalid-inputs

Conversation

@Matthew-Meeco
Copy link
Contributor

Used testHasher for all tests and moved to top of test file

Created 2 testing utils to clean up code:
createTestDisclosurePackage
createPayloadWithDisclosures

Copy link
Contributor

@vijayshiyani vijayshiyani left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@Matthew-Meeco Matthew-Meeco merged commit be3f52b into main May 29, 2025
2 checks passed
@Matthew-Meeco Matthew-Meeco deleted the task/MPD-2090-sd-jwt-refactor-test-and-add-more-test-cases-for-invalid-inputs branch May 29, 2025 05:31
@leikind
Copy link

leikind commented Jun 26, 2025

@CodiumAI-Agent /improve

@QodoAI-Agent
Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

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.

4 participants