test(lockfile): cover remote tarball fallback lookup#815
Conversation
Greptile SummaryThis PR extends the
Confidence Score: 5/5Test-only change; no production code is touched. The diff touches a single test fixture by adding a peer-context suffix to two YAML keys. The assertions remain identical in intent (integrity and tarball URL survive a write/re-read cycle), and the change deliberately routes execution through the URL-based fallback lookup that already exists in the parser, providing coverage that was previously absent for that code path. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "test(lockfile): cover remote tarball fal..." | Re-trigger Greptile |
Summary
Tests
Note
Low Risk
Test-only lockfile fixture change; no production code in this diff.
Overview
Updates the
remote_tarball_integrity_survives_lockfile_reuse_roundtripfixture so the remote tarballdemoentry uses pnpm’s peer-context dep-path form (…tgz(react@18.2.0)) in bothpackagesandsnapshots, instead of the bare URL key.The test still checks that integrity and
RemoteTarballSourcesurvive parse/write, but now also covers fallback lookup when the lockfile keys the same tarball URL with a peer suffix.Reviewed by Cursor Bugbot for commit f216149. Bugbot is set up for automated code reviews on this repo. Configure here.