Extending jsonreport.py to also include all branch attributes

SalvatoreZagaria · SalvatoreZagaria · commit f5eb5f215918 · 2020-04-07T10:52:03.000+01:00
Signed-off-by: Salvatore Zagaria &lt;salvatore.zagaria@vector.com&gt;
diff --git a/CONTRIBUTORS.txt b/CONTRIBUTORS.txt
@@ -114,6 +114,7 @@ Rodrigue Cloutier
 Roger Hu
 Ross Lawley
 Roy Williams
+Salvatore Zagaria
 Sandra Martocchia
 Scott Belden
 Sigve Tjora
diff --git a/coverage/jsonreport.py b/coverage/jsonreport.py
@@ -60,6 +60,8 @@ def report(self, morfs, outfile=None):
             self.report_data["totals"].update({
                 'num_branches': self.total.n_branches,
                 'num_partial_branches': self.total.n_partial_branches,
+                'covered_branches': self.total.n_executed_branches,
+                'missing_branches': self.total.n_missing_branches,
             })
 
         json.dump(
diff --git a/tests/test_json.py b/tests/test_json.py
@@ -65,7 +65,9 @@ def test_branch_coverage(self):
                 'num_branches': 2,
                 'excluded_lines': 0,
                 'num_partial_branches': 1,
-                'percent_covered': 60.0
+                'percent_covered': 60.0,
+                'covered_branches': 1,
+                'missing_branches': 1
             }
         }
         self._assert_expected_json_report(cov, expected_result)

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,9 @@ def test_branch_coverage(self):`
`65`	`65`	`'num_branches': 2,`
`66`	`66`	`'excluded_lines': 0,`
`67`	`67`	`'num_partial_branches': 1,`
`68`		`- 'percent_covered': 60.0`
	`68`	`+ 'percent_covered': 60.0,`
	`69`	`+ 'covered_branches': 1,`
	`70`	`+ 'missing_branches': 1`
`69`	`71`	`}`
`70`	`72`	`}`
`71`	`73`	`self._assert_expected_json_report(cov, expected_result)`