Skip to content

tests: add test coverage#1898

Merged
Wwwsylvia merged 1 commit into
oras-project:mainfrom
vr-varad:test/display_metadata_text_resolve
Nov 26, 2025
Merged

tests: add test coverage#1898
Wwwsylvia merged 1 commit into
oras-project:mainfrom
vr-varad:test/display_metadata_text_resolve

Conversation

@vr-varad

@vr-varad vr-varad commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1866

Please check the following list:

  • Does the affected code have corresponding tests, e.g. unit test, E2E test?
  • Does this change require a documentation update?
  • Does this introduce breaking changes that would require an announcement or bumping the major version?
  • Do all new files have an appropriate license header?

@codecov

codecov Bot commented Nov 7, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.29%. Comparing base (9cb3ed1) to head (69644e0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1898      +/-   ##
==========================================
+ Coverage   87.26%   87.29%   +0.03%     
==========================================
  Files         143      143              
  Lines        5535     5535              
==========================================
+ Hits         4830     4832       +2     
+ Misses        419      417       -2     
  Partials      286      286              

☔ View full report in Codecov by Sentry.
📢 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.

@TerryHowe TerryHowe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@Wwwsylvia Wwwsylvia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Wwwsylvia Wwwsylvia changed the title Fix: Add test coverage tests: add test coverage Nov 26, 2025
Signed-off-by: vr-varad <varadgupta21@gmail.com>
@Wwwsylvia Wwwsylvia force-pushed the test/display_metadata_text_resolve branch from f44d24e to 69644e0 Compare November 26, 2025 01:21
@Wwwsylvia Wwwsylvia merged commit d7f0dcf into oras-project:main Nov 26, 2025
8 checks passed
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.

Add test coverage for cmd/oras/internal/display/metadata/text/resolve.go

3 participants