8000 test: Improve Test Coverage for `package_list_parser.py` · Issue #4875 · intel/cve-bin-tool · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

test: Improve Test Coverage for package_list_parser.py #4875

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
JigyasuRajput opened this issue Mar 2, 2025 · 3 comments · Fixed by #4909
Closed

test: Improve Test Coverage for package_list_parser.py #4875

JigyasuRajput opened this issue Mar 2, 2025 · 3 comments · Fixed by #4909
Assignees
@JigyasuRajput

Comments

@JigyasuRajput
Copy link
Contributor
JigyasuRajput commented Mar 2, 2025 •
edited
Loading

Description:

cve_bin_tool/package_list_parser.py lacks sufficient test coverage.

Functions Needing Tests:

  1. parse_list(self) ~ report

    • Parses package lists from different package managers (dpkg, rpm, pacman, pip).
    • Needs tests for different distributions, missing packages, and JSON output parsing.

  2. add_vendor(self, vendor_package_pairs) ~ report

    • Associates vendors with package names using CVE database data.
    • Needs tests for vendor matching and edge cases where no vendor is found.

  3. parse_data(self) ~ report

    • Converts parsed package data into structured output.
    • Needs tests for handling missing or malformed data.

  4. check_file(self) ~ report

    • Validates the package list file before parsing.
    • Needs tests for handling missing files, incorrect formats, and invalid package lists.
@JigyasuRajput
Copy link
Contributor Author

There was a hiccup, I need to update this issue

@JigyasuRajput JigyasuRajput changed the title test: Increase Test Coverage for helper_script.py test: Increase Improve Test Coverage for package_list_parser.py Mar 2, 2025
@JigyasuRajput
Copy link
Contributor Author

updated the issue!
I checked the old reports by mistake

@JigyasuRajput JigyasuRajput changed the title test: Increase Improve Test Coverage for package_list_parser.py test: Improve Test Coverage for package_list_parser.py Mar 2, 2025
@JigyasuRajput
Copy link
Contributor Author

Hey! I'm working on this issue

@JigyasuRajput JigyasuRajput mentioned this issue Mar 6, 2025
Merged
terriko added a commit that referenced this issue Apr 14, 2025
@JigyasuRajput @terriko
test(package_list_parser): Improve test coverage for package_list (#4909
e36aabe 
)

* fixes #4875

Co-authored-by: Terri Oda <terri.oda@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JigyasuRajput JigyasuRajput

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test(package_list_parser): Improve test coverage for package_list JigyasuRajput/cve-bin-tool
1 participant
@JigyasuRajput
0