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());

bin/roave-backward-compatibility-check.php

Spacing +5 added lines, -5 removed lines

@@ -31,11 +31,11 @@ 
 use Symfony\Component\Console\Output\ConsoleOutput;
 use function file_exists;
 
-(function () : void {
-    (function () : void {
+(function() : void {
+    (function() : void {
         $autoloaderLocations = [
-            __DIR__ . '/../vendor/autoload.php', // Installed by cloning the project and running `composer install`
-            __DIR__ . '/../../../autoload.php',  // Installed via `composer require`
+            __DIR__.'/../vendor/autoload.php', // Installed by cloning the project and running `composer install`
+            __DIR__.'/../../../autoload.php', // Installed via `composer require`
         ];
 
         foreach ($autoloaderLocations as $autoload) {
@@ -64,7 +64,7 @@ 
         new GetVersionCollectionFromGitRepository(),
         new PickLastMinorVersionFromCollection(),
         new LocateDependenciesViaComposer(
-            function (string $installationPath) use ($composerIo) : Installer {
+            function(string $installationPath) use ($composerIo) : Installer {
                 return Installer::create(
                     $composerIo,
                     (new Factory())->createComposer(

test/unit/Command/AssertBackwardsCompatibleTest.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
 
     public function setUp() : void
     {
-        $repositoryPath = realpath(__DIR__ . '/../../../');
+        $repositoryPath = realpath(__DIR__.'/../../../');
 
         self::assertInternalType('string', $repositoryPath);
 
@@ -274,7 +274,7 @@ 
 
         $this->output->expects(self::any())
             ->method('writeln')
-            ->willReturnCallback(function (string $output) use ($changeToExpect) : void {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
                 self::assertContains($changeToExpect, $output);
             });
     }
@@ -358,7 +358,7 @@ 
 
         $this->getVersions->expects(self::once())
             ->method('fromRepository')
-            ->with(self::callback(function (CheckedOutRepository $checkedOutRepository) : bool {
+            ->with(self::callback(function(CheckedOutRepository $checkedOutRepository) : bool {
                 self::assertEquals($this->sourceRepository, $checkedOutRepository);
                 return true;
             }))

test/unit/Git/GitCheckoutRevisionToTemporaryPathTest.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
     public function testExceptionIsThrownWhenTwoPathsCollide() : void
     {
-        $git              = new GitCheckoutRevisionToTemporaryPath(function () : string {
+        $git              = new GitCheckoutRevisionToTemporaryPath(function() : string {
             return 'foo';
         });
         $sourceRepository = $this->sourceRepository();
@@ -75,7 +75,7 @@ 
 
     private function sourceRepository() : CheckedOutRepository
     {
-        $repositoryPath = realpath(__DIR__ . '/../../..');
+        $repositoryPath = realpath(__DIR__.'/../../..');
 
         self::assertInternalType('string', $repositoryPath);
 

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
             ),

src/Git/PickLastMinorVersionFromCollection.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
                 new ComparisonConstraint(ComparisonConstraint::OPERATOR_LTE, $lastVersion),
                 new ComparisonConstraint(
                     ComparisonConstraint::OPERATOR_GTE,
-                    Version::fromString($lastVersion->getMajor() . '.' . $lastVersion->getMinor() . '.0')
+                    Version::fromString($lastVersion->getMajor().'.'.$lastVersion->getMinor().'.0')
                 )
             ))
             ->sortedAscending();

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()
                     ),
                 ]),