Roave / BackwardCompatibilityCheck

test/e2e/Command/AssertBackwardsCompatibleTest.php

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

src/LocateDependencies/LocateDependenciesViaComposer.php

Spacing +5 added lines, -5 removed lines

@@ -44,9 +44,9 @@ 
     public function __invoke(string $installationPath) : SourceLocator
     {
         Assert::that($installationPath)->directory();
-        Assert::that($installationPath . '/composer.json')->file();
+        Assert::that($installationPath.'/composer.json')->file();
 
-        $this->runInDirectory(function () use ($installationPath) : void {
+        $this->runInDirectory(function() use ($installationPath) : void {
             $installer = ($this->makeComposerInstaller)($installationPath);
 
             Assert::that($installer)->isInstanceOf(Installer::class);
@@ -63,7 +63,7 @@ 
             $installer->run();
         }, $installationPath);
 
-        $autoloadStatic = $installationPath . '/vendor/composer/autoload_static.php';
+        $autoloadStatic = $installationPath.'/vendor/composer/autoload_static.php';
 
         Assert::that($autoloadStatic)->file();
 
@@ -106,7 +106,7 @@ 
     {
         $filesMapProperty = $autoloadStatic->getProperty('files');
 
-        if (! $filesMapProperty) {
+        if (!$filesMapProperty) {
             return new AggregateSourceLocator();
         }
 
@@ -121,7 +121,7 @@ 
               ->file();
 
         return new AggregateSourceLocator(array_values(array_map(
-            function (string $path) : SourceLocator {
+            function(string $path) : SourceLocator {
                 $realPath = realpath($path);
 
                 Assert::that($realPath)->string();

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/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;
             }

src/DetectChanges/BCBreak/ClassBased/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();
         }