Merge pull request php-vcr#212 from mjanser/add-file-check

Add a file check to StreamProcessor

Author: renatomefi

src/VCR/Util/StreamProcessor.php

@@ -171,6 +171,10 @@ protected function shouldProcess($uri)
  */
  public function stream_open($path, $mode, $options, &$openedPath)
  {
+ if ('r' === substr($mode, 0, 1) && !is_file($path)) {
+ return false;
+ }
+
  $this->restore();
 
  if (isset($this->context)) {

tests/VCR/Util/StreamProcessorTest.php

@@ -58,6 +58,33 @@ public function streamOpenAppendFilterProvider()
  );
  }
 
+ public function streamOpenFileModesWhichDoNotCreateFiles()
+ {
+ return array(
+ array('r'),
+ array('rb'),
+ array('rt'),
+ array('r+')
+ );
+ }
+ /**
+ * @dataProvider streamOpenFileModesWhichDoNotCreateFiles
+ */
+ public function testStreamOpenShouldNotFailOnNonExistingFile($fileMode)
+ {
+ $test = $this;
+ set_error_handler(function ($errno, $errstr, $errfile, $errline) use ($test) {
+ $test->fail('should not throw errors');
+ });
+
+ $processor = new StreamProcessor();
+
+ $result = $processor->stream_open('tests/fixtures/unknown', $fileMode, StreamProcessor::STREAM_OPEN_FOR_INCLUDE, $fullPath);
+ $this->assertFalse($result);
+
+ restore_error_handler();
+ }
+
  public function testUrlStatSuccessfully()
  {
  $test = $this;