fix: npm version parse test & enable logging

[lili2311]

• [x] Tests written and linted
• [ ] Documentation written / README.md updated https://snyk.io/docs/snyk-for-node/
• [x] Follows CONTRIBUTING agreement
• [x] Commit history is tidy https://git-scm.com/book/en/v2/Git-Branching-Rebasing
• [ ] Reviewed by Snyk team

What this does

Log the full error in debug mode when we fail to grab the lockfile version Add a test for the parsong function to show it works as expected on Node 12 Enable tests on Node 12 as CLI still supports Node 12

[JamesPatrickGill]

Done some digging and writing up findings:

The CLI is currently using snyk-nodejs-lockfile-parser at version 1.45.1. This uses an old version of this function - seen here.

This function is causing an issue as it is looking for an exact string "lockfileVersion: 1, - note the comma. The new function introduced should not have this issue, and so this will be fixed when we bump the CLI in this PR. I aim to get this merged very shortly.

Therefore I say we merge in this PR, the test is a very welcome addition and should have been here the whole time. Then alter the other PR to bump to the newest version - ready for it to be merged.

Thanks again :)

[snyksec]

:tada: This PR is included in version 1.47.4 :tada:

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: