Skip to content

Conversation

elribonazo
Copy link
Contributor

Description:

Simple fix for a bug found during integration tests, when the first input descriptor path is not found but is instead in a secondary path.

Added an improvement in the validateInputDescriptor function and fixed the issue

Checklist:

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

…g the paths to validate it

Signed-off-by: Francisco Javier Ribo Labrador <elribonazo@gmail.com>
@elribonazo elribonazo requested a review from a team as a code owner April 1, 2025 11:31
Signed-off-by: Francisco Javier Ribo Labrador <elribonazo@gmail.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14195952772

Details

  • 11 of 11 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 75.413%

Totals Coverage Status
Change from base Build 14082236348: 0.04%
Covered Lines: 3444
Relevant Lines: 4375

💛 - Coveralls

@elribonazo elribonazo self-assigned this Apr 1, 2025
@elribonazo elribonazo added the bug Something isn't working label Apr 1, 2025
@curtis-h curtis-h merged commit 509bd9b into main Apr 16, 2025
7 checks passed
@curtis-h curtis-h deleted the fix/presentation-paths branch April 16, 2025 11:05
@github-project-automation github-project-automation bot moved this from In Progress to Done in Hyperledger Identus - Roadmap Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

3 participants