Merge pull request #18236 from geoffw0/percent

Rust: Add % of files extracted without errors to summary stats.
github · Dec 6, 2024 · a555a44 · a555a44
2 parents 9513b6c + 1ceee76
commit a555a44
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 0 deletions.
diff --git a/rust/ql/integration-tests/hello-project/summary.expected b/rust/ql/integration-tests/hello-project/summary.expected
@@ -5,6 +5,7 @@
 | Files extracted - total | 5 |
 | Files extracted - with errors | 1 |
 | Files extracted - without errors | 4 |
+| Files extracted - without errors % | 80 |
 | Inconsistencies - AST | 0 |
 | Inconsistencies - CFG | 0 |
 | Inconsistencies - data flow | 0 |

diff --git a/rust/ql/integration-tests/hello-workspace/summary.cargo.expected b/rust/ql/integration-tests/hello-workspace/summary.cargo.expected
@@ -5,6 +5,7 @@
 | Files extracted - total | 4 |
 | Files extracted - with errors | 0 |
 | Files extracted - without errors | 4 |
+| Files extracted - without errors % | 100 |
 | Inconsistencies - AST | 0 |
 | Inconsistencies - CFG | 0 |
 | Inconsistencies - data flow | 0 |

diff --git a/rust/ql/integration-tests/hello-workspace/summary.rust-project.expected b/rust/ql/integration-tests/hello-workspace/summary.rust-project.expected
@@ -5,6 +5,7 @@
 | Files extracted - total | 4 |
 | Files extracted - with errors | 0 |
 | Files extracted - without errors | 4 |
+| Files extracted - without errors % | 100 |
 | Inconsistencies - AST | 0 |
 | Inconsistencies - CFG | 0 |
 | Inconsistencies - data flow | 0 |

diff --git a/rust/ql/src/queries/summary/SummaryStats.ql b/rust/ql/src/queries/summary/SummaryStats.ql
@@ -32,6 +32,11 @@ where
   key = "Files extracted - without errors" and
   value = count(SuccessfullyExtractedFile f | exists(f.getRelativePath()))
   or
+  key = "Files extracted - without errors %" and
+  value =
+    (count(SuccessfullyExtractedFile f | exists(f.getRelativePath())) * 100) /
+      count(ExtractedFile f | exists(f.getRelativePath()))
+  or
   key = "Lines of code extracted" and value = getLinesOfCode()
   or
   key = "Lines of user code extracted" and value = getLinesOfUserCode()

diff --git a/rust/ql/test/query-tests/diagnostics/SummaryStats.expected b/rust/ql/test/query-tests/diagnostics/SummaryStats.expected
@@ -5,6 +5,7 @@
 | Files extracted - total | 7 |
 | Files extracted - with errors | 3 |
 | Files extracted - without errors | 4 |
+| Files extracted - without errors % | 57 |
 | Inconsistencies - AST | 0 |
 | Inconsistencies - CFG | 0 |
 | Inconsistencies - data flow | 0 |