Skip to content

Change ifds' output format to Sarif#178

Merged
lehvolk merged 1 commit intodevelopfrom
volivan239/sarif
Sep 12, 2023
Merged

Change ifds' output format to Sarif#178
lehvolk merged 1 commit intodevelopfrom
volivan239/sarif

Conversation

@volivan239
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 9, 2023

Test results on JDK 8

1 037 tests  ±0   1 026 ✔️ ±0   9m 34s ⏱️ +31s
     45 suites ±0        11 💤 ±0 
     45 files   ±0          0 ±0 

Results for commit 13274a8. ± Comparison against base commit b3704b8.

♻️ This comment has been updated with latest results.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 9, 2023

Codecov Report

Merging #178 (13274a8) into develop (b3704b8) will increase coverage by 0.48%.
The diff coverage is 82.35%.

@@              Coverage Diff              @@
##             develop     #178      +/-   ##
=============================================
+ Coverage      76.71%   77.20%   +0.48%     
- Complexity      1511     1518       +7     
=============================================
  Files            157      157              
  Lines           8581     8615      +34     
  Branches        1464     1475      +11     
=============================================
+ Hits            6583     6651      +68     
+ Misses          1438     1405      -33     
+ Partials         560      559       -1     
Files Changed Coverage Δ
...in/kotlin/org/jacodb/analysis/engine/TraceGraph.kt 100.00% <ø> (+24.32%) ⬆️
...tlin/org/jacodb/analysis/library/RunnersLibrary.kt 76.92% <0.00%> (+36.92%) ⬆️
...jacodb/analysis/library/analyzers/AliasAnalyzer.kt 0.00% <0.00%> (ø)
...in/kotlin/org/jacodb/analysis/sarif/DataClasses.kt 80.76% <80.76%> (ø)
.../org/jacodb/analysis/engine/MainIfdsUnitManager.kt 93.75% <100.00%> (ø)
...otlin/org/jacodb/analysis/engine/SummaryStorage.kt 78.26% <100.00%> (ø)
...rg/jacodb/analysis/engine/VulnerabilityInstance.kt 100.00% <100.00%> (ø)
...g/jacodb/analysis/library/analyzers/NpeAnalyzer.kt 89.92% <100.00%> (+0.15%) ⬆️
...analysis/library/analyzers/SqlInjectionAnalyzer.kt 100.00% <100.00%> (ø)
...jacodb/analysis/library/analyzers/TaintAnalyzer.kt 79.82% <100.00%> (-2.89%) ⬇️
... and 2 more

... and 5 files with indirect coverage changes

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 9, 2023

Test results on JDK 19

1 037 tests  ±0   1 026 ✔️ ±0   8m 50s ⏱️ +10s
     45 suites ±0        11 💤 ±0 
     45 files   ±0          0 ±0 

Results for commit 13274a8. ± Comparison against base commit b3704b8.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 9, 2023

Test results on JDK 11

1 037 tests  ±0   1 029 ✔️ ±0   9m 58s ⏱️ ±0s
     45 suites ±0          8 💤 ±0 
     45 files   ±0          0 ±0 

Results for commit 13274a8. ± Comparison against base commit b3704b8.

♻️ This comment has been updated with latest results.

@lehvolk lehvolk merged commit f6c3dfb into develop Sep 12, 2023
@lehvolk lehvolk deleted the volivan239/sarif branch September 12, 2023 13:26
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.

2 participants