Make the build faster (#96)

Author: theofidry

Makefile

@@ -20,10 +20,17 @@ build: src vendor vendor-bin/box/vendor
 	rm -f bin/php-scoper.phar
 	rm -rf build
 
+	# Remove unnecessary packages
 	composer install --no-dev --prefer-dist
 
+	# Prefixes the code to be bundled
 	php -d zend.enable_gc=0 bin/php-scoper add-prefix --force
-	composer dump-autoload -d build  --classmap-authoritative
+
+	# Re-dump the loader to account for the prefixing
+	# and optimize the loader
+	composer dump-autoload -d build --classmap-authoritative
+
+	# Build the PHAR
 	php -d zend.enable_gc=0 -d phar.readonly=0 $(BOX) build $(CONFIG)
 
 	# Install back all the dependencies

box.json.dist

@@ -3,10 +3,7 @@
     "chmod": "0755",
     "main": "bin/php-scoper",
     "output": "bin/php-scoper.phar",
-    "directories": [
-        "src",
-        "tests"
-    ],
+    "directories": ["src"],
     "finder": [
         {
             "notName": "/LICENSE|.*\\.md|.*\\.dist|Makefile|composer\\.json|composer\\.lock/",

scoper.inc.php

@@ -12,13 +12,35 @@
  * file that was distributed with this source code.
  */
 
+use Symfony\Component\Finder\Finder;
+
 return [
     'global_namespace_whitelist' => [
         'AppKernel',
         function (string $className): bool {
             return 'PHPUnit' === substr($className, 0, 6);
         },
     ],
+    'finders' => [
+        Finder::create()->files()->in('src'),
+        Finder::create()
+            ->files()
+            ->ignoreVCS(true)
+            ->notName('/LICENSE|.*\\.md|.*\\.dist|Makefile|composer\\.json|composer\\.lock/')
+            ->exclude([
+                'doc',
+                'test',
+                'test_old',
+                'tests',
+                'Tests',
+                'vendor-bin',
+            ])
+            ->in('vendor'),
+        Finder::create()->append([
+            'bin/php-scoper',
+            'composer.json',
+        ]),
+    ],
     'patchers' => [
         function (string $filePath, string $prefix, string $content): string {
             //

src/Console/Command/AddPrefixCommand.php

@@ -272,7 +272,7 @@ private function retrieveConfig(InputInterface $input, OutputStyle $io): Configu
             if (false === file_exists($configFile)) {
                 $io->writeln(
                     sprintf(
-                        'Patch file "%s" not found. Skipping.',
+                        'Config file "%s" not found. Skipping.',
                         $configFile
                     ),
                     OutputStyle::VERBOSITY_DEBUG

tests/Console/Command/AddPrefixCommandIntegrationTest.php

@@ -65,6 +65,8 @@ protected function setUp()
 
         $this->cwd = getcwd();
         $this->tmp = make_tmp_dir('scoper', __CLASS__);
+
+        chdir($this->tmp);
     }
 
     /**

tests/Console/Command/AddPrefixCommandTest.php

@@ -79,6 +79,8 @@ protected function setUp()
         $this->handleProphecy = $this->prophesize(HandleAddPrefix::class);
 
         $this->appTester = $this->createAppTester();
+
+        chdir($this->tmp);
     }
 
     public function test_get_help_menu()
@@ -366,7 +368,7 @@ public function test_scope_the_current_working_directory_if_no_path_given()
             ->__invoke(
                 'MyPrefix',
                 [
-                    $this->cwd,
+                    $this->tmp,
                 ],
                 $this->tmp,
                 Argument::type('array'),
@@ -414,8 +416,8 @@ public function test_relative_paths_are_relative_to_the_current_working_director
                 'MyPrefix',
                 [
                     escape_path('/path/to/dir1'),
-                    escape_path($this->cwd.'/relative-path/to/dir2'),
-                    escape_path($this->cwd.'/relative-path/to/file'),
+                    escape_path($this->tmp.'/relative-path/to/dir2'),
+                    escape_path($this->tmp.'/relative-path/to/file'),
                 ],
                 $this->tmp,
                 Argument::type('array'),
@@ -702,7 +704,7 @@ public function test_throws_an_error_when_passing_a_non_existent_path_file()
 
             $this->fail('Expected exception to be thrown.');
         } catch (RuntimeException $exception) {
-            $patchFile = escape_path($this->cwd.'/unknown');
+            $patchFile = escape_path($this->tmp.'/unknown');
 
             $this->assertSame(
                 "Could not find the file \"$patchFile\".",