Set signature attribute on the <line> element (and reorder attributes to match XSD)

Author: sebastianbergmann

src/Report/Clover.php

@@ -109,6 +109,7 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                         'ccn'        => $method['ccn'],
                         'count'      => $methodCount,
                         'type'       => 'method',
+                        'signature'  => $method['signature'],
                         'visibility' => $method['visibility'],
                     ];
                 }
@@ -149,15 +150,20 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                 $xmlLine->setAttribute('num', (string) $line);
                 $xmlLine->setAttribute('type', $data['type']);
 
-                if (isset($data['visibility'])) {
-                    $xmlLine->setAttribute('visibility', $data['visibility']);
-                }
-
                 if (isset($data['ccn'])) {
                     $xmlLine->setAttribute('complexity', (string) $data['ccn']);
                 }
 
                 $xmlLine->setAttribute('count', (string) $data['count']);
+
+                if (isset($data['signature'])) {
+                    $xmlLine->setAttribute('signature', $data['signature']);
+                }
+
+                if (isset($data['visibility'])) {
+                    $xmlLine->setAttribute('visibility', $data['visibility']);
+                }
+
                 $xmlFile->appendChild($xmlLine);
             }
 

tests/_files/BankAccount-clover-line.xml

@@ -9,16 +9,16 @@
         <class name="BankAccount">
           <metrics complexity="5" elements="12" coveredelements="8" conditionals="0" coveredconditionals="0" statements="8" coveredstatements="5" methods="4" coveredmethods="3"/>
         </class>
-        <line num="6" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="6" type="method" complexity="1" count="2" signature="getBalance()" visibility="public"/>
         <line num="8" type="stmt" count="2"/>
-        <line num="11" type="method" visibility="protected" complexity="2" count="0"/>
+        <line num="11" type="method" complexity="2" count="0" signature="setBalance($balance)" visibility="protected"/>
         <line num="13" type="stmt" count="0"/>
         <line num="14" type="stmt" count="0"/>
         <line num="16" type="stmt" count="0"/>
-        <line num="20" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="20" type="method" complexity="1" count="2" signature="depositMoney($balance)" visibility="public"/>
         <line num="22" type="stmt" count="2"/>
         <line num="24" type="stmt" count="1"/>
-        <line num="27" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="27" type="method" complexity="1" count="2" signature="withdrawMoney($balance)" visibility="public"/>
         <line num="29" type="stmt" count="2"/>
         <line num="31" type="stmt" count="1"/>
       </file>

tests/_files/BankAccount-clover-path.xml

@@ -9,16 +9,16 @@
         <class name="BankAccount">
           <metrics complexity="5" elements="19" coveredelements="11" conditionals="7" coveredconditionals="3" statements="8" coveredstatements="5" methods="4" coveredmethods="3"/>
         </class>
-        <line num="6" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="6" type="method" complexity="1" count="2" signature="getBalance()" visibility="public"/>
         <line num="8" type="stmt" count="2"/>
-        <line num="11" type="method" visibility="protected" complexity="2" count="0"/>
+        <line num="11" type="method" complexity="2" count="0" signature="setBalance($balance)" visibility="protected"/>
         <line num="13" type="stmt" count="0"/>
         <line num="14" type="stmt" count="0"/>
         <line num="16" type="stmt" count="0"/>
-        <line num="20" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="20" type="method" complexity="1" count="2" signature="depositMoney($balance)" visibility="public"/>
         <line num="22" type="stmt" count="2"/>
         <line num="24" type="stmt" count="1"/>
-        <line num="27" type="method" visibility="public" complexity="1" count="2"/>
+        <line num="27" type="method" complexity="1" count="2" signature="withdrawMoney($balance)" visibility="public"/>
         <line num="29" type="stmt" count="2"/>
         <line num="31" type="stmt" count="1"/>
       </file>

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

@@ -9,7 +9,7 @@
         <class name="CoveredClassWithAnonymousFunctionInStaticMethod">
           <metrics complexity="1" elements="9" coveredelements="9" conditionals="0" coveredconditionals="0" statements="8" coveredstatements="8" methods="1" coveredmethods="1"/>
         </class>
-        <line num="5" type="method" visibility="public" complexity="1" count="1"/>
+        <line num="5" type="method" complexity="1" count="1" signature="runAnonymous()" visibility="public"/>
         <line num="7" type="stmt" count="1"/>
         <line num="9" type="stmt" count="1"/>
         <line num="10" type="stmt" count="1"/>