Skip to content
New issue

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

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

Already on GitHub? # to your account

[#noissue] Cleanup PluginTest #11886

Merged
merged 1 commit into from
Jan 2, 2025
Merged

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Jan 2, 2025

No description provided.

@emeroad emeroad added the cleanup label Jan 2, 2025
@emeroad emeroad added this to the 3.1.0 milestone Jan 2, 2025
Copy link

codecov bot commented Jan 2, 2025

Codecov Report

Attention: Patch coverage is 13.40206% with 84 lines in your changes missing coverage. Please review.

Project coverage is 37.37%. Comparing base (407ea8e) to head (28d95a0).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...navercorp/pinpoint/test/plugin/util/FileUtils.java 0.00% 19 Missing ⚠️
...unit5/launcher/SharedPluginForkedTestLauncher.java 0.00% 16 Missing ⚠️
...p/pinpoint/test/plugin/DefaultPluginTestSuite.java 0.00% 9 Missing ⚠️
.../navercorp/pinpoint/test/plugin/util/URLUtils.java 0.00% 9 Missing ⚠️
...oint/test/plugin/DefaultPluginForkedTestSuite.java 0.00% 5 Missing ⚠️
...inpoint/test/plugin/PluginTestInstanceFactory.java 0.00% 5 Missing ⚠️
...rp/pinpoint/test/plugin/maven/MavenRepository.java 0.00% 5 Missing ⚠️
...navercorp/pinpoint/test/plugin/util/ClassPath.java 0.00% 5 Missing ⚠️
...nt/test/plugin/SharedPluginForkedTestInstance.java 0.00% 4 Missing ⚠️
...point/test/plugin/shared/SharedProcessManager.java 0.00% 3 Missing ⚠️
... and 4 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11886      +/-   ##
============================================
- Coverage     37.37%   37.37%   -0.01%     
- Complexity    12650    12654       +4     
============================================
  Files          4149     4149              
  Lines        100710   100720      +10     
  Branches      10482    10482              
============================================
- Hits          37645    37643       -2     
- Misses        60024    60036      +12     
  Partials       3041     3041              

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

@emeroad emeroad changed the title [#noissue] Cleanup [#noissue] Cleanup PluginTest Jan 2, 2025
@emeroad emeroad merged commit f075fbd into pinpoint-apm:master Jan 2, 2025
1 of 3 checks passed
@emeroad emeroad deleted the #noissue_path3 branch January 2, 2025 07:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant