Merge pull request #1573 from frasercrmck/fix-div-by-zero

kbenzie · web-flow · commit 6a51bc4ad2d3 · 2024-05-02T19:32:46.000+01:00
[CTS] Avoid division by zero (when there are no results)
diff --git a/scripts/ctest_parser.py b/scripts/ctest_parser.py
@@ -24,7 +24,7 @@ def get_cts_test_suite_names(working_directory):
     ]
 
 def percent(amount, total):
-    return round((amount / total) * 100, 2)
+    return round((amount / (total or 1)) * 100, 2)
 
 def summarize_results(results):
     total = results['Total']

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ def get_cts_test_suite_names(working_directory):`
`24`	`24`	`]`
`25`	`25`
`26`	`26`	`def percent(amount, total):`
`27`		`- return round((amount / total) * 100, 2)`
	`27`	`+ return round((amount / (total or 1)) * 100, 2)`
`28`	`28`
`29`	`29`	`def summarize_results(results):`
`30`	`30`	`total = results['Total']`