Skip to content

Commit 9266b5e

Browse files
committed
NonStandard pathCovered for clover lines report
1 parent 76ba107 commit 9266b5e

File tree

4 files changed

+31
-28
lines changed

4 files changed

+31
-28
lines changed

src/CodeCoverage/Report/Clover.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ public function process(PHP_CodeCoverage $coverage, $target = null, $name = null
8888
'ccn' => $method['ccn'],
8989
'count' => $methodCount,
9090
'crap' => $method['crap'],
91+
'pathCovered' => $methodCount ? 1 : 0,
9192
'type' => 'method',
9293
'visibility' => $method['visibility'],
9394
'name' => $methodName
@@ -164,8 +165,9 @@ public function process(PHP_CodeCoverage $coverage, $target = null, $name = null
164165
}
165166

166167
$lines[$line] = [
167-
'count' => count($data['tests']),
168-
'type' => 'stmt',
168+
'count' => count($data['tests']),
169+
'pathCovered' => $data['pathCovered'] ? 1 : 0,
170+
'type' => 'stmt',
169171
];
170172
}
171173

@@ -192,6 +194,7 @@ public function process(PHP_CodeCoverage $coverage, $target = null, $name = null
192194
$xmlLine->setAttribute('crap', $data['crap']);
193195
}
194196

197+
$xmlLine->setAttribute('pathCovered', $data['pathCovered']);
195198
$xmlLine->setAttribute('count', $data['count']);
196199
$xmlFile->appendChild($xmlLine);
197200
}

tests/_files/BankAccount-clover.xml

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,20 +5,20 @@
55
<class name="BankAccount" namespace="global">
66
<metrics complexity="5" methods="4" coveredmethods="3" conditionals="5" coveredconditionals="1" statements="10" coveredstatements="5" elements="14" coveredelements="8"/>
77
</class>
8-
<line num="6" type="method" name="getBalance" visibility="public" complexity="1" crap="1" count="2"/>
9-
<line num="8" type="stmt" count="2"/>
10-
<line num="11" type="method" name="setBalance" visibility="protected" complexity="2" crap="6" count="0"/>
11-
<line num="13" type="stmt" count="0"/>
12-
<line num="14" type="stmt" count="0"/>
13-
<line num="15" type="stmt" count="0"/>
14-
<line num="16" type="stmt" count="0"/>
15-
<line num="18" type="stmt" count="0"/>
16-
<line num="20" type="method" name="depositMoney" visibility="public" complexity="1" crap="1" count="2"/>
17-
<line num="22" type="stmt" count="2"/>
18-
<line num="24" type="stmt" count="1"/>
19-
<line num="27" type="method" name="withdrawMoney" visibility="public" complexity="1" crap="1" count="2"/>
20-
<line num="29" type="stmt" count="2"/>
21-
<line num="31" type="stmt" count="1"/>
8+
<line num="6" type="method" name="getBalance" visibility="public" complexity="1" crap="1" pathCovered="1" count="2"/>
9+
<line num="8" type="stmt" pathCovered="1" count="2"/>
10+
<line num="11" type="method" name="setBalance" visibility="protected" complexity="2" crap="6" pathCovered="0" count="0"/>
11+
<line num="13" type="stmt" pathCovered="0" count="0"/>
12+
<line num="14" type="stmt" pathCovered="0" count="0"/>
13+
<line num="15" type="stmt" pathCovered="0" count="0"/>
14+
<line num="16" type="stmt" pathCovered="0" count="0"/>
15+
<line num="18" type="stmt" pathCovered="0" count="0"/>
16+
<line num="20" type="method" name="depositMoney" visibility="public" complexity="1" crap="1" pathCovered="1" count="2"/>
17+
<line num="22" type="stmt" pathCovered="0" count="2"/>
18+
<line num="24" type="stmt" pathCovered="0" count="1"/>
19+
<line num="27" type="method" name="withdrawMoney" visibility="public" complexity="1" crap="1" pathCovered="1" count="2"/>
20+
<line num="29" type="stmt" pathCovered="0" count="2"/>
21+
<line num="31" type="stmt" pathCovered="0" count="1"/>
2222
<metrics loc="33" ncloc="33" classes="1" methods="4" coveredmethods="3" conditionals="5" coveredconditionals="1" statements="10" coveredstatements="5" elements="14" coveredelements="8"/>
2323
</file>
2424
<metrics files="1" loc="33" ncloc="33" classes="1" methods="4" coveredmethods="3" conditionals="5" coveredconditionals="1" statements="10" coveredstatements="5" elements="14" coveredelements="8"/>

tests/_files/class-with-anonymous-function-clover.xml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,16 @@
55
<class name="CoveredClassWithAnonymousFunctionInStaticMethod" namespace="global">
66
<metrics complexity="2" methods="2" coveredmethods="1" conditionals="2" coveredconditionals="0" statements="5" coveredstatements="4" elements="7" coveredelements="5"/>
77
</class>
8-
<line num="5" type="method" name="runAnonymous" visibility="public" complexity="1" crap="1.04" count="1"/>
9-
<line num="7" type="stmt" count="1"/>
10-
<line num="9" type="stmt" count="1"/>
11-
<line num="10" type="stmt" count="0"/>
12-
<line num="11" type="method" name="anonymous function" complexity="1" crap="1" count="1"/>
13-
<line num="12" type="stmt" count="1"/>
14-
<line num="13" type="stmt" count="1"/>
15-
<line num="14" type="stmt" count="1"/>
16-
<line num="17" type="stmt" count="1"/>
17-
<line num="18" type="stmt" count="1"/>
8+
<line num="5" type="method" name="runAnonymous" visibility="public" complexity="1" crap="1.04" pathCovered="1" count="1"/>
9+
<line num="7" type="stmt" pathCovered="0" count="1"/>
10+
<line num="9" type="stmt" pathCovered="0" count="1"/>
11+
<line num="10" type="stmt" pathCovered="0" count="0"/>
12+
<line num="11" type="method" name="anonymous function" complexity="1" crap="1" pathCovered="1" count="1"/>
13+
<line num="12" type="stmt" pathCovered="0" count="1"/>
14+
<line num="13" type="stmt" pathCovered="0" count="1"/>
15+
<line num="14" type="stmt" pathCovered="0" count="1"/>
16+
<line num="17" type="stmt" pathCovered="0" count="1"/>
17+
<line num="18" type="stmt" pathCovered="0" count="1"/>
1818
<metrics loc="19" ncloc="17" classes="1" methods="2" coveredmethods="1" conditionals="2" coveredconditionals="0" statements="8" coveredstatements="7" elements="10" coveredelements="8"/>
1919
</file>
2020
<metrics files="1" loc="19" ncloc="17" classes="1" methods="2" coveredmethods="1" conditionals="2" coveredconditionals="0" statements="8" coveredstatements="7" elements="10" coveredelements="8"/>

tests/_files/ignored-lines-clover.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
<class name="Bar" namespace="global">
99
<metrics complexity="1" methods="0" coveredmethods="0" conditionals="1" coveredconditionals="0" statements="0" coveredstatements="0" elements="0" coveredelements="0"/>
1010
</class>
11-
<line num="2" type="stmt" count="1"/>
12-
<line num="6" type="stmt" count="0"/>
11+
<line num="2" type="stmt" pathCovered="0" count="1"/>
12+
<line num="6" type="stmt" pathCovered="0" count="0"/>
1313
<metrics loc="37" ncloc="25" classes="2" methods="0" coveredmethods="0" conditionals="3" coveredconditionals="0" statements="2" coveredstatements="1" elements="2" coveredelements="1"/>
1414
</file>
1515
<metrics files="1" loc="37" ncloc="25" classes="2" methods="0" coveredmethods="0" conditionals="3" coveredconditionals="0" statements="2" coveredstatements="1" elements="2" coveredelements="1"/>

0 commit comments

Comments
 (0)