PHPLIB-199: Convert Exceptions to E_USER_WARNING in StreamWrapper

Unfortunately, PHP's StreamWrapper API requires that we trigger errors instead of throwing exceptions. This will affect driver exceptions thrown during IO and CorruptFileException thrown by ReadableStream.

Author: jmikola

src/GridFS/StreamWrapper.php

@@ -2,6 +2,8 @@
 
 namespace MongoDB\GridFS;
 
+use Exception;
+
 /**
  * Stream wrapper for reading and writing a GridFS file.
  *
@@ -105,7 +107,12 @@ public function stream_read($count)
             return '';
         }
 
-        return $this->stream->downloadNumBytes($count);
+        try {
+            return $this->stream->downloadNumBytes($count);
+        } catch (Exception $e) {
+            trigger_error(sprintf('%s: %s', get_class($e), $e->getMessage()), \E_USER_WARNING);
+            return false;
+        }
     }
 
     /**
@@ -137,8 +144,12 @@ public function stream_write($data)
             return 0;
         }
 
-        $this->stream->insertChunks($data);
-        return strlen($data);
+        try {
+            return $this->stream->insertChunks($data);
+        } catch (Exception $e) {
+            trigger_error(sprintf('%s: %s', get_class($e), $e->getMessage()), \E_USER_WARNING);
+            return false;
+        }
     }
 
     /**

tests/GridFS/BucketFunctionalTest.php

@@ -125,7 +125,7 @@ public function testDeleteStillRemovesChunksIfFileDoesNotExist($input, $expected
     }
 
     /**
-     * @expectedException MongoDB\GridFS\Exception\CorruptFileException
+     * @expectedException PHPUnit_Framework_Error_Warning
      */
     public function testDownloadingFileWithMissingChunk()
     {
@@ -137,7 +137,7 @@ public function testDownloadingFileWithMissingChunk()
     }
 
     /**
-     * @expectedException MongoDB\GridFS\Exception\CorruptFileException
+     * @expectedException PHPUnit_Framework_Error_Warning
      */
     public function testDownloadingFileWithUnexpectedChunkIndex()
     {
@@ -152,7 +152,7 @@ public function testDownloadingFileWithUnexpectedChunkIndex()
     }
 
     /**
-     * @expectedException MongoDB\GridFS\Exception\CorruptFileException
+     * @expectedException PHPUnit_Framework_Error_Warning
      */
     public function testDownloadingFileWithUnexpectedChunkSize()
     {

tests/GridFS/SpecFunctionalTest.php

@@ -6,9 +6,9 @@
 use MongoDB\BSON\Binary;
 use MongoDB\BSON\ObjectId;
 use MongoDB\BSON\UTCDateTime;
-use MongoDB\Exception\RuntimeException;
 use MongoDB\Operation\BulkWrite;
 use DateTime;
+use Exception;
 use IteratorIterator;
 use LogicException;
 use MultipleIterator;
@@ -50,7 +50,7 @@ public function testSpecification(array $initialData, array $test)
 
         try {
             $result = $this->executeAct($test['act']);
-        } catch (RuntimeException $e) {
+        } catch (Exception $e) {
             $result = $e;
         }
 
@@ -333,7 +333,10 @@ private function getExceptionClassForError($error)
 
             case 'ChunkIsMissing':
             case 'ChunkIsWrongSize':
-                return 'MongoDB\GridFS\Exception\CorruptFileException';
+                /* Although ReadableStream throws a CorruptFileException, the
+                 * stream wrapper will convert it to a PHP error of type
+                 * E_USER_WARNING. */
+                return 'PHPUnit_Framework_Error_Warning';
 
             default:
                 throw new LogicException('Unsupported error: ' . $error);