Skip to content

test: Backport trust sidecar fixture path fix - #341

Closed
scouten-adobe wants to merge 1 commit into
stablefrom
backport-339-trust-sidecar
Closed

scouten-adobe wants to merge 1 commit into
stablefrom
backport-339-trust-sidecar

Conversation

@scouten-adobe

@scouten-adobe scouten-adobe commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Manual stable backport after the automated #339 backport failed.

This backports the stable-compatible part of #339:

  • fixes the trust-sidecar test fixture path from the old workspace layout (../../cli/...) to the standalone repo path (../tests/...).

The stronger trust.anchors assertion from #339 depends on c2pa 0.91+, while current stable still has c2pa 0.90.x where that field does not exist. Stable therefore keeps the existing trust_anchors assertion until the release-line c2pa 0.91 bump lands separately in #340.

The stable CI/workflow parity work has been split into #342.

Validation:

  • cargo test --bins apply_trust_sidecars_reads_official_pem

Backport-of: #339

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Backports the stable-compatible part of #339: the test fixture now uses the
standalone repository path instead of the old workspace path. The stronger
trust.anchors assertion from main depends on c2pa 0.91+, so stable keeps the
existing trust_anchors assertion until the release-line dependency bump lands.

(cherry picked from commit f316b90)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (stable@fbabc2e). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             stable     #341   +/-   ##
=========================================
  Coverage          ?   33.68%           
=========================================
  Files             ?        4           
  Lines             ?     1045           
  Branches          ?        0           
=========================================
  Hits              ?      352           
  Misses            ?      693           
  Partials          ?        0           

☔ 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

Copy link
Copy Markdown
Collaborator Author

Closing in favor of #342, which combines the stable-compatible #339 backport with the CI update needed for stable PRs to validate cleanly.

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.

1 participant