fix: rpm extractor for windows

[b31ngd3v]

No description provided.

[anthonyharrison]

There should be a check that the zstd application is available before trying to run the zsd command. However could we not use the zstandard Python library instead of running the zstd command, noting that the zstandard library is already included as part of the dependencies listed in the requirements.txt file?

[codecov-commenter]

Codecov Report

Merging #1696 (7379ae0) into main (09ebae5) will decrease coverage by 0.37%.
The diff coverage is 40.62%.

@@            Coverage Diff             @@
##             main    #1696      +/-   ##
==========================================
- Coverage   78.76%   78.39%   -0.38%     
==========================================
  Files         298      298              
  Lines        6189     6216      +27     
  Branches     1006     1011       +5     
==========================================
- Hits         4875     4873       -2     
- Misses       1096     1124      +28     
- Partials      218      219       +1     

Flag	Coverage Δ
longtests	78.39% <40.62%> (-0.38%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
cve_bin_tool/extractor.py	54.04% <0.00%> (-4.12%)	⬇️
test/test_extractor.py	100.00% <100.00%> (ø)
cve_bin_tool/nvd_api.py	71.20% <0.00%> (-12.80%)	⬇️
cve_bin_tool/cli.py	70.90% <0.00%> (+0.40%)	⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[b31ngd3v]

hi @anthonyharrison i've updated the function, can you please take a look at it now? and thank you for reviewing this pr <3

[anthonyharrison]

Do we need to add some new tests for this building on #1682?

[b31ngd3v]

Do we need to add some new tests for this building on #1682?

i checked the PR (#1683), it has tests for zst and pkg but i think we need one more rpm extractor test (which uses zstd), so i added the test in this PR.

[terriko]

Looks good; thanks for making those changes!