Merge pull request php-vcr#414 from php-vcr/fix/ci

Fix/ci

Author: higidi

phpstan-baseline.neon

@@ -21,7 +21,12 @@ parameters:
 path: src/VCR/Util/SoapClient.php
 
 -
-message: "#^Parameter \\#2 \\$length of function fread expects int\\<0, max\\>, int given\\.$#"
+message: "#^Offset 'uri' does not exist on array\\{timed_out\\: bool, blocked\\: bool, eof\\: bool, unread_bytes\\: int, stream_type\\: string, wrapper_type\\: string, wrapper_data\\: mixed, mode\\: string, \\.\\.\\.\\}\\.$#"
+count: 1
+path: src/VCR/Util/StreamProcessor.php
+
+-
+message: "#^Parameter \\#2 \\$length of function fread expects int\\<1, max\\>, int given\\.$#"
 count: 1
 path: src/VCR/Util/StreamProcessor.php
 
@@ -30,6 +35,11 @@ parameters:
 count: 1
 path: src/VCR/Util/StreamProcessor.php
 
+-
+message: "#^Parameter \\#3 \\$microseconds of function stream_set_timeout expects int, int\\|null given\\.$#"
+count: 1
+path: src/VCR/Util/StreamProcessor.php
+
 -
 message: "#^Cannot call method record\\(\\) on VCR\\\\Cassette\\|null\\.$#"
 count: 1
@@ -55,6 +65,31 @@ parameters:
 count: 1
 path: tests/Unit/RequestTest.php
 
+-
+message: "#^Parameter \\#1 \\$stream of function fclose expects resource, resource\\|false given\\.$#"
+count: 1
+path: tests/Unit/Util/StreamProcessorTest.php
+
+-
+message: "#^Parameter \\#1 \\$stream of function stream_set_blocking expects resource, resource\\|false given\\.$#"
+count: 1
+path: tests/Unit/Util/StreamProcessorTest.php
+
+-
+message: "#^Parameter \\#1 \\$stream of function stream_set_read_buffer expects resource, resource\\|false given\\.$#"
+count: 1
+path: tests/Unit/Util/StreamProcessorTest.php
+
+-
+message: "#^Parameter \\#1 \\$stream of function stream_set_timeout expects resource, resource\\|false given\\.$#"
+count: 2
+path: tests/Unit/Util/StreamProcessorTest.php
+
+-
+message: "#^Parameter \\#1 \\$stream of function stream_set_write_buffer expects resource, resource\\|false given\\.$#"
+count: 1
+path: tests/Unit/Util/StreamProcessorTest.php
+
 -
 message: "#^Call to an undefined method Symfony\\\\Contracts\\\\EventDispatcher\\\\EventDispatcherInterface\\:\\:addListener\\(\\)\\.$#"
 count: 1

src/VCR/CodeTransform/AbstractCodeTransform.php

@@ -24,7 +24,7 @@ public function register(): void
  $isRegistered = stream_filter_register(static::NAME, static::class);
  Assertion::true(
  $isRegistered,
- sprintf('Failed registering stream filter "%s" on stream "%s"', static::class, static::NAME)
+ \sprintf('Failed registering stream filter "%s" on stream "%s"', static::class, static::NAME)
  );
  }
  }

src/VCR/Exceptions/InvalidHostException.php

@@ -8,6 +8,6 @@ class InvalidHostException extends \InvalidArgumentException
 {
  public static function create(?string $url): self
  {
- return new self(sprintf('Could not read host from URL "%s". Please check the URL syntax.', $url));
+ return new self(\sprintf('Could not read host from URL "%s". Please check the URL syntax.', $url));
  }
 }

src/VCR/LibraryHooks/SoapHook.php

@@ -49,7 +49,7 @@ public function doRequest(string $request, string $location, string $action, int
  } else { // >= SOAP_1_2
  $vcrRequest->setHeader(
  'Content-Type',
- sprintf('application/soap+xml; charset=utf-8; action="%s"', $action)
+ \sprintf('application/soap+xml; charset=utf-8; action="%s"', $action)
  );
  }
 

src/VCR/VCRFactory.php

@@ -48,7 +48,7 @@ protected function createStorage(string $cassetteName): Storage
  Assertion::subclassOf(
  $className,
  Storage::class,
- sprintf('Storage class "%s" is not a subclass of "%s".', $className, Storage::class)
+ \sprintf('Storage class "%s" is not a subclass of "%s".', $className, Storage::class)
  );
 
  /** @var Storage $storage */

src/VCR/Videorecorder.php

@@ -171,7 +171,7 @@ public function handleRequest(Request $request): Response
  || VCR::MODE_ONCE === $this->config->getMode()
  && false === $this->cassette->isNew()
  ) {
- throw new \LogicException(sprintf("The request does not match a previously recorded request and the 'mode' is set to '%s'. If you want to send the request anyway, make sure your 'mode' is set to 'new_episodes'. ".'Please see http://php-vcr.github.io/documentation/configuration/#record-modes.'."\nCassette: %s \n Request: %s", $this->config->getMode(), $this->cassette->getName(), print_r($request->toArray(), true)));
+ throw new \LogicException(\sprintf("The request does not match a previously recorded request and the 'mode' is set to '%s'. If you want to send the request anyway, make sure your 'mode' is set to 'new_episodes'. ".'Please see http://php-vcr.github.io/documentation/configuration/#record-modes.'."\nCassette: %s \n Request: %s", $this->config->getMode(), $this->cassette->getName(), print_r($request->toArray(), true)));
  }
 
  $this->disableLibraryHooks();

tests/Unit/CodeTransform/CurlCodeTransformTest.php

@@ -14,7 +14,7 @@ final class CurlCodeTransformTest extends TestCase
  */
  public function testTransformCode(string $expected, string $code): void
  {
- $codeTransform = new class() extends CurlCodeTransform {
+ $codeTransform = new class extends CurlCodeTransform {
  // A proxy to access the protected transformCode method.
  public function publicTransformCode(string $code): string
  {

tests/Unit/CodeTransform/SoapCodeTransformTest.php

@@ -14,7 +14,7 @@ final class SoapCodeTransformTest extends TestCase
  */
  public function testTransformCode(string $expected, string $code): void
  {
- $codeTransform = new class() extends SoapCodeTransform {
+ $codeTransform = new class extends SoapCodeTransform {
  // A proxy to access the protected transformCode method.
  public function publicTransformCode(string $code): string
  {

tests/Unit/RequestTest.php

@@ -89,7 +89,7 @@ public function testStorePostFields(): void
  'headers' => [
  'User-Agent' => 'Unit-Test',
  'Host' => 'example.com',
-  ],
+ ],
  'post_fields' => ['para1' => 'val1'],
  ],
  $this->request->toArray()
@@ -107,7 +107,7 @@ public function testRestorePostFields(): void
  'headers' => [
  'User-Agent' => 'Unit-Test',
  'Host' => 'example.com',
-  ],
+ ],
  'post_fields' => ['para1' => 'val1'],
  ],
  $restoredRequest->toArray()
@@ -177,7 +177,7 @@ public function testRestorePostFiles(): void
  'headers' => [
  'User-Agent' => 'Unit-Test',
  'Host' => 'example.com',
-  ],
+ ],
  'post_files' => [$file],
  ],
  $restoredRequest->toArray()

tests/Unit/VCRTest.php

@@ -114,7 +114,7 @@ public function testShouldThrowExceptionIfNoCassettePresent(): void
  VCR::configure()->enableLibraryHooks(['stream_wrapper']);
  VCR::turnOn();
  // If there is no cassette inserted, a request should throw an exception
- file_get_contents('http://example.com'); // @phpstan-ignore-line
+ file_get_contents('http://example.com');
  VCR::turnOff();
  }