Add comprehensive test coverage improvements#142
Merged
slachiewicz merged 1 commit intocodehaus-plexus:masterfrom Apr 3, 2026
Merged
Add comprehensive test coverage improvements#142slachiewicz merged 1 commit intocodehaus-plexus:masterfrom
slachiewicz merged 1 commit intocodehaus-plexus:masterfrom
Conversation
Enhanced test coverage from 73% to 79% instruction coverage: - UrlUtils: 1→13 tests covering path normalization edge cases - Launcher: 7→20 tests covering getters/setters and argument handling - OsgiBundleStrategy: 3→19 tests covering loadClass, getResource, getResources - ParentFirstStrategy: 3→19 tests covering loadClass, getResource, getResources - Added JaCoCo plugin (0.8.12) for coverage reporting Key improvements: - Test corner cases in UrlUtils.normalizeUrlPath() - Test null handling and edge cases in strategies - Test unconfigured Launcher scenarios - Test argument variations (null, empty, array) Co-authored-by: Cerebras Agent <193945191+isaact-cerebras@users.noreply.github.com> Add comprehensive test coverage for exceptions, strategies, and integration scenarios - Add tests for all exception classes (ClassWorldException, DuplicateRealmException, NoSuchRealmException, ConfigurationException) - Add tests for all strategy implementations (SelfFirst, ParentFirst, OsgiBundle, StrategyFactory) - Add tests for ClassWorldListener notification mechanism - Add integration tests for edge cases and concurrency scenarios - Enhanced ClassWorldTest with 14 additional tests covering constructors, disposal, listeners, and realm management - Overall test count increased from 92 to 155 (+68%) Co-authored-by: Cerebras Agent <193945191+isaact-cerebras@users.noreply.github.com>
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
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enhanced test coverage from 73% to 79% instruction coverage:
Key improvements:
Add comprehensive test coverage for exceptions, strategies, and integration scenarios