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.
Overview
This fixes a parser error when a
package-lock.json
dependency'sresolved
key is a boolean rather than a string.Acceptance criteria
The parser for
package-lock.json
dependencies should ignore a booleanresolved
key.Testing plan
I added unit tests for the dependency parser. I also manually ran
fossa analyze
against the repo referenced in #718 and verified that it now parses successfully.To reproduce you can run the master branch's
fossa analyze
against the repo in the above issue.Risks
References
Closes fossas/team-analysis#849
Closes #718
Checklist
docs/
.Changelog.md
if this change is externally facing. If this PR did not mark a release, I added my changes into an# Unreleased
section at the top.*schema.json
if I have made changes for.fossa.yml
,fossa-deps.{json, yaml, yml}
. You may also need to update these if you have added/removed new dependency (e.g. pip) or analysis target type (e.g. poetry).