Skip to content

Conversation

@mnsboev
Copy link
Collaborator

@mnsboev mnsboev commented Jul 22, 2025

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • Appropriate label is added to auto generate release notes.
  • I used gofmt for formatting the code before submitting the pull request.
  • PR description is clear and concise, and it includes the proposed solution/fix.

Description
This PR refactors the VerificationResponse struct in the evidence verification model to align its field naming and structure with the model used by the jf evd get command. Specifically, the SubjectPath and SubjectChecksum fields are now nested within a new Subject struct, improving consistency across evidence-related commands.
Changes
Refactored VerificationResponse to use a nested Subject struct (Path and Sha256).
Updated all affected test files in jfrog-cli-artifactory/evidence to use the new structure.
Fixed test assertions to match updated output where necessary.

@mnsboev mnsboev added the improvement Automatically generated release notes label Jul 22, 2025
@dortam888 dortam888 added ignore for release Automatically generated release notes and removed ignore for release Automatically generated release notes labels Jul 23, 2025
@mnsboev mnsboev merged commit 665880a into jfrog:main Jul 23, 2025
10 of 11 checks passed
naveenku-jfrog pushed a commit to naveenku-jfrog/jfrog-cli-artifactory that referenced this pull request Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants