Skip to content

test: add test coverage for transformPeerDependenciesMeta#10600

Closed
Copilot wants to merge 103 commits into
mainfrom
copilot/sub-pr-10591-another-one
Closed

test: add test coverage for transformPeerDependenciesMeta#10600
Copilot wants to merge 103 commits into
mainfrom
copilot/sub-pr-10591-another-one

Conversation

Copilot AI commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

The transformPeerDependenciesMeta function lacked test coverage for its behavioral transformation where the optional field defaults to false when unspecified.

Changes

  • Added pkg-manifest/exportable-manifest/test/peerDependenciesMeta.test.ts with 8 test cases covering:
    • optional field defaulting to false when undefined
    • Explicit true/false values preserved correctly
    • Multiple entries with mixed values
    • Additional properties preserved during transformation
    • Absent or undefined peerDependenciesMeta handled correctly

Coverage Impact

  • peerDependenciesMeta.js: 25% → 100% line coverage

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issues based on feedback for npm publish refactor test: add test coverage for transformPeerDependenciesMeta Feb 12, 2026
Copilot AI requested a review from KSXGitHub February 12, 2026 13:16
KSXGitHub added a commit that referenced this pull request Feb 12, 2026
@zkochan zkochan closed this in cc7c0d2 Feb 12, 2026
Base automatically changed from use-libnpmpublish-without-merge-conflict-test0 to main February 12, 2026 20:10
@KSXGitHub KSXGitHub deleted the copilot/sub-pr-10591-another-one branch February 12, 2026 20:47
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.

2 participants