Skip to content
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.

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

test: improve flaky java language scanner test #3362

Merged
merged 2 commits into from
Oct 3, 2023
Merged

Conversation

terriko
Copy link
Contributor

@terriko terriko commented Sep 27, 2023

For some reason we're getting non-deterministic behaviour around commons-io (with a dash) vs commons_io (with an underscore) during detection. Since the test doesn't really need to detect that partcular library, we're leaving it out for now while the issue is investigated further.

* Related: intel#3360
* Fixes: intel#3361

For some reason we're getting non-deterministic behaviour around commons-io
(with a dash) vs commons_io (with an underscore) during detection.  Since the
test doesn't really need to detect that partcular library, we're leaving it out
for now while the issue is investigated further.

Signed-off-by: Terri Oda <terri.oda@intel.com>
Signed-off-by: Terri Oda <terri.oda@intel.com>
terriko added a commit to terriko/cve-bin-tool that referenced this pull request Oct 2, 2023
This is just in case intel#3362 doesn't work as expected.

Signed-off-by: Terri Oda <terri.oda@intel.com>
@codecov-commenter
Copy link

Codecov Report

Merging #3362 (bc55cd2) into main (55afd3c) will decrease coverage by 2.15%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3362      +/-   ##
==========================================
- Coverage   81.07%   78.92%   -2.15%     
==========================================
  Files         750      748       -2     
  Lines       11534    11392     -142     
  Branches     1560     1330     -230     
==========================================
- Hits         9351     8991     -360     
- Misses       1734     1965     +231     
+ Partials      449      436      -13     
Flag Coverage Δ
longtests ?
win-longtests 78.92% <ø> (-0.15%) ⬇️

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

Files Coverage Δ
test/test_language_scanner.py 92.85% <ø> (ø)

... and 22 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@terriko terriko merged commit 4233f47 into intel:main Oct 3, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test: improve test_java_package test to be more stable
3 participants