add missing typehints

Author: guillaume-sainthillier

src/Exception/MissingPackageException.php

@@ -16,7 +16,7 @@
  */
 final class MissingPackageException extends \RuntimeException
 {
-    public function __construct($message = '', \Throwable $previous = null)
+    public function __construct(string $message = '', \Throwable $previous = null)
     {
         parent::__construct($message, 0, $previous);
     }

src/Lazy/LazyFactory.php

@@ -11,6 +11,7 @@
 
 namespace League\FlysystemBundle\Lazy;
 
+use League\Flysystem\FilesystemOperator;
 use Psr\Container\ContainerInterface;
 
 /**
@@ -27,7 +28,7 @@ public function __construct(ContainerInterface $storages)
         $this->storages = $storages;
     }
 
-    public function createStorage(string $source, string $storageName)
+    public function createStorage(string $source, string $storageName): FilesystemOperator
     {
         if ($source === $storageName) {
             throw new \InvalidArgumentException('The "lazy" adapter source is referring to itself as "'.$source.'", which would lead to infinite recursion.');

tests/Adapter/AdapterDefinitionFactoryTest.php

@@ -30,7 +30,7 @@ public function provideConfigOptions(): \Generator
     /**
      * @dataProvider provideConfigOptions
      */
-    public function testCreateDefinition($name, $options)
+    public function testCreateDefinition($name, $options): void
     {
         $factory = new AdapterDefinitionFactory();
 

tests/Adapter/Builder/AsyncAwsAdapterDefinitionBuilderTest.php

@@ -41,7 +41,7 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $definition = $this->createBuilder()->createDefinition($options, Visibility::PRIVATE);
         $this->assertSame(AsyncAwsS3Adapter::class, $definition->getClass());

tests/Adapter/Builder/AwsAdapterDefinitionBuilderTest.php

@@ -49,12 +49,12 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(AwsS3V3Adapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
 
-    public function testOptionsBehavior()
+    public function testOptionsBehavior(): void
     {
         $definition = $this->createBuilder()->createDefinition([
             'client' => 'my_client',

tests/Adapter/Builder/AzureAdapterDefinitionBuilderTest.php

@@ -40,12 +40,12 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(AzureBlobStorageAdapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
 
-    public function testOptionsBehavior()
+    public function testOptionsBehavior(): void
     {
         $definition = $this->createBuilder()->createDefinition([
             'client' => 'my_client',

tests/Adapter/Builder/FtpAdapterDefinitionBuilderTest.php

@@ -48,12 +48,12 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(FtpAdapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
 
-    public function testOptionsBehavior()
+    public function testOptionsBehavior(): void
     {
         $definition = $this->createBuilder()->createDefinition([
             'host' => 'ftp.example.com',

tests/Adapter/Builder/GcloudAdapterDefinitionBuilderTest.php

@@ -55,12 +55,12 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(GoogleCloudStorageAdapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
 
-    public function testOptionsBehavior()
+    public function testOptionsBehavior(): void
     {
         $definition = $this->createBuilder()->createDefinition([
             'client' => 'my_client',

tests/Adapter/Builder/GridFSAdapterDefinitionBuilderTest.php

@@ -60,7 +60,7 @@ public static function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(GridFSAdapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
@@ -86,7 +86,7 @@ public static function provideInvalidOptions(): \Generator
     /**
      * @dataProvider provideInvalidOptions
      */
-    public function testInvalidOptions(array $options, string $message)
+    public function testInvalidOptions(array $options, string $message): void
     {
         $builder = $this->createBuilder();
 
@@ -96,7 +96,7 @@ public function testInvalidOptions(array $options, string $message)
         $builder->createDefinition($options, null);
     }
 
-    public function testInitializeBucketFromDocumentManager()
+    public function testInitializeBucketFromDocumentManager(): void
     {
         $client = new Client();
         $config = new Configuration();
@@ -112,7 +112,7 @@ public function testInitializeBucketFromDocumentManager()
         $this->assertSame('avatars', $bucket->getBucketName());
     }
 
-    public function testInitializeBucketFromConfig()
+    public function testInitializeBucketFromConfig(): void
     {
         $bucket = GridFSAdapterDefinitionBuilder::initializeBucketFromConfig(
             'mongodb://server:27017/',

tests/Adapter/Builder/LocalAdapterDefinitionBuilderTest.php

@@ -88,12 +88,12 @@ public function provideValidOptions(): \Generator
     /**
      * @dataProvider provideValidOptions
      */
-    public function testCreateDefinition($options)
+    public function testCreateDefinition($options): void
     {
         $this->assertSame(LocalFilesystemAdapter::class, $this->createBuilder()->createDefinition($options, null)->getClass());
     }
 
-    public function testOptionsBehavior()
+    public function testOptionsBehavior(): void
     {
         $permissions = [
             'file' => [