Skip to content

test(internal/docker): increase coverage#799

Merged
codyoss merged 2 commits intogoogleapis:mainfrom
codyoss:docker-coverage
Jul 8, 2025
Merged

test(internal/docker): increase coverage#799
codyoss merged 2 commits intogoogleapis:mainfrom
codyoss:docker-coverage

Conversation

@codyoss
Copy link
Copy Markdown
Member

@codyoss codyoss commented Jul 8, 2025

Fixes: #754

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 8, 2025

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 29.95%. Comparing base (c75bc54) to head (eac2256).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
internal/docker/environment.go 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #799      +/-   ##
==========================================
+ Coverage   23.42%   29.95%   +6.52%     
==========================================
  Files          24       23       -1     
  Lines        2075     1833     -242     
==========================================
+ Hits          486      549      +63     
+ Misses       1570     1259     -311     
- Partials       19       25       +6     

☔ 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.

@codyoss codyoss requested a review from zhumin8 July 8, 2025 17:57
zhumin8
zhumin8 previously approved these changes Jul 8, 2025
@codyoss codyoss enabled auto-merge (squash) July 8, 2025 21:36
@codyoss codyoss merged commit bd4dea5 into googleapis:main Jul 8, 2025
6 checks passed
@codyoss codyoss deleted the docker-coverage branch July 8, 2025 21:39
cuiy0006 pushed a commit to cuiy0006/librarian that referenced this pull request Jul 9, 2025
julieqiu pushed a commit that referenced this pull request Jul 31, 2025
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.

test(docker): add unit tests for environment.go

2 participants