Ensure HTTP status code is not lost when catching exception by jonesbusy · Pull Request #550 · oras-project/oras-java
Navigation Menu
{{ message }}
oras-project / oras-java Public
- Notifications You must be signed in to change notification settings
- Fork 22
Merged
jonesbusy merged 1 commit intooras-project:mainfrom
Feb 16, 2026Merged
Ensure HTTP status code is not lost when catching exception#550
jonesbusy merged 1 commit intooras-project:mainfrom
Ensure HTTP status code is not lost when catching exception#550
jonesbusy merged 1 commit intooras-project:mainfrom
Conversation
Copy link
Collaborator
Description
Ensure HTTP status code is not lost when catching exception
Testing done
Added one test
Submitter checklist
- I have read and understood the CONTRIBUTING guide
- I have run
mvn license:update-file-header,mvn spotless:apply,pre-commit run -a,mvn clean installbefore opening the PR
jonesbusy
added
the
tests
label
jonesbusy
force-pushed
the
feature/test-cleanup-1
branch
from
7771b6d to
80e9eda
Compare
Copy link
Codecov Report
❌ Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.39%. Comparing base (e652f07) to head (1bcd75e).
⚠️ Report is 2 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| src/main/java/land/oras/auth/HttpClient.java | 66.66% | 2 Missing ⚠️ |
Additional details and impacted files
@@ Coverage Diff @@ ## main #550 +/- ## ============================================ + Coverage 87.57% 88.39% +0.82% - Complexity 768 771 +3 ============================================ Files 42 42 Lines 2253 2258 +5 Branches 277 278 +1 ============================================ + Hits 1973 1996 +23 + Misses 169 153 -16 + Partials 111 109 -2
☔ 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.
jonesbusy
force-pushed
the
feature/test-cleanup-1
branch
from
80e9eda to
a78327c
Compare
jonesbusy
added
the
chore
label
jonesbusy
changed the title
Some test code cleanup
Ensure HTTP status code is not lost when catching exception
jonesbusy
force-pushed
the
feature/test-cleanup-1
branch
from
a78327c to
8e3d94c
Compare
jonesbusy
marked this pull request as ready for review
jonesbusy
force-pushed
the
feature/test-cleanup-1
branch
from
8e3d94c to
a533c6b
Compare
Signed-off-by: Valentin Delaye <jonesbusy@users.noreply.github.com>
jonesbusy
force-pushed
the
feature/test-cleanup-1
branch
from
a533c6b to
1bcd75e
Compare
jonesbusy
deleted the
feature/test-cleanup-1
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment