Roave / BackwardCompatibilityCheck

src/SourceLocator/StubClassSourceLocator.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     protected function createLocatedSource(Identifier $identifier) : ?LocatedSource
     {
-        if (! $identifier->isClass()) {
+        if (!$identifier->isClass()) {
             return null;
         }
 

test/e2e/Command/AssertBackwardsCompatibleTest.php

Indentation +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
             (new Process('git add -A', $this->sourcesRepository))->mustRun();
             (new Process(sprintf('git commit -am "Class sources v%d"', $key + 1), $this->sourcesRepository))->mustRun();
             $this->versions[$key] = trim((new Process('git rev-parse HEAD', $this->sourcesRepository))->mustRun()
-                                                                                                      ->getOutput());
+                                                                                                        ->getOutput());
         }
     }
 

Spacing +16 added lines, -16 removed lines

@@ -105,20 +105,20 @@ 
 
         unlink($this->sourcesRepository);
         mkdir($this->sourcesRepository);
-        mkdir($this->sourcesRepository . '/src');
+        mkdir($this->sourcesRepository.'/src');
 
         self::assertDirectoryExists($this->sourcesRepository);
-        self::assertDirectoryExists($this->sourcesRepository . '/src');
+        self::assertDirectoryExists($this->sourcesRepository.'/src');
 
         (new Process('git init', $this->sourcesRepository))->mustRun();
 
-        file_put_contents($this->sourcesRepository . '/composer.json', self::COMPOSER_MANIFEST);
+        file_put_contents($this->sourcesRepository.'/composer.json', self::COMPOSER_MANIFEST);
 
         (new Process('git add -A', $this->sourcesRepository))->mustRun();
         (new Process('git commit -am "Initial commit with composer manifest"', $this->sourcesRepository))->mustRun();
 
         foreach (self::CLASS_VERSIONS as $key => $classCode) {
-            file_put_contents($this->sourcesRepository . '/src/TheClass.php', $classCode);
+            file_put_contents($this->sourcesRepository.'/src/TheClass.php', $classCode);
 
             (new Process('git add -A', $this->sourcesRepository))->mustRun();
             (new Process(sprintf('git commit -am "Class sources v%d"', $key + 1), $this->sourcesRepository))->mustRun();
@@ -143,9 +143,9 @@ 
     {
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--from=' . $this->versions[0],
-                '--to=' . $this->versions[1],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--from='.$this->versions[0],
+                '--to='.$this->versions[1],
             ],
             $this->sourcesRepository
         );
@@ -165,7 +165,7 @@ 
     public function testWillNotRunWithoutTagsNorSpecifiedVersions() : void
     {
         $check = new Process(
-            __DIR__ . '/../../../bin/roave-backward-compatibility-check',
+            __DIR__.'/../../../bin/roave-backward-compatibility-check',
             $this->sourcesRepository
         );
 
@@ -180,9 +180,9 @@ 
     {
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--from=' . $this->versions[0],
-                '--to=' . $this->versions[3],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--from='.$this->versions[0],
+                '--to='.$this->versions[3],
             ],
             $this->sourcesRepository
         );
@@ -200,8 +200,8 @@ 
 
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--to=' . $this->versions[2],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--to='.$this->versions[2],
             ],
             $this->sourcesRepository
         );
@@ -229,8 +229,8 @@ 
 
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--to=' . $this->versions[2],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--to='.$this->versions[2],
             ],
             $this->sourcesRepository
         );
@@ -268,7 +268,7 @@ 
                 'tag',
                 $tagName,
                 '-m',
-                'A tag for version ' . $version,
+                'A tag for version '.$version,
             ],
             $this->sourcesRepository
         ))->mustRun();

test/unit/ChangeTest.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, false);
-        self::assertSame('     ADDED: ' . $changeText, (string) $change);
+        self::assertSame('     ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -29,7 +29,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, true);
-        self::assertSame('[BC] ADDED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -40,7 +40,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, false);
-        self::assertSame('     CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('     CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -50,7 +50,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, true);
-        self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -61,7 +61,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, false);
-        self::assertSame('     REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('     REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());
@@ -72,7 +72,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, true);
-        self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());
@@ -83,7 +83,7 @@ 
     {
         $failure = new Exception('changeText');
         $change  = Change::skippedDueToFailure($failure);
-        self::assertSame('[BC] SKIPPED: ' . $failure->getMessage(), (string) $change);
+        self::assertSame('[BC] SKIPPED: '.$failure->getMessage(), (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());

test/unit/LocateDependencies/LocateDependenciesViaComposerTest.php

Spacing +11 added lines, -11 removed lines

@@ -53,7 +53,7 @@ 
         $this->originalCwd       = $originalCwd;
         $this->composerInstaller = $this->createMock(Installer::class);
         $this->astLocator        = (new BetterReflection())->astLocator();
-        $this->makeInstaller     = function (string $installationPath) : Installer {
+        $this->makeInstaller     = function(string $installationPath) : Installer {
             self::assertSame($this->expectedInstallatonPath, $installationPath);
 
             return $this->composerInstaller;
@@ -102,13 +102,13 @@ 
 
     public function testWillLocateDependencies() : void
     {
-        $this->expectedInstallatonPath = $this->realpath(__DIR__ . '/../../asset/composer-installation-structure');
+        $this->expectedInstallatonPath = $this->realpath(__DIR__.'/../../asset/composer-installation-structure');
 
         $this
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () : void {
+            ->willReturnCallback(function() : void {
                 self::assertSame($this->expectedInstallatonPath, getcwd());
             });
 
@@ -128,8 +128,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/AClassName.php'),
-                    'B\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/BClassName.php'),
+                    'A\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure/AClassName.php'),
+                    'B\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure/BClassName.php'),
                 ],
                 $this->astLocator
             ),
@@ -138,11 +138,11 @@ 
         self::assertEquals(
             new AggregateSourceLocator([
                 new SingleFileSourceLocator(
-                    $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-1.php'),
+                    $this->realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-1.php'),
                     $this->astLocator
                 ),
                 new SingleFileSourceLocator(
-                    $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-2.php'),
+                    $this->realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-2.php'),
                     $this->astLocator
                 ),
             ]),
@@ -154,13 +154,13 @@ 
 
     public function testWillLocateDependenciesEvenWithoutAutoloadFiles() : void
     {
-        $this->expectedInstallatonPath = $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files');
+        $this->expectedInstallatonPath = $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files');
 
         $this
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () : void {
+            ->willReturnCallback(function() : void {
                 self::assertSame($this->expectedInstallatonPath, getcwd());
             });
 
@@ -180,8 +180,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
-                    'B\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
+                    'A\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
+                    'B\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
                 ],
                 $this->astLocator
             ),

test/unit/LocateSources/LocateSourcesViaComposerJsonTest.php

Spacing +13 added lines, -13 removed lines

@@ -45,37 +45,37 @@
     {
         return [
             'empty composer definition' => [
-                __DIR__ . '/../../asset/located-sources/empty',
+                __DIR__.'/../../asset/located-sources/empty',
                 new AggregateSourceLocator([
                     new DirectoriesSourceLocator([], $this->astLocator()),
                 ]),
             ],
             'composer definition with everything' => [
-                __DIR__ . '/../../asset/located-sources/composer-definition-with-everything',
+                __DIR__.'/../../asset/located-sources/composer-definition-with-everything',
                 new AggregateSourceLocator([
                     new DirectoriesSourceLocator(
                         [
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/foo0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/bar4'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_1'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_2'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_3'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap1'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/foo0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/bar4'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_1'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_2'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_3'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap1'),
                         ],
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap2/file.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap2/file.php'),
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/files/foo.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/files/foo.php'),
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/files/bar.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/files/bar.php'),
                         $this->astLocator()
                     ),
                 ]),

src/LocateSources/LocateSourcesViaComposerJson.php

Indentation +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
         $composerJsonPath = $realInstallationPath . '/composer.json';
 
         Assert::that($composerJsonPath)
-              ->file()
-              ->readable();
+                ->file()
+                ->readable();
 
         $composerDefinitionString = file_get_contents($composerJsonPath);
 

Spacing +8 added lines, -8 removed lines

@@ -40,7 +40,7 @@ 
 
         Assert::that($realInstallationPath)->string();
 
-        $composerJsonPath = $realInstallationPath . '/composer.json';
+        $composerJsonPath = $realInstallationPath.'/composer.json';
 
         Assert::that($composerJsonPath)
               ->file()
@@ -75,7 +75,7 @@ 
                 ),
             ],
             array_map(
-                function (string $path) : SourceLocator {
+                function(string $path) : SourceLocator {
                     return new SingleFileSourceLocator($path, $this->astLocator);
                 },
                 array_map(
@@ -84,7 +84,7 @@ 
                 )
             ),
             array_map(
-                function (string $path) : SourceLocator {
+                function(string $path) : SourceLocator {
                     return new SingleFileSourceLocator($path, $this->astLocator);
                 },
                 array_map(
@@ -104,7 +104,7 @@ 
     {
         return array_values(array_filter(
             $autoloadDefinition['classmap'] ?? [],
-            function (string $path) use ($installationPath) : bool {
+            function(string $path) use ($installationPath) : bool {
                 $filePath = ($this->prependInstallationPath($installationPath))($path);
 
                 return is_dir($filePath);
@@ -121,7 +121,7 @@ 
     {
         return array_values(array_filter(
             $autoloadDefinition['classmap'] ?? [],
-            function (string $path) use ($installationPath) : bool {
+            function(string $path) use ($installationPath) : bool {
                 $filePath = ($this->prependInstallationPath($installationPath))($path);
 
                 return is_file($filePath);
@@ -168,12 +168,12 @@ 
     /** @return callable (string $path) : string */
     private function prependInstallationPath(string $installationPath) : callable
     {
-        return static function (string $path) use ($installationPath) : string {
+        return static function(string $path) use ($installationPath) : string {
             if (strpos($path, './') === 0) {
-                return $installationPath . '/' . substr($path, 2);
+                return $installationPath.'/'.substr($path, 2);
             }
 
-            return $installationPath . '/' . ltrim($path, '/');
+            return $installationPath.'/'.ltrim($path, '/');
         };
     }
 

src/Git/GitCheckoutRevisionToTemporaryPath.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     private function generateTemporaryPathFor(Revision $revision) : string
     {
         $uniquePathGenerator = $this->uniquenessFunction;
-        $checkoutDirectory   = sys_get_temp_dir() . '/api-compare-' . $uniquePathGenerator($revision . '_');
+        $checkoutDirectory   = sys_get_temp_dir().'/api-compare-'.$uniquePathGenerator($revision.'_');
 
         if (file_exists($checkoutDirectory)) {
             throw new RuntimeException(sprintf(

src/DetectChanges/BCBreak/InterfaceBased/AncestorRemoved.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             array_diff($fromClass->getInterfaceNames(), $toClass->getInterfaceNames())
         );
 
-        if (! $removedAncestors) {
+        if (!$removedAncestors) {
             return Changes::empty();
         }
 

src/Changes.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
         $instance = new self();
 
         $instance->bufferedChanges   = [];
-        $instance->unBufferedChanges = (function () use ($other) : Generator {
+        $instance->unBufferedChanges = (function() use ($other) : Generator {
             foreach ($this as $change) {
                 yield $change;
             }