Adding tests

Signed-off-by: RJ Garcia <[email protected]>

Author: ragboyjr

src/PhpIncPlugin.php

@@ -49,25 +49,27 @@ public function onPreAutoloadDump(Event $event) {
         $package = $event->getComposer()->getPackage();
 
         $rootDir = dirname($event->getComposer()->getConfig()->get('vendor-dir'));
-        $phpIncConfig = $package->getExtra()['php-inc'] ?? $this->createDefaultConfig();
+        $phpIncConfig = \array_merge($this->createDefaultConfig(), $package->getExtra()['php-inc'] ?? []);
 
         $rootPrefix = $rootDir . '/';
 
         $srcPath = $phpIncConfig['src-path'] ?? null;
+        $srcDir = $srcPath ? $rootDir . '/' . $srcPath : null;
         $testPath = $phpIncConfig['test-path'] ?? null;
+        $testDir = $testPath ? $rootDir . '/' . $testPath : null;
         $matches = $phpIncConfig['matches'] ?? null;
         $devSrcMatches = $phpIncConfig['matches-dev-src'] ?? null;
         $devTestMatches = $phpIncConfig['matches-dev-test'] ?? null;
 
         $additionalAutoloadFiles = [];
         $additionalAutoloadDevFiles = [];
 
-        if ($srcPath) {
+        if ($srcDir && \file_exists($srcDir)) {
             $sourceDir = $rootDir . '/' . $srcPath;
             $additionalAutoloadFiles = $this->matchFiles($rootPrefix, $matches, $sourceDir);
             $additionalAutoloadDevFiles = $this->matchFiles($rootPrefix, $devSrcMatches, $sourceDir);
         }
-        if ($testPath) {
+        if ($testPath && \file_exists($testDir)) {
             $testDir = $rootDir . '/' . $testPath;
             $additionalAutoloadDevFiles = \array_merge($additionalAutoloadDevFiles, $this->matchFiles($rootPrefix, $devTestMatches, $testDir));
         }
@@ -82,7 +84,7 @@ public function onPreAutoloadDump(Event $event) {
         $autoload['files'] = \array_unique(\array_merge($autoload['files'] ?? [], $additionalAutoloadFiles));
         $package->setAutoload($autoload);
         $autoloadDev = $package->getDevAutoload();
-        $autoloadDev['files'] = \array_unique(\array_merge($autoload['files'] ?? [], $additionalAutoloadDevFiles));
+        $autoloadDev['files'] = \array_unique(\array_merge($autoloadDev['files'] ?? [], $additionalAutoloadDevFiles));
         $package->setDevAutoload($autoloadDev);
     }
 
@@ -135,7 +137,6 @@ private function createDefaultConfig(): array {
     ]
   }
 }
-JSON
-        );
+JSON, true);
     }
 }

test/fixtures/src/A.php

@@ -0,0 +1 @@
+<?php

test/fixtures/src/Tests/c.php

@@ -0,0 +1 @@
+<?php

test/fixtures/src/b.php

@@ -0,0 +1 @@
+<?php

test/fixtures/tests/d.php

@@ -0,0 +1,2 @@
+<?php
+

test/php-inc.spec.php

@@ -31,3 +31,42 @@
       expect($matchedFiles)->equal($expectedFiles);
    });
 });
+
+describe('PhpIncPlugin', function() {
+    describe('preAutoLoadDump listener', function() {
+        beforeEach(function() {
+            $this->event = new \Composer\Script\Event(
+                'preAutoLoadDump',
+                (function() {
+                    $composer = new \Composer\Composer();
+                    $composer->setConfig((function() {
+                        $config = new \Composer\Config();
+                        $config->merge(['config' => ['vendor-dir' => __DIR__ . '/fixtures/vendor']]);
+                        return $config;
+                    })());
+                    $composer->setPackage(new \Composer\Package\RootPackage('Test Package', 'v0.1', 'Version 0.1'));
+                    return $composer;
+                })(),
+                new \Composer\IO\NullIO()
+            );
+        });
+
+        it('appends autoload and autoload-dev according to matched files', function() {
+            $plugin = new PhpIncPlugin();
+            $this->event->getComposer()->getPackage()->setAutoload(['files' => ['src/foo.php']]);
+            $this->event->getComposer()->getPackage()->setDevAutoload(['files' => ['tests/bar.php']]);
+            $plugin->onPreAutoloadDump($this->event);
+            $package = $this->event->getComposer()->getPackage();
+            expect($package->getAutoload()['files'])->equal(['src/foo.php', 'src/b.php']);
+            expect($package->getDevAutoload()['files'])->equal(['tests/bar.php', 'src/Tests/c.php', 'tests/d.php']);
+        });
+        it('does not include if dir does not exist', function() {
+            $plugin = new PhpIncPlugin();
+            $this->event->getComposer()->getPackage()->setExtra(['php-inc' => ['test-path' => 'bad']]);
+            $plugin->onPreAutoloadDump($this->event);
+            $package = $this->event->getComposer()->getPackage();
+            expect($package->getAutoload()['files'])->equal(['src/b.php']);
+            expect($package->getDevAutoload()['files'])->equal(['src/Tests/c.php']);
+        });
+    });
+});