[TASK] Make ProviderFactory available in service container

CPS-IT · Mar 22, 2023 · fad9ebf · fad9ebf
1 parent c6d3533
commit fad9ebf
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 15 deletions.
diff --git a/src/Bootstrap.php b/src/Bootstrap.php
@@ -95,7 +95,7 @@ public static function simulateCreateProject(Script\Event $event): never
     private static function createApplication(IO\Messenger $messenger, string $targetDirectory): Console\Application
     {
         $filesystem = new Filesystem\Filesystem();
-        $providerFactory = Template\Provider\ProviderFactory::create($messenger, $filesystem);
+        $providerFactory = new Template\Provider\ProviderFactory($messenger, $filesystem);
 
         return new Console\Application(
             $messenger,

diff --git a/src/Template/Provider/ProviderFactory.php b/src/Template/Provider/ProviderFactory.php
@@ -36,21 +36,20 @@
 final class ProviderFactory
 {
     /**
-     * @param non-empty-list<ProviderInterface> $providers
+     * @var non-empty-list<ProviderInterface>
      */
-    private function __construct(
-        private readonly array $providers,
-    ) {
-    }
+    private readonly array $providers;
 
-    public static function create(IO\Messenger $messenger, Filesystem\Filesystem $filesystem): self
-    {
-        return new self([
+    public function __construct(
+        IO\Messenger $messenger,
+        Filesystem\Filesystem $filesystem,
+    ) {
+        $this->providers = [
             // sorted by priority
             new PackagistProvider($messenger, $filesystem),
             new ComposerProvider($messenger, $filesystem),
             new VcsProvider($messenger, $filesystem),
-        ]);
+        ];
     }
 
     /**

diff --git a/tests/src/Template/Provider/ProviderFactoryTest.php b/tests/src/Template/Provider/ProviderFactoryTest.php
@@ -25,7 +25,6 @@
 
 use CPSIT\ProjectBuilder as Src;
 use CPSIT\ProjectBuilder\Tests;
-use Symfony\Component\Filesystem;
 
 /**
  * ProviderFactoryTest.
@@ -39,10 +38,7 @@ final class ProviderFactoryTest extends Tests\ContainerAwareTestCase
 
     protected function setUp(): void
     {
-        $this->subject = Src\Template\Provider\ProviderFactory::create(
-            self::$container->get(Src\IO\Messenger::class),
-            self::$container->get(Filesystem\Filesystem::class),
-        );
+        $this->subject = self::$container->get(Src\Template\Provider\ProviderFactory::class);
     }
 
     /**