Merge branch '5.3'

Author: sebastianbergmann

ChangeLog-5.3.md

@@ -2,7 +2,11 @@
 
 All notable changes of the PHP_CodeCoverage 5.3 release series are documented in this file using the [Keep a CHANGELOG](http://keepachangelog.com/) principles.
 
-## [5.3.1] - 2017-MM-DD
+## [5.3.1] - 2018-04-06
+
+### Changed
+
+* `Clover`, `Crap4j`, and `PHP` report writers now raise an exception when their call to `file_put_contents()` fails
 
 ### Fixed
 

ChangeLog-6.0.md

@@ -2,6 +2,12 @@
 
 All notable changes of the PHP_CodeCoverage 6.0 release series are documented in this file using the [Keep a CHANGELOG](http://keepachangelog.com/) principles.
 
+## [6.0.2] - 2018-MM-DD
+
+### Changed
+
+* `Clover`, `Crap4j`, and `PHP` report writers now raise an exception when their call to `file_put_contents()` fails
+
 ## [6.0.1] - 2018-02-02
 
 * Fixed [#584](https://github.com/sebastianbergmann/php-code-coverage/issues/584): Target directories are not created recursively
@@ -22,6 +28,7 @@ All notable changes of the PHP_CodeCoverage 6.0 release series are documented in
 * Implemented [#562](https://github.com/sebastianbergmann/php-code-coverage/issues/562): Remove code specific to Hack language constructs
 * Implemented [#563](https://github.com/sebastianbergmann/php-code-coverage/issues/563): Drop support for PHP 7.0
 
+[6.0.2]: https://github.com/sebastianbergmann/php-code-coverage/compare/6.0.1...6.0.2
 [6.0.1]: https://github.com/sebastianbergmann/php-code-coverage/compare/6.0.0...6.0.1
 [6.0.0]: https://github.com/sebastianbergmann/php-code-coverage/compare/5.2...6.0.0
 

src/Report/Clover.php

@@ -12,6 +12,7 @@
 
 use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\CodeCoverage\Node\File;
+use SebastianBergmann\CodeCoverage\RuntimeException;
 
 /**
  * Generates a Clover XML logfile from a code coverage object.
@@ -238,7 +239,14 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                 throw new \RuntimeException(\sprintf('Directory "%s" was not created', \dirname($target)));
             }
 
-            \file_put_contents($target, $buffer);
+            if (@\file_put_contents($target, $buffer) === false) {
+                throw new RuntimeException(
+                    \sprintf(
+                        'Could not write to "%s',
+                        $target
+                    )
+                );
+            }
         }
 
         return $buffer;

src/Report/Crap4j.php

@@ -12,6 +12,7 @@
 
 use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\CodeCoverage\Node\File;
+use SebastianBergmann\CodeCoverage\RuntimeException;
 
 final class Crap4j
 {
@@ -120,7 +121,14 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                 throw new \RuntimeException(\sprintf('Directory "%s" was not created', \dirname($target)));
             }
 
-            \file_put_contents($target, $buffer);
+            if (@\file_put_contents($target, $buffer) === false) {
+                throw new RuntimeException(
+                    \sprintf(
+                        'Could not write to "%s',
+                        $target
+                    )
+                );
+            }
         }
 
         return $buffer;

src/Report/PHP.php

@@ -21,7 +21,7 @@ public function process(CodeCoverage $coverage, ?string $target = null): string
     {
         $filter = $coverage->filter();
 
-        $output = \sprintf(
+        $buffer = \sprintf(
             '<?php
 $coverage = new SebastianBergmann\CodeCoverage\CodeCoverage;
 $coverage->setData(%s);
@@ -37,9 +37,16 @@ public function process(CodeCoverage $coverage, ?string $target = null): string
         );
 
         if ($target !== null) {
-            return \file_put_contents($target, $output);
+            if (@\file_put_contents($target, $buffer) === false) {
+                throw new RuntimeException(
+                    \sprintf(
+                        'Could not write to "%s',
+                        $target
+                    )
+                );
+            }
         }
 
-        return $output;
+        return $buffer;
     }
 }