Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adds read_cai test for PDF with content credentials #366

Merged
merged 2 commits into from
Jan 23, 2024
Merged

Conversation

dyro
Copy link
Contributor

@dyro dyro commented Jan 10, 2024

Changes in this pull request

Adds read_cai test for PDF with content credentials. This test replaces a previous test which added fake content credentials to a file to verify the read functionality.

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.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a36caaf) 79.47% compared to head (965a031) 79.46%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   79.47%   79.46%   -0.01%     
==========================================
  Files          80       80              
  Lines       23939    23928      -11     
==========================================
- Hits        19026    19015      -11     
  Misses       4913     4913              

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

Copy link
Collaborator

@mauricefisher64 mauricefisher64 left a comment

Choose a reason for hiding this comment

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

👍

@dyro dyro merged commit 16fe91e into main Jan 23, 2024
22 checks passed
@dyro dyro deleted the dyross/CAI-5197 branch January 23, 2024 17:48
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.

3 participants