feat: fixed #96 and added some test for the InstallationManager (#114)

| Q               | A
| --------------- | ---
| Bug fix?        | no
| New feature?    | yes
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | fixes #96 
| License         | MIT
| Doc PR          | -

Author: prisis

src/Automatic/Automatic.php

@@ -216,18 +216,23 @@ public function activate(Composer $composer, IOInterface $io): void
 
         $this->container = new Container($composer, $io);
 
-        /** @var \Composer\Installer\InstallationManager $installationManager */
-        $installationManager = $composer->getInstallationManager();
-        $installationManager->addInstaller($this->container->get(ConfiguratorInstaller::class));
-        $installationManager->addInstaller($this->container->get(SkeletonInstaller::class));
-
         /** @var \Narrowspark\Automatic\LegacyTagsManager $tagsManager */
         $tagsManager = $this->container->get(LegacyTagsManager::class);
 
         $this->configureLegacyTagsManager($io, $tagsManager, $this->container->get('composer-extra'));
 
         $composer->setRepositoryManager($this->extendRepositoryManager($composer, $io, $tagsManager));
 
+        // overwrite composer instance
+        $this->container->set(Composer::class, static function () use ($composer) {
+            return $composer;
+        });
+
+        /** @var \Composer\Installer\InstallationManager $installationManager */
+        $installationManager = $composer->getInstallationManager();
+        $installationManager->addInstaller($this->container->get(ConfiguratorInstaller::class));
+        $installationManager->addInstaller($this->container->get(SkeletonInstaller::class));
+
         $this->container->get(Lock::class)->add('@readme', [
             'This file locks the automatic information of your project to a known state',
             'This file is @generated automatically',
@@ -237,9 +242,9 @@ public function activate(Composer $composer, IOInterface $io): void
 
         $this->extendComposer(\debug_backtrace(), $tagsManager);
 
-        $this->container->set(InstallationManager::class, static function (Container $container) use ($composer) {
+        $this->container->set(InstallationManager::class, static function (Container $container) {
             return new InstallationManager(
-                $composer,
+                $container->get(Composer::class),
                 $container->get(IOInterface::class),
                 $container->get(InputInterface::class)
             );
@@ -1077,7 +1082,7 @@ private function extendComposer($backtrace, LegacyTagsManager $tagsManager): voi
     }
 
     /**
-     * Extend the repository manager with a truncated composer repository.
+     * Extend the repository manager with a truncated composer repository and parallel downloader.
      *
      * @param \Composer\Composer                       $composer
      * @param \Composer\IO\IOInterface                 $io

src/Automatic/Prefetcher/ParallelDownloader.php

@@ -108,7 +108,7 @@ public function __construct(IOInterface $io, Config $config, array $options = []
     {
         $this->io = $io;
 
-        if (! \method_exists(RemoteFilesystem::class, 'getRemoteContents')) {
+        if (! \method_exists(parent::class, 'getRemoteContents')) {
             $this->io->writeError('Composer >=1.7 not found, downloads will happen in sequence', true, IOInterface::DEBUG);
         // @codeCoverageIgnoreStart
         } elseif (! \extension_loaded('curl')) {

src/Automatic/Prefetcher/Prefetcher.php

@@ -58,11 +58,6 @@ class Prefetcher
      */
     private $config;
 
-    /**
-     * @var bool
-     */
-    private $cacheDirPopulated = false;
-
     /**
      * Patch to the file cache.
      *
@@ -71,10 +66,19 @@ class Prefetcher
     private $cacheFilesDir;
 
     /**
+     * Check if the repo cache should be populated.
+     *
      * @var null|bool
      */
     private $populateRepoCacheDir;
 
+    /**
+     * Check if the dir cache should be populated.
+     *
+     * @var bool
+     */
+    private $cacheDirPopulated = false;
+
     /**
      * @var array
      */

src/Common/Installer/AbstractInstallationManager.php

@@ -129,10 +129,6 @@ public function __construct(Composer $composer, IOInterface $io, InputInterface
 
         $this->versionSelector = new VersionSelector($pool);
         $this->localRepository = $this->composer->getRepositoryManager()->getLocalRepository();
-
-        foreach ($this->localRepository->getPackages() as $package) {
-            $this->installedPackages[\strtolower($package->getName())] = \ltrim($package->getPrettyVersion(), 'v');
-        }
     }
 
     /**

tests/Automatic/AutomaticTest.php

@@ -59,6 +59,8 @@ final class AutomaticTest extends MockeryTestCase
      */
     protected function setUp(): void
     {
+        parent::setUp();
+
         $this->composerCachePath = __DIR__ . '/AutomaticTest';
 
         @\mkdir($this->composerCachePath);
@@ -120,9 +122,6 @@ public function testActivate(): void
             ->andReturn(null);
 
         $localRepositoryMock = $this->mock(WritableRepositoryInterface::class);
-        $localRepositoryMock->shouldReceive('getPackages')
-            ->once()
-            ->andReturn([]);
 
         $repositoryMock = $this->mock(RepositoryManager::class);
         $repositoryMock->shouldReceive('getLocalRepository')