Skip to content

Conversation

@anthonyharrison
Copy link
Contributor

No description provided.

anthonyharrison and others added 30 commits May 27, 2020 12:55
@terriko
Copy link
Contributor

terriko commented May 3, 2022

Created a conflict with the excel macro filter code/tests. I believe I've resolved it.

@terriko
Copy link
Contributor

terriko commented Jun 16, 2022

I'm confused. it claims it's failing on a line you didn't change in console.py:

    time_of_last_update: datetime | str,

Were we not calling that function at all in the tests or is something else going on?

I'm going to update the branch and see if that helps.

Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, a simple re-run didn't help. @anthonyharrison can you take a look and figure out what's going on here with the tests?

@codecov-commenter
Copy link

Codecov Report

Merging #1658 (81d2563) into main (86455c1) will decrease coverage by 1.96%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main    #1658      +/-   ##
==========================================
- Coverage   81.18%   79.21%   -1.97%     
==========================================
  Files         299      299              
  Lines        6286     6356      +70     
  Branches     1029     1044      +15     
==========================================
- Hits         5103     5035      -68     
- Misses        976     1103     +127     
- Partials      207      218      +11     
Flag Coverage Δ
longtests 79.21% <57.14%> (-1.97%) ⬇️

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

Impacted Files Coverage Δ
cve_bin_tool/output_engine/__init__.py 53.27% <0.00%> (-5.10%) ⬇️
cve_bin_tool/output_engine/console.py 97.41% <100.00%> (+0.53%) ⬆️
cve_bin_tool/output_engine/util.py 93.18% <100.00%> (+0.87%) ⬆️
cve_bin_tool/checkers/xml2.py 61.36% <0.00%> (-27.28%) ⬇️
test/test_json.py 70.00% <0.00%> (-20.00%) ⬇️
test/test_scanner.py 62.30% <0.00%> (-12.31%) ⬇️
cve_bin_tool/cvedb.py 71.88% <0.00%> (-12.23%) ⬇️
test/test_cli.py 81.39% <0.00%> (-9.31%) ⬇️
cve_bin_tool/nvd_api.py 75.20% <0.00%> (-8.80%) ⬇️
cve_bin_tool/checkers/glibc.py 95.83% <0.00%> (-4.17%) ⬇️
... and 5 more

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

@anthonyharrison
Copy link
Contributor Author

@terriko Chnaged the code to remove the extra |str from the parameter which was introduced in #1632 and that seem to work

@anthonyharrison anthonyharrison requested a review from terriko June 19, 2022 20:25
Copy link
Contributor

@terriko terriko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@terriko terriko merged commit a8d9eeb into intel:main Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants