Skip to content

fix: Update sample manifest for c2pa v2 - #359

Merged
scouten-adobe merged 1 commit into
mainfrom
fix-v2-test-manifest
Oct 8, 2026
Merged

scouten-adobe merged 1 commit into
mainfrom
fix-v2-test-manifest

Conversation

@scouten-adobe

Copy link
Copy Markdown
Collaborator

Changes in this pull request

The integration fixture used a v1 c2pa.opened action without an ingredient, which conflicts with the current c2pa-rs validation and generated edit intent. Update it to a v2 c2pa.edited action so the fixture matches the signing intent; parent-ingredient tests can still generate their own opened action.

Validation: cargo test --bins --tests — passed (14 unit tests and 49 integration tests).

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@scouten-adobe scouten-adobe changed the title test: update sample manifest for c2pa v2 fix: Update sample manifest for c2pa v2 Oct 8, 2026
@codecov

codecov Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.39%. Comparing base (74ce1bd) to head (81d0f1c).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #359   +/-   ##
=======================================
  Coverage   79.39%   79.39%           
=======================================
  Files           4        4           
  Lines        1097     1097           
=======================================
  Hits          871      871           
  Misses        226      226           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@scouten-adobe scouten-adobe added the backport-stable Cherry-pick this merged main PR onto the stable release line label Oct 8, 2026
@scouten-adobe
scouten-adobe merged commit 55b8266 into main Oct 8, 2026
18 of 19 checks passed
@scouten-adobe
scouten-adobe deleted the fix-v2-test-manifest branch October 8, 2026 14:08
@caiopensrc

caiopensrc commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Backport-action backported this pull request in workflow run 37790095914.

Target Status
stable ✅ Created #360

scouten-adobe added a commit that referenced this pull request Oct 8, 2026
(cherry picked from commit 55b8266)

Co-authored-by: Eric Scouten <[email protected]>
Co-authored-by: Copilot App <[email protected]>
@caiopensrc caiopensrc mentioned this pull request Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-stable Cherry-pick this merged main PR onto the stable release line

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants