Merged
Conversation
Agent-Logs-Url: https://github.com/LinkedSoftwareDependencies/Components.js/sessions/35e962ab-b185-4419-9cae-11b5fe144fe6 Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
rubensworks
April 2, 2026 12:06
View session
Pull Request Test Coverage Report for Build 23897264330Details
💛 - Coveralls |
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
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.
Completes the jest 30 upgrade started in #161. Jest 30 removed several deprecated matcher aliases and improved branch coverage detection, requiring both API updates and new test cases.
Dependency updates
jest,@types/jest,jest-mock:^29→^30ts-jestunchanged (29.x already declares jest 30 peer support)Removed deprecated matcher aliases
Jest 30 dropped aliases that were deprecated in jest 27+:
81
toThrowError→toThrowreplacements across 9 test files; 4toBeCalledTimes→toHaveBeenCalledTimesinComponentsManagerBuilder-test.ts.New tests for previously-uncounted branches
Jest 30's coverage engine detects branches that jest 29 missed, causing the 100% branch threshold to fail:
ModuleStateBuilder.warn()— theif (this.logger)false-branch was never exercised because no existing test triggered awarn()call without a logger set. Added abuildComponentContextstest with incompatible major versions and no logger.OverrideMapEntry.findProperties()— theif (collectEntries.list)false-branch (direct resource, not an RDF list) was never hit. Added a test that replaces the component'scollectEntriesproperty with a bareNamedNoderesource.