Only run dump autoload no-dev if target is not vendor

Author: BrianHenryIE

src/Pipeline/Autoload/DumpAutoload.php

@@ -111,7 +111,14 @@ protected function generatedMainAutoloader(): void
 //        $generator->setApcu($apcu, $apcuPrefix);
 //        $generator->setPlatformRequirementFilter($this->getPlatformRequirementFilter($input));
         $optimize = true; // $input->getOption('optimize') || $config->get('optimize-autoloader');
-        $generator->setDevMode(false);
+
+        /**
+         * If the target directory is different to the vendor directory, then we do not want to include dev
+         * dependencies, but if it is vendor, then unless composer install was run with --no-dev, we do want them.
+         */
+        if ($this->config->getVendorDirectory() !== $this->config->getTargetDirectory()) {
+            $generator->setDevMode(false);
+        }
 
         $localRepo = new InstalledFilesystemRepository(new JsonFile($this->config->getTargetDirectory() . 'composer/installed.json'));
 
@@ -179,6 +186,10 @@ protected function createInstalledVersionsFiles(): void
 
     protected function prefixNewAutoloader(): void
     {
+        if ($this->config->getVendorDirectory() === $this->config->getTargetDirectory()) {
+            return;
+        }
+
         $this->logger->debug('Prefixing the new Composer autoloader.');
 
         $projectFiles = $this->fileEnumerator->compileFileListForPaths([

tests/Issues/StraussIssue198Test.php

@@ -0,0 +1,52 @@
+<?php
+/**
+ * @see https://github.com/BrianHenryIE/strauss/pull/198
+ */
+
+namespace BrianHenryIE\Strauss\Tests\Issues;
+
+use BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase;
+
+/**
+ * @package BrianHenryIE\Strauss\Tests\Issues
+ * @coversNothing
+ */
+class StraussIssue198Test extends IntegrationTestCase
+{
+    public function test_does_not_remove_vendor_autoload_dev_entries()
+    {
+
+        $composerJsonString = <<<'EOD'
+{
+  "require": {
+    "psr/log": "*"
+  },
+  "require-dev": {
+    "psr/simple-cache": "*"
+  },
+  "extra": {
+    "strauss": {
+      "target_directory": "vendor",
+      "namespace_prefix": "Company\\Project\\"
+    }
+  }
+}
+EOD;
+
+        chdir($this->testsWorkingDir);
+
+        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+
+        exec('composer install');
+        $exitCode = $this->runStrauss($output);
+        assert(0 === $exitCode, $output);
+
+        $php_string = file_get_contents($this->testsWorkingDir . 'vendor/composer/installed.json');
+        $this->assertStringContainsString("Company\\\\Project\\\\Psr\\\\Log\\\\", $php_string);
+        $this->assertStringContainsString("\"Psr\\\\SimpleCache\\\\", $php_string);
+
+        $php_string = file_get_contents($this->testsWorkingDir . 'vendor/composer/autoload_psr4.php');
+        $this->assertStringContainsString("Company\\\\Project\\\\Psr\\\\Log\\\\", $php_string);
+        $this->assertStringContainsString("Psr\\\\SimpleCache\\\\", $php_string);
+    }
+}