google / google/osv-scanner

Add cases to `output` tests to cover license summary

Open
#1,840 1 comment 0 reactions 0 assignees View on GitHub
backlog
Dominant language
Go
Stars
11k
Forks
792
Avg merge
1d 20h
Merged PRs (30d)
48

Description

It turns out we're not covering logic in the `output` package related to the `LicenseSummary` field - we should expand our test fixtures for that package to include those cases.

I think we need at least two new sets of tests related to license summarizing since you can have `ShowViolations` either true or false - it might be possible though to reuse some of the existing test cases with a field or two changed, so we might want to introduce a new function(s) that return the test cases and accept some parameters

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.