Minor update so code matches Coding Standard

ek9 · ek9 · commit e724bb2a4c01 · 2018-03-21T21:16:40.000Z
diff --git a/src/Common/Driver/Factory.php b/src/Common/Driver/Factory.php
@@ -3,6 +3,7 @@
  * Code Coverage Driver Factory
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
diff --git a/src/Common/Driver/HHVM.php b/src/Common/Driver/HHVM.php
@@ -3,6 +3,7 @@
  * HHVM Code Coverage Driver
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -27,7 +28,7 @@ class HHVM implements DriverInterface
      */
     public function __construct()
     {
-        if ( ! defined('HPHP_VERSION')) {
+        if (! defined('HPHP_VERSION')) {
             throw new \SebastianBergmann\CodeCoverage\RuntimeException('This driver requires HHVM');
         }
     }
diff --git a/src/Common/Driver/Stub.php b/src/Common/Driver/Stub.php
@@ -3,6 +3,7 @@
  * Code Coverage Stub Driver
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
diff --git a/src/Common/Driver/XCache.php b/src/Common/Driver/XCache.php
@@ -3,6 +3,7 @@
  * XCache Code Coverage Driver
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -26,7 +27,7 @@ class XCache implements DriverInterface
      */
     public function __construct()
     {
-        if ( ! extension_loaded('xcache')) {
+        if (! extension_loaded('xcache')) {
             throw new \SebastianBergmann\CodeCoverage\RuntimeException('This driver requires XCache');
         }
 
diff --git a/src/Common/Model/Aggregate.php b/src/Common/Model/Aggregate.php
@@ -3,6 +3,7 @@
  * Aggregate
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -36,14 +37,14 @@ public function __construct()
      */
     public function update($class, array $counts)
     {
-        if ( ! isset($this->coverage[$class])) {
+        if (! isset($this->coverage[$class])) {
             $this->coverage[$class] = $counts;
 
             return;
         }
 
         foreach ($counts as $line => $status) {
-            if ( ! isset($this->coverage[$class][$line]) || $status > 0) {
+            if (! isset($this->coverage[$class][$line]) || $status > 0) {
                 // converts "hits" to "status"
                 $status = ! $status ? -1 : ($status > 1 ? 1 : $status);
 
diff --git a/src/Common/Report/Clover.php b/src/Common/Report/Clover.php
@@ -3,6 +3,7 @@
  * Code Coverage Clover Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -34,11 +35,11 @@ class Clover implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! isset($options['target'])) {
+        if (! isset($options['target'])) {
             $options['target'] = null;
         }
 
-        if ( ! isset($options['name'])) {
+        if (! isset($options['name'])) {
             $options['name'] = null;
         }
 
diff --git a/src/Common/Report/Crap4j.php b/src/Common/Report/Crap4j.php
@@ -3,6 +3,7 @@
  * Code Coverage Crap4j Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -11,6 +12,7 @@
 use LeanPHP\Behat\CodeCoverage\Common\ReportInterface;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\CodeCoverage\Report\Crap4j as Crap4jReport;
+
 /**
  * Crap4j report
  *
@@ -33,15 +35,15 @@ class Crap4j implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! class_exists('SebastianBergmann\CodeCoverage\Report\Crap4j')) {
+        if (! class_exists('SebastianBergmann\CodeCoverage\Report\Crap4j')) {
             throw new \Exception('Crap4j requires CodeCoverage 4.0+');
         }
 
-        if ( ! isset($options['target'])) {
+        if (! isset($options['target'])) {
             $options['target'] = null;
         }
 
-        if ( ! isset($options['name'])) {
+        if (! isset($options['name'])) {
             $options['name'] = null;
         }
 
diff --git a/src/Common/Report/Factory.php b/src/Common/Report/Factory.php
@@ -3,6 +3,7 @@
  * Code Coverage Report Factory
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -26,7 +27,7 @@ class Factory
     public function create($reportType, array $options)
     {
         if (in_array($reportType, array('clover', 'crap4j', 'html', 'php', 'text', 'xml'))) {
-            $className = '\LeanPHP\Behat\CodeCoverage\Common\Report\\' . ucfirst($reportType);
+            $className = '\LeanPHP\Behat\CodeCoverage\Common\Report\\'.ucfirst($reportType);
 
             return new $className($options);
         }
diff --git a/src/Common/Report/Html.php b/src/Common/Report/Html.php
@@ -3,6 +3,7 @@
  * Code Coverage HTML Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -34,27 +35,27 @@ class Html implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! isset($options['target'])) {
+        if (! isset($options['target'])) {
             $options['target'] = null;
         }
 
-        if ( ! isset($options['charset'])) {
+        if (! isset($options['charset'])) {
             $options['charset'] = 'UTF-8';
         }
 
-        if ( ! isset($options['highlight'])) {
+        if (! isset($options['highlight'])) {
             $options['highlight'] = false;
         }
 
-        if ( ! isset($options['lowUpperBound'])) {
+        if (! isset($options['lowUpperBound'])) {
             $options['lowUpperBound'] = 35;
         }
 
-        if ( ! isset($options['highUpperBound'])) {
+        if (! isset($options['highUpperBound'])) {
             $options['highUpperBound'] = 70;
         }
 
-        if ( ! isset($options['generator'])) {
+        if (! isset($options['generator'])) {
             $options['generator'] = '';
         }
 
diff --git a/src/Common/Report/Php.php b/src/Common/Report/Php.php
@@ -3,6 +3,7 @@
  * Code Coverage PHP Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -34,7 +35,7 @@ class Php implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! isset($options['target'])) {
+        if (! isset($options['target'])) {
             $options['target'] = null;
         }
 
diff --git a/src/Common/Report/Text.php b/src/Common/Report/Text.php
@@ -3,6 +3,7 @@
  * Code Coverage Text Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
@@ -34,23 +35,23 @@ class Text implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! isset($options['showColors'])) {
+        if (! isset($options['showColors'])) {
             $options['showColors'] = false;
         }
 
-        if ( ! isset($options['printer'])) {
+        if (! isset($options['printer'])) {
             $options['printer'] = null;
         }
 
-        if ( ! isset($options['lowUpperBound'])) {
+        if (! isset($options['lowUpperBound'])) {
             $options['lowUpperBound'] = 35;
         }
 
-        if ( ! isset($options['highUpperBound'])) {
+        if (! isset($options['highUpperBound'])) {
             $options['highUpperBound'] = 70;
         }
 
-        if ( ! isset($options['showUncoveredFiles'])) {
+        if (! isset($options['showUncoveredFiles'])) {
             $options['showUncoveredFiles'] = false;
         }
 
@@ -64,7 +65,7 @@ public function __construct(array $options)
                 $options['showUncoveredFiles']
             );
         } else {
-            if ( ! isset($options['showOnlySummary'])) {
+            if (! isset($options['showOnlySummary'])) {
                 $options['showOnlySummary'] = false;
             }
 
@@ -90,6 +91,11 @@ public function process(CodeCoverage $coverage)
         );
     }
 
+    /**
+     * return version of CodeCoverage
+     *
+     * @return string
+     */
     private function getVersion()
     {
         $reflectionMethod = new \ReflectionMethod('SebastianBergmann\CodeCoverage\Report\Text', '__construct');
diff --git a/src/Common/Report/Xml.php b/src/Common/Report/Xml.php
@@ -3,14 +3,15 @@
  * Code Coverage XML Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
 namespace LeanPHP\Behat\CodeCoverage\Common\Report;
 
 use LeanPHP\Behat\CodeCoverage\Common\ReportInterface;
 use SebastianBergmann\CodeCoverage\CodeCoverage;
-use SebastianBergmann\CodeCoverage\Report\Xml\Facade;;
+use SebastianBergmann\CodeCoverage\Report\Xml\Facade;
 
 /**
  * XML report
@@ -34,11 +35,11 @@ class Xml implements ReportInterface
      */
     public function __construct(array $options)
     {
-        if ( ! class_exists('SebastianBergmann\CodeCoverage\Report\Xml\Facade')) {
+        if (! class_exists('SebastianBergmann\CodeCoverage\Report\Xml\Facade')) {
             throw new \Exception('XML requires CodeCoverage 4.0+');
         }
 
-        if ( ! isset($options['target'])) {
+        if (! isset($options['target'])) {
             $options['target'] = null;
         }
 
diff --git a/src/Common/ReportInterface.php b/src/Common/ReportInterface.php
@@ -3,6 +3,7 @@
  * Report
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-3-Clause
  */
 
diff --git a/src/Compiler/DriverPass.php b/src/Compiler/DriverPass.php
@@ -3,6 +3,7 @@
  * Driver Compiler Pass
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-2-Clause
  */
 
diff --git a/src/Compiler/FactoryPass.php b/src/Compiler/FactoryPass.php
@@ -3,6 +3,7 @@
  * Factory Compiler Pass
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-2-Clause
  */
 
diff --git a/src/Compiler/FilterPass.php b/src/Compiler/FilterPass.php
@@ -3,6 +3,7 @@
  * Filter Compiler Pass
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-2-Clause
  */
 
@@ -28,6 +29,9 @@ public function process(ContainerBuilder $container)
         $this->processCodeCoverageFilter($container);
     }
 
+    /**
+     * @param ContainerBuilder $container
+     */
     private function processCodeCoverage(ContainerBuilder $container)
     {
         if (! $container->hasDefinition('behat.code_coverage.php_code_coverage')) {
@@ -55,6 +59,9 @@ private function processCodeCoverage(ContainerBuilder $container)
         );
     }
 
+    /**
+     * @param ContainerBuilder $container
+     */
     private function processCodeCoverageFilter(ContainerBuilder $container)
     {
         if (! $container->hasDefinition('behat.code_coverage.php_code_coverage_filter')) {
diff --git a/src/Driver/Proxy.php b/src/Driver/Proxy.php
@@ -3,6 +3,7 @@
  * Proxy Code Coverage Driver
  *
  * @copyright 2013 Anthon Pang
+ *
  * @license BSD-2-Clause
  */
 
@@ -51,7 +52,7 @@ public function start($determineUnusedAndDead = true)
      */
     public function stop()
     {
-        $aggregate = new Aggregate;
+        $aggregate = new Aggregate();
 
         foreach ($this->drivers as $driver) {
             $coverage = $driver->stop();
diff --git a/src/Driver/RemoteXdebug.php b/src/Driver/RemoteXdebug.php
diff --git a/src/Extension.php b/src/Extension.php
diff --git a/src/Listener/EventListener.php b/src/Listener/EventListener.php
diff --git a/src/Service/ReportService.php b/src/Service/ReportService.php

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage Driver Factory`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* HHVM Code Coverage Driver`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -27,7 +28,7 @@ class HHVM implements DriverInterface`
`27`	`28`	`*/`
`28`	`29`	`public function __construct()`
`29`	`30`	`{`
`30`		`- if ( ! defined('HPHP_VERSION')) {`
	`31`	`+ if (! defined('HPHP_VERSION')) {`
`31`	`32`	`throw new \SebastianBergmann\CodeCoverage\RuntimeException('This driver requires HHVM');`
`32`	`33`	`}`
`33`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage Stub Driver`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* XCache Code Coverage Driver`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -26,7 +27,7 @@ class XCache implements DriverInterface`
`26`	`27`	`*/`
`27`	`28`	`public function __construct()`
`28`	`29`	`{`
`29`		`- if ( ! extension_loaded('xcache')) {`
	`30`	`+ if (! extension_loaded('xcache')) {`
`30`	`31`	`throw new \SebastianBergmann\CodeCoverage\RuntimeException('This driver requires XCache');`
`31`	`32`	`}`
`32`	`33`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage Clover Report`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -34,11 +35,11 @@ class Clover implements ReportInterface`
`34`	`35`	`*/`
`35`	`36`	`public function __construct(array $options)`
`36`	`37`	`{`
`37`		`- if ( ! isset($options['target'])) {`
	`38`	`+ if (! isset($options['target'])) {`
`38`	`39`	`$options['target'] = null;`
`39`	`40`	`}`
`40`	`41`
`41`		`- if ( ! isset($options['name'])) {`
	`42`	`+ if (! isset($options['name'])) {`
`42`	`43`	`$options['name'] = null;`
`43`	`44`	`}`
`44`	`45`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage Crap4j Report`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -11,6 +12,7 @@`
`11`	`12`	`use LeanPHP\Behat\CodeCoverage\Common\ReportInterface;`
`12`	`13`	`use SebastianBergmann\CodeCoverage\CodeCoverage;`
`13`	`14`	`use SebastianBergmann\CodeCoverage\Report\Crap4j as Crap4jReport;`
	`15`	`+`
`14`	`16`	`/**`
`15`	`17`	`* Crap4j report`
`16`	`18`	`*`
`@@ -33,15 +35,15 @@ class Crap4j implements ReportInterface`
`33`	`35`	`*/`
`34`	`36`	`public function __construct(array $options)`
`35`	`37`	`{`
`36`		`- if ( ! class_exists('SebastianBergmann\CodeCoverage\Report\Crap4j')) {`
	`38`	`+ if (! class_exists('SebastianBergmann\CodeCoverage\Report\Crap4j')) {`
`37`	`39`	`throw new \Exception('Crap4j requires CodeCoverage 4.0+');`
`38`	`40`	`}`
`39`	`41`
`40`		`- if ( ! isset($options['target'])) {`
	`42`	`+ if (! isset($options['target'])) {`
`41`	`43`	`$options['target'] = null;`
`42`	`44`	`}`
`43`	`45`
`44`		`- if ( ! isset($options['name'])) {`
	`46`	`+ if (! isset($options['name'])) {`
`45`	`47`	`$options['name'] = null;`
`46`	`48`	`}`
`47`	`49`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage Report Factory`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -26,7 +27,7 @@ class Factory`
`26`	`27`	`public function create($reportType, array $options)`
`27`	`28`	`{`
`28`	`29`	`if (in_array($reportType, array('clover', 'crap4j', 'html', 'php', 'text', 'xml'))) {`
`29`		`- $className = '\LeanPHP\Behat\CodeCoverage\Common\Report\\' . ucfirst($reportType);`
	`30`	`+ $className = '\LeanPHP\Behat\CodeCoverage\Common\Report\\'.ucfirst($reportType);`
`30`	`31`
`31`	`32`	`return new $className($options);`
`32`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage HTML Report`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -34,27 +35,27 @@ class Html implements ReportInterface`
`34`	`35`	`*/`
`35`	`36`	`public function __construct(array $options)`
`36`	`37`	`{`
`37`		`- if ( ! isset($options['target'])) {`
	`38`	`+ if (! isset($options['target'])) {`
`38`	`39`	`$options['target'] = null;`
`39`	`40`	`}`
`40`	`41`
`41`		`- if ( ! isset($options['charset'])) {`
	`42`	`+ if (! isset($options['charset'])) {`
`42`	`43`	`$options['charset'] = 'UTF-8';`
`43`	`44`	`}`
`44`	`45`
`45`		`- if ( ! isset($options['highlight'])) {`
	`46`	`+ if (! isset($options['highlight'])) {`
`46`	`47`	`$options['highlight'] = false;`
`47`	`48`	`}`
`48`	`49`
`49`		`- if ( ! isset($options['lowUpperBound'])) {`
	`50`	`+ if (! isset($options['lowUpperBound'])) {`
`50`	`51`	`$options['lowUpperBound'] = 35;`
`51`	`52`	`}`
`52`	`53`
`53`		`- if ( ! isset($options['highUpperBound'])) {`
	`54`	`+ if (! isset($options['highUpperBound'])) {`
`54`	`55`	`$options['highUpperBound'] = 70;`
`55`	`56`	`}`
`56`	`57`
`57`		`- if ( ! isset($options['generator'])) {`
	`58`	`+ if (! isset($options['generator'])) {`
`58`	`59`	`$options['generator'] = '';`
`59`	`60`	`}`
`60`	`61`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`* Code Coverage PHP Report`
`4`	`4`	`*`
`5`	`5`	`* @copyright 2013 Anthon Pang`
	`6`	`+ *`
`6`	`7`	`* @license BSD-3-Clause`
`7`	`8`	`*/`
`8`	`9`
`@@ -34,7 +35,7 @@ class Php implements ReportInterface`
`34`	`35`	`*/`
`35`	`36`	`public function __construct(array $options)`
`36`	`37`	`{`
`37`		`- if ( ! isset($options['target'])) {`
	`38`	`+ if (! isset($options['target'])) {`
`38`	`39`	`$options['target'] = null;`
`39`	`40`	`}`
`40`	`41`