fix(pack): detect unresolved package properly

[zepatrik]

See bug report in #3132
The test case was failing before adjusting the check.

References

Fixes #3132

[wraithgar]

We'd prefer if the test didn't make real network calls if possible. Not sure what's happening here, read-package-json-fast adds this to the manifest, and pacote uses that to parse package.json files:

https://github.com/npm/pacote/blob/latest/lib/file.js#L29

https://github.com/npm/read-package-json-fast/blob/master/index.js#L32-L36

const add_id = data => {
  if (data.name && data.version)
    data._id = `${data.name}@${data.version}`
  return data
}

[zepatrik]

Hm interesting, I did not check other packages but the one I have here is returned by pacote without _id...
I will close this PR then, as it probably needs an upstream fix.

[wraithgar]

The root cause is that npm-pick-manifest returns different data than read-package-json-fast. We're discussing it to see what path to take.

[RonSherfey]

changes reviewed