gravityview / GravityView

vendor/brianhenryie/strauss/tests/Unit/LicenserTest.php

Indentation +142 added lines, -142 removed lines

@@ -22,80 +22,80 @@ 
 {
 
 
-    /**
-     * @covers ::findLicenseFiles
-     *
-     */
-    public function testFindLicenceFilesPathsAreRelative()
-    {
-        $config = $this->createStub(StraussConfig::class);
-        $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
+	/**
+	 * @covers ::findLicenseFiles
+	 *
+	 */
+	public function testFindLicenceFilesPathsAreRelative()
+	{
+		$config = $this->createStub(StraussConfig::class);
+		$workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
-        $dependencies = array();
+		$dependencies = array();
 
-        $dependency = $this->createStub(ComposerPackage::class);
-        $dependency->method('getPath')->willReturn('developer-name/project-name/');
-        $dependencies[] = $dependency;
+		$dependency = $this->createStub(ComposerPackage::class);
+		$dependency->method('getPath')->willReturn('developer-name/project-name/');
+		$dependencies[] = $dependency;
 
-        $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
+		$sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
 
-        $finder = $this->createStub(Finder::class);
+		$finder = $this->createStub(Finder::class);
 
-        $file = $this->createStub(\SplFileInfo::class);
-        $file->method('getPathname')
-            ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
+		$file = $this->createStub(\SplFileInfo::class);
+		$file->method('getPathname')
+			->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
 
-        $finderArrayIterator = new ArrayIterator(array(
-            $file
-        ));
+		$finderArrayIterator = new ArrayIterator(array(
+			$file
+		));
 
-        $finder->method('getIterator')->willReturn($finderArrayIterator);
+		$finder->method('getIterator')->willReturn($finderArrayIterator);
 
-        // Make the rest fluent.
-        $callableConstraintNotGetIterator = function ($methodName) {
-            return 'getIterator' !== $methodName;
-        };
-        $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
+		// Make the rest fluent.
+		$callableConstraintNotGetIterator = function ($methodName) {
+			return 'getIterator' !== $methodName;
+		};
+		$finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
 
-        $sut->findLicenseFiles($finder);
+		$sut->findLicenseFiles($finder);
 
-        $result = $sut->getDiscoveredLicenseFiles();
+		$result = $sut->getDiscoveredLicenseFiles();
 
-        // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
-        $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
-    }
+		// Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
+		$this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
+	}
 
-    /**
-     * Licence files should be found regardless of case and regardless of British/US-English spelling.
-     *
-     * @see https://www.phpliveregex.com/p/A5y
-     */
+	/**
+	 * Licence files should be found regardless of case and regardless of British/US-English spelling.
+	 *
+	 * @see https://www.phpliveregex.com/p/A5y
+	 */
 
-    // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/
+	// https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/
 
 
-    /**
-     * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php
-     *
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testAppendHeaderCommentInformationNoHeader()
-    {
+	/**
+	 * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php
+	 *
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testAppendHeaderCommentInformationNoHeader()
+	{
 
-        $author = 'BrianHenryIE';
+		$author = 'BrianHenryIE';
 
-        $config = $this->createStub(StraussConfig::class);
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        //     "license": "proprietary",
+		//     "license": "proprietary",
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * @license proprietary
@@ -107,22 +107,22 @@ 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testWithLicenceAlreadyInHeader(): void
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testWithLicenceAlreadyInHeader(): void
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
 /**
  * Handles dismissing admin notices.
@@ -137,7 +137,7 @@ 
 namespace Yeah;
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
 /**
  * Handles dismissing admin notices.
@@ -155,25 +155,25 @@ 
 namespace Yeah;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
 
-    /**
-     * Shouldn't matter too much but y'know regexes.
-     *
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testWithTwoCommentsBeforeFirstCode()
-    {
+	/**
+	 * Shouldn't matter too much but y'know regexes.
+	 *
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testWithTwoCommentsBeforeFirstCode()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -194,7 +194,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -219,22 +219,22 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testUnusualHeaderCommentStyle()
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testUnusualHeaderCommentStyle()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 /*******************************************************************************
 * FPDF                                                                         *
@@ -247,7 +247,7 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /*******************************************************************************
 * FPDF                                                                         *
@@ -264,22 +264,22 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testCommentWithLicenseWord()
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testCommentWithLicenseWord()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 
 /**
@@ -297,7 +297,7 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 
 /**
@@ -318,27 +318,27 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * This was matching the "no header comment" regex.
-     *
-     * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect.
-     * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw
-     *
-     * Seems files loaded are treated different to strings passed.
-     */
-    public function testIncorrectlyMatching()
-    {
+	/**
+	 * This was matching the "no header comment" regex.
+	 *
+	 * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect.
+	 * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw
+	 *
+	 * Seems files loaded are treated different to strings passed.
+	 */
+	public function testIncorrectlyMatching()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $fileContents = <<<'EOD'
+		$fileContents = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -361,12 +361,12 @@ 
 }
 EOD;
 
-        // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $fileContents);
-        $given = file_get_contents($tmpfname);
+		// Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $fileContents);
+		$given = file_get_contents($tmpfname);
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -393,22 +393,22 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * The licence was being inserted after every `<?php` in the file.
-     */
-    public function testLicenseDetailsOnlyInsertedOncePerFile()
-    {
+	/**
+	 * The licence was being inserted after every `<?php` in the file.
+	 */
+	public function testLicenseDetailsOnlyInsertedOncePerFile()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $fileContents = <<<'EOD'
+		$fileContents = <<<'EOD'
 <?php
 
 ?>
@@ -418,7 +418,7 @@ 
 ?>
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * @license MIT
@@ -435,8 +435,8 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 }

Spacing +47 added lines, -47 removed lines

@@ -28,41 +28,41 @@ 
      */
     public function testFindLicenceFilesPathsAreRelative()
     {
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
         $dependencies = array();
 
-        $dependency = $this->createStub(ComposerPackage::class);
-        $dependency->method('getPath')->willReturn('developer-name/project-name/');
-        $dependencies[] = $dependency;
+        $dependency = $this->createStub( ComposerPackage::class );
+        $dependency->method( 'getPath' )->willReturn( 'developer-name/project-name/' );
+        $dependencies[ ] = $dependency;
 
-        $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
+        $sut = new Licenser( $config, $workingDir, $dependencies, 'BrianHenryIE' );
 
-        $finder = $this->createStub(Finder::class);
+        $finder = $this->createStub( Finder::class );
 
-        $file = $this->createStub(\SplFileInfo::class);
-        $file->method('getPathname')
-            ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
+        $file = $this->createStub( \SplFileInfo::class );
+        $file->method( 'getPathname' )
+            ->willReturn( __DIR__ . '/vendor/developer-name/project-name/license.md' );
 
-        $finderArrayIterator = new ArrayIterator(array(
+        $finderArrayIterator = new ArrayIterator( array(
             $file
-        ));
+        ) );
 
-        $finder->method('getIterator')->willReturn($finderArrayIterator);
+        $finder->method( 'getIterator' )->willReturn( $finderArrayIterator );
 
         // Make the rest fluent.
-        $callableConstraintNotGetIterator = function ($methodName) {
+        $callableConstraintNotGetIterator = function( $methodName ) {
             return 'getIterator' !== $methodName;
         };
-        $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
+        $finder->method( new Callback( $callableConstraintNotGetIterator ) )->willReturn( $finder );
 
-        $sut->findLicenseFiles($finder);
+        $sut->findLicenseFiles( $finder );
 
         $result = $sut->getDiscoveredLicenseFiles();
 
         // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
-        $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
+        $this->assertStringContainsString( 'developer-name/project-name/license.md', $result[ 0 ] );
     }
 
     /**
@@ -84,8 +84,8 @@ 
 
         $author = 'BrianHenryIE';
 
-        $config = $this->createStub(StraussConfig::class);
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $config = $this->createStub( StraussConfig::class );
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -107,9 +107,9 @@ 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'proprietary' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -118,9 +118,9 @@ 
     public function testWithLicenceAlreadyInHeader(): void
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
@@ -155,9 +155,9 @@ 
 namespace Yeah;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'GPL-2.0-or-later' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
 
@@ -169,9 +169,9 @@ 
     public function testWithTwoCommentsBeforeFirstCode()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -219,9 +219,9 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -230,9 +230,9 @@ 
     public function testUnusualHeaderCommentStyle()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -264,9 +264,9 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'proprietary' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -275,9 +275,9 @@ 
     public function testCommentWithLicenseWord()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -318,9 +318,9 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -334,9 +334,9 @@ 
     public function testIncorrectlyMatching()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $fileContents = <<<'EOD'
 <?php
@@ -362,9 +362,9 @@ 
 EOD;
 
         // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $fileContents);
-        $given = file_get_contents($tmpfname);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $fileContents );
+        $given = file_get_contents( $tmpfname );
 
         $expected = <<<'EOD'
 <?php
@@ -393,9 +393,9 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -404,9 +404,9 @@ 
     public function testLicenseDetailsOnlyInsertedOncePerFile()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $fileContents = <<<'EOD'
 <?php
@@ -435,8 +435,8 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $fileContents, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 }

Braces +8 added lines, -16 removed lines

@@ -18,16 +18,14 @@ 
  * @package BrianHenryIE\Strauss\Tests\Unit
  * @coversDefaultClass \BrianHenryIE\Strauss\Licenser
  */
-class LicenserTest extends TestCase
-{
+class LicenserTest extends TestCase {
 
 
     /**
      * @covers ::findLicenseFiles
      *
      */
-    public function testFindLicenceFilesPathsAreRelative()
-    {
+    public function testFindLicenceFilesPathsAreRelative() {
         $config = $this->createStub(StraussConfig::class);
         $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
@@ -79,8 +77,7 @@ 
      *
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testAppendHeaderCommentInformationNoHeader()
-    {
+    public function testAppendHeaderCommentInformationNoHeader() {
 
         $author = 'BrianHenryIE';
 
@@ -166,8 +163,7 @@ 
      *
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testWithTwoCommentsBeforeFirstCode()
-    {
+    public function testWithTwoCommentsBeforeFirstCode() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -227,8 +223,7 @@ 
     /**
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testUnusualHeaderCommentStyle()
-    {
+    public function testUnusualHeaderCommentStyle() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -272,8 +267,7 @@ 
     /**
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testCommentWithLicenseWord()
-    {
+    public function testCommentWithLicenseWord() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -331,8 +325,7 @@ 
      *
      * Seems files loaded are treated different to strings passed.
      */
-    public function testIncorrectlyMatching()
-    {
+    public function testIncorrectlyMatching() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -401,8 +394,7 @@ 
     /**
      * The licence was being inserted after every `<?php` in the file.
      */
-    public function testLicenseDetailsOnlyInsertedOncePerFile()
-    {
+    public function testLicenseDetailsOnlyInsertedOncePerFile() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';

Upper-Lower-Casing +14 added lines, -14 removed lines

@@ -91,7 +91,7 @@ 
 <?php
 
 namespace net\authorize\api\contract\v1;
-EOD;
+eod;
 
         //     "license": "proprietary",
 
@@ -105,7 +105,7 @@ 
  */
 
 namespace net\authorize\api\contract\v1;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
@@ -135,7 +135,7 @@ 
  */
 
 namespace Yeah;
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
@@ -153,7 +153,7 @@ 
  */
 
 namespace Yeah;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
 
@@ -192,7 +192,7 @@ 
  * Exit if called directly.
  */
 if ( ! defined( 'WPINC' ) ) {
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -216,7 +216,7 @@ 
  * Exit if called directly.
  */
 if ( ! defined( 'WPINC' ) ) {
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
@@ -245,7 +245,7 @@ 
 *******************************************************************************/
 
 define('FPDF_VERSION','1.82');
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -262,7 +262,7 @@ 
 */
 
 define('FPDF_VERSION','1.82');
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
@@ -295,7 +295,7 @@ 
  */
 
 namespace Your_Domain\Assert;
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -316,7 +316,7 @@ 
  */
 
 namespace Your_Domain\Assert;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
@@ -359,7 +359,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 	die;
 }
-EOD;
+eod;
 
         // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
         $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
@@ -390,7 +390,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 	die;
 }
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
@@ -416,7 +416,7 @@ 
 <?php
 
 ?>
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -432,7 +432,7 @@ 
 <?php
 
 ?>
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');

vendor/brianhenryie/strauss/tests/Unit/Console/Commands/ComposeTest.php

Indentation +190 added lines, -190 removed lines

@@ -11,194 +11,194 @@
 class ComposeTest extends TestCase
 {
 
-    /**
-     * When composer.json is absent, instead of failing with:
-     * "failed to open stream: No such file or directory"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_fails_gracefully_when_composer_json_absent(): void
-    {
-        chdir(sys_get_temp_dir());
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-             ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When json_decode fails, instead of
-     * "Trying to get property 'extra' of non-object"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_malformed_json_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra is absent, instead of
-     * "Undefined property: stdClass::$extra"
-     * a better message should be written to the OutputInterface.
-     *
-     * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'.
-     *
-     */
-    public function test_it_handles_absent_extra_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-
-    /**
-     * When composer.json->extra is not an object, instead of
-     * "Trying to get property 'mozart' of non-object"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_malformed_extra_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra->mozart is absent, instead of
-     * "Undefined property: stdClass::$mozart"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_absent_mozart_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra->mozart is malformed, instead of
-     * "Undefined property: stdClass::$mozart"
-     * a better message should be written to the OutputInterface.
-     *
-     * is_object() added.
-     *
-     * @test
-     */
-    public function it_handles_malformed_mozart_config__with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
+	/**
+	 * When composer.json is absent, instead of failing with:
+	 * "failed to open stream: No such file or directory"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_fails_gracefully_when_composer_json_absent(): void
+	{
+		chdir(sys_get_temp_dir());
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+			 ->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When json_decode fails, instead of
+	 * "Trying to get property 'extra' of non-object"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_json_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra is absent, instead of
+	 * "Undefined property: stdClass::$extra"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'.
+	 *
+	 */
+	public function test_it_handles_absent_extra_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+
+	/**
+	 * When composer.json->extra is not an object, instead of
+	 * "Trying to get property 'mozart' of non-object"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_extra_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra->mozart is absent, instead of
+	 * "Undefined property: stdClass::$mozart"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_absent_mozart_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra->mozart is malformed, instead of
+	 * "Undefined property: stdClass::$mozart"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * is_object() added.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_mozart_config__with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
 }

Spacing +52 added lines, -52 removed lines

@@ -20,20 +20,20 @@ 
      */
     public function it_fails_gracefully_when_composer_json_absent(): void
     {
-        chdir(sys_get_temp_dir());
+        chdir( sys_get_temp_dir() );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-             ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+             ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -50,22 +50,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -83,22 +83,22 @@ 
 
         $badComposerJson = '{ }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -116,22 +116,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -148,22 +148,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -182,22 +182,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }

Braces +7 added lines, -14 removed lines

@@ -8,8 +8,7 @@ 
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class ComposeTest extends TestCase
-{
+class ComposeTest extends TestCase {
 
     /**
      * When composer.json is absent, instead of failing with:
@@ -29,8 +28,7 @@ 
              ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -61,8 +59,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -94,8 +91,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -127,8 +123,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -159,8 +154,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -193,8 +187,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);

vendor/brianhenryie/strauss/tests/Unit/Console/ApplicationTest.php

Indentation +13 added lines, -13 removed lines

@@ -9,22 +9,22 @@
 class ApplicationTest extends TestCase
 {
 
-    /**
-     * Test the Symfony\Component\Console\Application instance contains the Compose command.
-     */
-    public function testInstantiation()
-    {
+	/**
+	 * Test the Symfony\Component\Console\Application instance contains the Compose command.
+	 */
+	public function testInstantiation()
+	{
 
-        $version = '1.0.0';
+		$version = '1.0.0';
 
-        $sut = new Application($version);
+		$sut = new Application($version);
 
-        $commands = $sut->all();
+		$commands = $sut->all();
 
-        $containsComposeCommand = array_reduce($commands, function ($carry, $item) {
-            return $carry || $item instanceof Compose;
-        }, false);
+		$containsComposeCommand = array_reduce($commands, function ($carry, $item) {
+			return $carry || $item instanceof Compose;
+		}, false);
 
-        $this->assertTrue($containsComposeCommand);
-    }
+		$this->assertTrue($containsComposeCommand);
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -17,14 +17,14 @@
 
         $version = '1.0.0';
 
-        $sut = new Application($version);
+        $sut = new Application( $version );
 
         $commands = $sut->all();
 
-        $containsComposeCommand = array_reduce($commands, function ($carry, $item) {
+        $containsComposeCommand = array_reduce( $commands, function( $carry, $item ) {
             return $carry || $item instanceof Compose;
-        }, false);
+        }, false );
 
-        $this->assertTrue($containsComposeCommand);
+        $this->assertTrue( $containsComposeCommand );
     }
 }

Braces +2 added lines, -4 removed lines

@@ -6,14 +6,12 @@
 use BrianHenryIE\Strauss\Console\Commands\Compose;
 use PHPUnit\Framework\TestCase;
 
-class ApplicationTest extends TestCase
-{
+class ApplicationTest extends TestCase {
 
     /**
      * Test the Symfony\Component\Console\Application instance contains the Compose command.
      */
-    public function testInstantiation()
-    {
+    public function testInstantiation() {
 
         $version = '1.0.0';
 

vendor/brianhenryie/strauss/tests/Integration/Util/IntegrationTestCase.php

Indentation +40 added lines, -40 removed lines

@@ -18,54 +18,54 @@
  */
 class IntegrationTestCase extends TestCase
 {
-    protected $testsWorkingDir;
+	protected $testsWorkingDir;
 
-    public function setUp(): void
-    {
-        parent::setUp();
+	public function setUp(): void
+	{
+		parent::setUp();
 
-        $this->testsWorkingDir =  rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR
-            . 'strausstestdir' . DIRECTORY_SEPARATOR;
+		$this->testsWorkingDir =  rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR
+			. 'strausstestdir' . DIRECTORY_SEPARATOR;
 
-        if (file_exists($this->testsWorkingDir)) {
-            $this->deleteDir($this->testsWorkingDir);
-        }
+		if (file_exists($this->testsWorkingDir)) {
+			$this->deleteDir($this->testsWorkingDir);
+		}
 
-        @mkdir($this->testsWorkingDir);
-    }
+		@mkdir($this->testsWorkingDir);
+	}
 
 
-    /**
-     * Delete $this->testsWorkingDir after each test.
-     *
-     * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it
-     */
-    public function tearDown(): void
-    {
-        parent::tearDown();
+	/**
+	 * Delete $this->testsWorkingDir after each test.
+	 *
+	 * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it
+	 */
+	public function tearDown(): void
+	{
+		parent::tearDown();
 
-        $dir = $this->testsWorkingDir;
+		$dir = $this->testsWorkingDir;
 
-        $this->deleteDir($dir);
-    }
+		$this->deleteDir($dir);
+	}
 
-    protected function deleteDir($dir)
-    {
-        if (!file_exists($dir)) {
-            return;
-        }
+	protected function deleteDir($dir)
+	{
+		if (!file_exists($dir)) {
+			return;
+		}
 
-        $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS);
-        $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST);
-        foreach ($files as $file) {
-            if (is_link($file)) {
-                unlink($file);
-            } elseif ($file->isDir()) {
-                rmdir($file->getRealPath());
-            } else {
-                unlink($file->getRealPath());
-            }
-        }
-        rmdir($dir);
-    }
+		$it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS);
+		$files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST);
+		foreach ($files as $file) {
+			if (is_link($file)) {
+				unlink($file);
+			} elseif ($file->isDir()) {
+				rmdir($file->getRealPath());
+			} else {
+				unlink($file->getRealPath());
+			}
+		}
+		rmdir($dir);
+	}
 }

Spacing +16 added lines, -16 removed lines

@@ -24,14 +24,14 @@ 
     {
         parent::setUp();
 
-        $this->testsWorkingDir =  rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR
+        $this->testsWorkingDir = rtrim( sys_get_temp_dir(), DIRECTORY_SEPARATOR ) . DIRECTORY_SEPARATOR
             . 'strausstestdir' . DIRECTORY_SEPARATOR;
 
-        if (file_exists($this->testsWorkingDir)) {
-            $this->deleteDir($this->testsWorkingDir);
+        if ( file_exists( $this->testsWorkingDir ) ) {
+            $this->deleteDir( $this->testsWorkingDir );
         }
 
-        @mkdir($this->testsWorkingDir);
+        @mkdir( $this->testsWorkingDir );
     }
 
 
@@ -46,26 +46,26 @@ 
 
         $dir = $this->testsWorkingDir;
 
-        $this->deleteDir($dir);
+        $this->deleteDir( $dir );
     }
 
-    protected function deleteDir($dir)
+    protected function deleteDir( $dir )
     {
-        if (!file_exists($dir)) {
+        if ( ! file_exists( $dir ) ) {
             return;
         }
 
-        $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS);
-        $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST);
-        foreach ($files as $file) {
-            if (is_link($file)) {
-                unlink($file);
-            } elseif ($file->isDir()) {
-                rmdir($file->getRealPath());
+        $it = new RecursiveDirectoryIterator( $dir, RecursiveDirectoryIterator::SKIP_DOTS );
+        $files = new RecursiveIteratorIterator( $it, RecursiveIteratorIterator::CHILD_FIRST );
+        foreach ( $files as $file ) {
+            if ( is_link( $file ) ) {
+                unlink( $file );
+            } elseif ( $file->isDir() ) {
+                rmdir( $file->getRealPath() );
             } else {
-                unlink($file->getRealPath());
+                unlink( $file->getRealPath() );
             }
         }
-        rmdir($dir);
+        rmdir( $dir );
     }
 }

Braces +2 added lines, -4 removed lines

@@ -16,8 +16,7 @@ 
  * @package BrianHenryIE\Strauss\Tests\Integration\Util
  * @coversNothing
  */
-class IntegrationTestCase extends TestCase
-{
+class IntegrationTestCase extends TestCase {
     protected $testsWorkingDir;
 
     public function setUp(): void
@@ -49,8 +48,7 @@ 
         $this->deleteDir($dir);
     }
 
-    protected function deleteDir($dir)
-    {
+    protected function deleteDir($dir) {
         if (!file_exists($dir)) {
             return;
         }

vendor/brianhenryie/strauss/tests/Integration/CopierIntegrationTest.php

Indentation +194 added lines, -194 removed lines

@@ -18,10 +18,10 @@ 
 class CopierIntegrationTest extends IntegrationTestCase
 {
 
-    public function testsPrepareTarget()
-    {
+	public function testsPrepareTarget()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -37,50 +37,50 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+		$projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+		$dependencies = array_map(function ($element) {
+			$dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
+			return new ComposerPackage($dir);
+		}, $projectComposerPackage->getRequiresNames());
 
-        $workingDir = $this->testsWorkingDir;
-        $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
+		$workingDir = $this->testsWorkingDir;
+		$relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
 
-        $config = $this->createStub(StraussConfig::class);
+		$config = $this->createStub(StraussConfig::class);
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
-        $fileEnumerator->compileFileList();
-        $filepaths = $fileEnumerator->getAllFilesAndDependencyList();
+		$fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+		$fileEnumerator->compileFileList();
+		$filepaths = $fileEnumerator->getAllFilesAndDependencyList();
 
-        $copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
+		$copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
 
-        $file = 'ContainerAwareTrait.php';
-        $relativePath = 'league/container/src/';
-        $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath;
-        $targetFile = $targetPath . $file;
+		$file = 'ContainerAwareTrait.php';
+		$relativePath = 'league/container/src/';
+		$targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath;
+		$targetFile = $targetPath . $file;
 
-        mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true);
+		mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true);
 
-        file_put_contents($targetFile, 'dummy file');
+		file_put_contents($targetFile, 'dummy file');
 
-        assert(file_exists($targetFile));
+		assert(file_exists($targetFile));
 
-        $copier->prepareTarget();
+		$copier->prepareTarget();
 
-        $this->assertFileDoesNotExist($targetFile);
-    }
+		$this->assertFileDoesNotExist($targetFile);
+	}
 
-    public function testsCopy()
-    {
+	public function testsCopy()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -96,183 +96,183 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+		$projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+		$dependencies = array_map(function ($element) {
+			$dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
+			return new ComposerPackage($dir);
+		}, $projectComposerPackage->getRequiresNames());
 
-        $workingDir = $this->testsWorkingDir;
-        $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
-        $config = $this->createStub(StraussConfig::class);
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
-        $fileEnumerator->compileFileList();
-        $filepaths = $fileEnumerator->getAllFilesAndDependencyList();
+		$workingDir = $this->testsWorkingDir;
+		$relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
+		$config = $this->createStub(StraussConfig::class);
+		$fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+		$fileEnumerator->compileFileList();
+		$filepaths = $fileEnumerator->getAllFilesAndDependencyList();
 
-        $copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
+		$copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
 
-        $file = 'Client.php';
-        $relativePath = 'google/apiclient/src/';
-        $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath;
-        $targetFile = $targetPath . $file;
+		$file = 'Client.php';
+		$relativePath = 'google/apiclient/src/';
+		$targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath;
+		$targetFile = $targetPath . $file;
 
-        $copier->prepareTarget();
+		$copier->prepareTarget();
 
-        $copier->copy();
+		$copier->copy();
 
-        $this->assertFileExists($targetFile);
-    }
+		$this->assertFileExists($targetFile);
+	}
 
 
 
 
-    /**
-     * Set up a common settings object.
-     * @see MoverTest.php
-     */
-    protected function createComposer(): void
-    {
+	/**
+	 * Set up a common settings object.
+	 * @see MoverTest.php
+	 */
+	protected function createComposer(): void
+	{
 //        parent::setUp();
 
-        $this->testsWorkingDir = __DIR__ . '/temptestdir/';
-        if (!file_exists($this->testsWorkingDir)) {
-            mkdir($this->testsWorkingDir);
-        }
-
-        $mozartConfig = new stdClass();
-        $mozartConfig->dep_directory = "/dep_directory/";
-        $mozartConfig->classmap_directory = "/classmap_directory/";
-        $mozartConfig->packages = array(
-            "pimple/pimple",
-            "ezyang/htmlpurifier"
-        );
-
-        $pimpleAutoload = new stdClass();
-        $pimpleAutoload->{'psr-0'} = new stdClass();
-        $pimpleAutoload->{'psr-0'}->Pimple = "src/";
-
-        $htmlpurifierAutoload = new stdClass();
-        $htmlpurifierAutoload->classmap = new stdClass();
-        $htmlpurifierAutoload->classmap->Pimple = "library/";
-
-        $mozartConfig->override_autoload = array();
-        $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload;
-        $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload;
-
-        $composer = new stdClass();
-        $composer->extra = new stdClass();
-        $composer->extra->mozart = $mozartConfig;
-
-        $composerFilepath = $this->testsWorkingDir . '/composer.json';
-        $composerJson = json_encode($composer) ;
-        file_put_contents($composerFilepath, $composerJson);
-
-        $this->config = StraussConfig::loadFromFile($composerFilepath);
-    }
-
-    /**
-     * If the specified `dep_directory` or `classmap_directory` are absent, create them.
-     * @see MoverTest.php
-     * @test
-     */
-    public function it_creates_absent_dirs(): void
-    {
-        $this->markTestIncomplete();
-
-        $mover = new Mover($this->testsWorkingDir, $this->config);
-
-        // Make sure the directories don't exist.
-        assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists");
-        assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory()));
-
-        $packages = array();
-
-        $mover->deleteTargetDirs($packages);
-
-        $this->assertTrue(file_exists($this->testsWorkingDir
-            . $this->config->getDepDirectory()));
-        $this->assertTrue(file_exists($this->testsWorkingDir
-            . $this->config->getClassmapDirectory()));
-    }
-
-    /**
-     * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue.
-     *
-     * @see MoverTest.php
-     *
-     * @test
-     */
-    public function it_is_unpertrubed_by_existing_dirs(): void
-    {
-        $this->markTestIncomplete();
-
-        $mover = new Mover($this->testsWorkingDir, $this->config);
-
-        if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) {
-            mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
-        }
-        if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) {
-            mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
-        }
-
-        $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory());
-        $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory());
-
-        $packages = array();
-
-        ob_start();
-
-        $mover->deleteTargetDirs($packages);
-
-        $output = ob_get_clean();
-
-        $this->assertEmpty($output);
-    }
-
-    /**
-     * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving,
-     * delete the subdir. aka:  If subfolders exist for dependencies we are about to manage, delete those subfolders.
-     *
-     * @see MoverTest.php
-     *
-     * @test
-     */
-    public function it_deletes_subdirs_for_packages_about_to_be_moved(): void
-    {
-        $this->markTestIncomplete();
-
-        $mover = new Mover($this->testsWorkingDir, $this->config);
-
-        @mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
-        @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
-
-        @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
-        @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang');
-
-        $packages = array();
-        foreach ($this->config->getPackages() as $packageString) {
-            $testDummyComposerDir = $this->testsWorkingDir  . 'vendor'
-                . DIRECTORY_SEPARATOR . $packageString;
-            @mkdir($testDummyComposerDir, 0777, true);
-            $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json';
-            $testDummyComposerContents = json_encode(new stdClass());
-
-            file_put_contents($testDummyComposerPath, $testDummyComposerContents);
-            $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]);
-            $parsedPackage->findAutoloaders();
-            $packages[] = $parsedPackage;
-        }
-
-        $mover->deleteTargetDirs($packages);
-
-        $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
-        $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang');
-    }
+		$this->testsWorkingDir = __DIR__ . '/temptestdir/';
+		if (!file_exists($this->testsWorkingDir)) {
+			mkdir($this->testsWorkingDir);
+		}
+
+		$mozartConfig = new stdClass();
+		$mozartConfig->dep_directory = "/dep_directory/";
+		$mozartConfig->classmap_directory = "/classmap_directory/";
+		$mozartConfig->packages = array(
+			"pimple/pimple",
+			"ezyang/htmlpurifier"
+		);
+
+		$pimpleAutoload = new stdClass();
+		$pimpleAutoload->{'psr-0'} = new stdClass();
+		$pimpleAutoload->{'psr-0'}->Pimple = "src/";
+
+		$htmlpurifierAutoload = new stdClass();
+		$htmlpurifierAutoload->classmap = new stdClass();
+		$htmlpurifierAutoload->classmap->Pimple = "library/";
+
+		$mozartConfig->override_autoload = array();
+		$mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload;
+		$mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload;
+
+		$composer = new stdClass();
+		$composer->extra = new stdClass();
+		$composer->extra->mozart = $mozartConfig;
+
+		$composerFilepath = $this->testsWorkingDir . '/composer.json';
+		$composerJson = json_encode($composer) ;
+		file_put_contents($composerFilepath, $composerJson);
+
+		$this->config = StraussConfig::loadFromFile($composerFilepath);
+	}
+
+	/**
+	 * If the specified `dep_directory` or `classmap_directory` are absent, create them.
+	 * @see MoverTest.php
+	 * @test
+	 */
+	public function it_creates_absent_dirs(): void
+	{
+		$this->markTestIncomplete();
+
+		$mover = new Mover($this->testsWorkingDir, $this->config);
+
+		// Make sure the directories don't exist.
+		assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists");
+		assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory()));
+
+		$packages = array();
+
+		$mover->deleteTargetDirs($packages);
+
+		$this->assertTrue(file_exists($this->testsWorkingDir
+			. $this->config->getDepDirectory()));
+		$this->assertTrue(file_exists($this->testsWorkingDir
+			. $this->config->getClassmapDirectory()));
+	}
+
+	/**
+	 * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue.
+	 *
+	 * @see MoverTest.php
+	 *
+	 * @test
+	 */
+	public function it_is_unpertrubed_by_existing_dirs(): void
+	{
+		$this->markTestIncomplete();
+
+		$mover = new Mover($this->testsWorkingDir, $this->config);
+
+		if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) {
+			mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
+		}
+		if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) {
+			mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
+		}
+
+		$this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory());
+		$this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory());
+
+		$packages = array();
+
+		ob_start();
+
+		$mover->deleteTargetDirs($packages);
+
+		$output = ob_get_clean();
+
+		$this->assertEmpty($output);
+	}
+
+	/**
+	 * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving,
+	 * delete the subdir. aka:  If subfolders exist for dependencies we are about to manage, delete those subfolders.
+	 *
+	 * @see MoverTest.php
+	 *
+	 * @test
+	 */
+	public function it_deletes_subdirs_for_packages_about_to_be_moved(): void
+	{
+		$this->markTestIncomplete();
+
+		$mover = new Mover($this->testsWorkingDir, $this->config);
+
+		@mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
+		@mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
+
+		@mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
+		@mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang');
+
+		$packages = array();
+		foreach ($this->config->getPackages() as $packageString) {
+			$testDummyComposerDir = $this->testsWorkingDir  . 'vendor'
+				. DIRECTORY_SEPARATOR . $packageString;
+			@mkdir($testDummyComposerDir, 0777, true);
+			$testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json';
+			$testDummyComposerContents = json_encode(new stdClass());
+
+			file_put_contents($testDummyComposerPath, $testDummyComposerContents);
+			$parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]);
+			$parsedPackage->findAutoloaders();
+			$packages[] = $parsedPackage;
+		}
+
+		$mover->deleteTargetDirs($packages);
+
+		$this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
+		$this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang');
+	}
 }

Spacing +66 added lines, -66 removed lines

@@ -37,44 +37,44 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+        $projectComposerPackage = new ProjectComposerPackage( $this->testsWorkingDir );
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+        $dependencies = array_map( function( $element ) {
+            $dir = $this->testsWorkingDir . 'vendor' . DIRECTORY_SEPARATOR . $element;
+            return new ComposerPackage( $dir );
+        }, $projectComposerPackage->getRequiresNames() );
 
         $workingDir = $this->testsWorkingDir;
         $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+        $fileEnumerator = new FileEnumerator( $dependencies, $workingDir, $config );
         $fileEnumerator->compileFileList();
         $filepaths = $fileEnumerator->getAllFilesAndDependencyList();
 
-        $copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
+        $copier = new Copier( $filepaths, $workingDir, $relativeTargetDir );
 
         $file = 'ContainerAwareTrait.php';
         $relativePath = 'league/container/src/';
         $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath;
         $targetFile = $targetPath . $file;
 
-        mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true);
+        mkdir( rtrim( $targetPath, DIRECTORY_SEPARATOR ), 0777, true );
 
-        file_put_contents($targetFile, 'dummy file');
+        file_put_contents( $targetFile, 'dummy file' );
 
-        assert(file_exists($targetFile));
+        assert( file_exists( $targetFile ) );
 
         $copier->prepareTarget();
 
-        $this->assertFileDoesNotExist($targetFile);
+        $this->assertFileDoesNotExist( $targetFile );
     }
 
     public function testsCopy()
@@ -96,27 +96,27 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+        $projectComposerPackage = new ProjectComposerPackage( $this->testsWorkingDir );
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+        $dependencies = array_map( function( $element ) {
+            $dir = $this->testsWorkingDir . 'vendor' . DIRECTORY_SEPARATOR . $element;
+            return new ComposerPackage( $dir );
+        }, $projectComposerPackage->getRequiresNames() );
 
         $workingDir = $this->testsWorkingDir;
         $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
-        $config = $this->createStub(StraussConfig::class);
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+        $config = $this->createStub( StraussConfig::class );
+        $fileEnumerator = new FileEnumerator( $dependencies, $workingDir, $config );
         $fileEnumerator->compileFileList();
         $filepaths = $fileEnumerator->getAllFilesAndDependencyList();
 
-        $copier = new Copier($filepaths, $workingDir, $relativeTargetDir);
+        $copier = new Copier( $filepaths, $workingDir, $relativeTargetDir );
 
         $file = 'Client.php';
         $relativePath = 'google/apiclient/src/';
@@ -127,7 +127,7 @@ 
 
         $copier->copy();
 
-        $this->assertFileExists($targetFile);
+        $this->assertFileExists( $targetFile );
     }
 
 
@@ -142,8 +142,8 @@ 
 //        parent::setUp();
 
         $this->testsWorkingDir = __DIR__ . '/temptestdir/';
-        if (!file_exists($this->testsWorkingDir)) {
-            mkdir($this->testsWorkingDir);
+        if ( ! file_exists( $this->testsWorkingDir ) ) {
+            mkdir( $this->testsWorkingDir );
         }
 
         $mozartConfig = new stdClass();
@@ -163,18 +163,18 @@ 
         $htmlpurifierAutoload->classmap->Pimple = "library/";
 
         $mozartConfig->override_autoload = array();
-        $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload;
-        $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload;
+        $mozartConfig->override_autoload[ "pimple/pimple" ] = $pimpleAutoload;
+        $mozartConfig->override_autoload[ "ezyang/htmlpurifier" ] = $htmlpurifierAutoload;
 
         $composer = new stdClass();
         $composer->extra = new stdClass();
         $composer->extra->mozart = $mozartConfig;
 
         $composerFilepath = $this->testsWorkingDir . '/composer.json';
-        $composerJson = json_encode($composer) ;
-        file_put_contents($composerFilepath, $composerJson);
+        $composerJson = json_encode( $composer );
+        file_put_contents( $composerFilepath, $composerJson );
 
-        $this->config = StraussConfig::loadFromFile($composerFilepath);
+        $this->config = StraussConfig::loadFromFile( $composerFilepath );
     }
 
     /**
@@ -186,20 +186,20 @@ 
     {
         $this->markTestIncomplete();
 
-        $mover = new Mover($this->testsWorkingDir, $this->config);
+        $mover = new Mover( $this->testsWorkingDir, $this->config );
 
         // Make sure the directories don't exist.
-        assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists");
-        assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory()));
+        assert( ! file_exists( $this->testsWorkingDir . $this->config->gett() ), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists" );
+        assert( ! file_exists( $this->testsWorkingDir . $this->config->getClassmapDirectory() ) );
 
         $packages = array();
 
-        $mover->deleteTargetDirs($packages);
+        $mover->deleteTargetDirs( $packages );
 
-        $this->assertTrue(file_exists($this->testsWorkingDir
-            . $this->config->getDepDirectory()));
-        $this->assertTrue(file_exists($this->testsWorkingDir
-            . $this->config->getClassmapDirectory()));
+        $this->assertTrue( file_exists( $this->testsWorkingDir
+            . $this->config->getDepDirectory() ) );
+        $this->assertTrue( file_exists( $this->testsWorkingDir
+            . $this->config->getClassmapDirectory() ) );
     }
 
     /**
@@ -213,27 +213,27 @@ 
     {
         $this->markTestIncomplete();
 
-        $mover = new Mover($this->testsWorkingDir, $this->config);
+        $mover = new Mover( $this->testsWorkingDir, $this->config );
 
-        if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) {
-            mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
+        if ( ! file_exists( $this->testsWorkingDir . $this->config->getDepDirectory() ) ) {
+            mkdir( $this->testsWorkingDir . $this->config->getDepDirectory() );
         }
-        if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) {
-            mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
+        if ( ! file_exists( $this->testsWorkingDir . $this->config->getClassmapDirectory() ) ) {
+            mkdir( $this->testsWorkingDir . $this->config->getClassmapDirectory() );
         }
 
-        $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory());
-        $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory());
+        $this->assertDirectoryExists( $this->testsWorkingDir . $this->config->getDepDirectory() );
+        $this->assertDirectoryExists( $this->testsWorkingDir . $this->config->getClassmapDirectory() );
 
         $packages = array();
 
         ob_start();
 
-        $mover->deleteTargetDirs($packages);
+        $mover->deleteTargetDirs( $packages );
 
         $output = ob_get_clean();
 
-        $this->assertEmpty($output);
+        $this->assertEmpty( $output );
     }
 
     /**
@@ -248,31 +248,31 @@ 
     {
         $this->markTestIncomplete();
 
-        $mover = new Mover($this->testsWorkingDir, $this->config);
+        $mover = new Mover( $this->testsWorkingDir, $this->config );
 
-        @mkdir($this->testsWorkingDir . $this->config->getDepDirectory());
-        @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory());
+        @mkdir( $this->testsWorkingDir . $this->config->getDepDirectory() );
+        @mkdir( $this->testsWorkingDir . $this->config->getClassmapDirectory() );
 
-        @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
-        @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang');
+        @mkdir( $this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple' );
+        @mkdir( $this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang' );
 
         $packages = array();
-        foreach ($this->config->getPackages() as $packageString) {
-            $testDummyComposerDir = $this->testsWorkingDir  . 'vendor'
+        foreach ( $this->config->getPackages() as $packageString ) {
+            $testDummyComposerDir = $this->testsWorkingDir . 'vendor'
                 . DIRECTORY_SEPARATOR . $packageString;
-            @mkdir($testDummyComposerDir, 0777, true);
+            @mkdir( $testDummyComposerDir, 0777, true );
             $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json';
-            $testDummyComposerContents = json_encode(new stdClass());
+            $testDummyComposerContents = json_encode( new stdClass() );
 
-            file_put_contents($testDummyComposerPath, $testDummyComposerContents);
-            $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]);
+            file_put_contents( $testDummyComposerPath, $testDummyComposerContents );
+            $parsedPackage = new ComposerPackageConfig( $testDummyComposerDir, $this->config->getOverrideAutoload()[ $packageString ] );
             $parsedPackage->findAutoloaders();
-            $packages[] = $parsedPackage;
+            $packages[ ] = $parsedPackage;
         }
 
-        $mover->deleteTargetDirs($packages);
+        $mover->deleteTargetDirs( $packages );
 
-        $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple');
-        $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang');
+        $this->assertDirectoryDoesNotExist( $this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple' );
+        $this->assertDirectoryDoesNotExist( $this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang' );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -15,11 +15,9 @@ 
  * @package BrianHenryIE\Strauss
  * @coversNothing
  */
-class CopierIntegrationTest extends IntegrationTestCase
-{
+class CopierIntegrationTest extends IntegrationTestCase {
 
-    public function testsPrepareTarget()
-    {
+    public function testsPrepareTarget() {
 
         $composerJsonString = <<<'EOD'
 {
@@ -77,8 +75,7 @@ 
         $this->assertFileDoesNotExist($targetFile);
     }
 
-    public function testsCopy()
-    {
+    public function testsCopy() {
 
         $composerJsonString = <<<'EOD'
 {

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
@@ -94,7 +94,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 

vendor/brianhenryie/strauss/tests/Integration/ReplacerIntegrationTest.php

Indentation +44 added lines, -44 removed lines

@@ -23,10 +23,10 @@ 
 class ReplacerIntegrationTest extends IntegrationTestCase
 {
 
-    public function testReplaceNamespace()
-    {
+	public function testReplaceNamespace()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -41,56 +41,56 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
-        $config = $projectComposerPackage->getStraussConfig();
+		$projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+		$config = $projectComposerPackage->getStraussConfig();
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+		$dependencies = array_map(function ($element) {
+			$dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
+			return new ComposerPackage($dir);
+		}, $projectComposerPackage->getRequiresNames());
 
-        $workingDir = $this->testsWorkingDir;
-        $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
-        $absoluteTargetDir = $workingDir . $relativeTargetDir;
+		$workingDir = $this->testsWorkingDir;
+		$relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
+		$absoluteTargetDir = $workingDir . $relativeTargetDir;
 
 //        $config = $this->createStub(StraussConfig::class);
 //        $config->method('getTargetDirectory')->willReturn('strauss' . DIRECTORY_SEPARATOR);
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
-        $fileEnumerator->compileFileList();
-        $fileList = $fileEnumerator->getAllFilesAndDependencyList();
-        $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
+		$fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+		$fileEnumerator->compileFileList();
+		$fileList = $fileEnumerator->getAllFilesAndDependencyList();
+		$phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
 
-        $copier = new Copier($fileList, $workingDir, $relativeTargetDir);
-        $copier->prepareTarget();
-        $copier->copy();
+		$copier = new Copier($fileList, $workingDir, $relativeTargetDir);
+		$copier->prepareTarget();
+		$copier->copy();
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList);
 
-        $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
-        $classes = $changeEnumerator->getDiscoveredClasses();
+		$namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
+		$classes = $changeEnumerator->getDiscoveredClasses();
 
-        $replacer = new Prefixer($config, $workingDir);
+		$replacer = new Prefixer($config, $workingDir);
 
-        $replacer->replaceInFiles($namespaces, $classes, $phpFileList);
+		$replacer->replaceInFiles($namespaces, $classes, $phpFileList);
 
-        $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php');
+		$updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php');
 
-        $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile);
-    }
+		$this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile);
+	}
 
 
-    public function testReplaceClass()
-    {
+	public function testReplaceClass()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -106,18 +106,18 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
 
-        $mozartCompose = new Compose();
+		$mozartCompose = new Compose();
 
-        $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
+		$result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
 
 
 //        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
@@ -150,8 +150,8 @@ 
 //
 //        $replacer->replaceInFiles($namespaces, $classes, $phpFileList);
 
-        $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php');
+		$updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php');
 
-        $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile);
-    }
+		$this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile);
+	}
 }

Spacing +24 added lines, -24 removed lines

@@ -41,19 +41,19 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+        $projectComposerPackage = new ProjectComposerPackage( $this->testsWorkingDir );
         $config = $projectComposerPackage->getStraussConfig();
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+        $dependencies = array_map( function( $element ) {
+            $dir = $this->testsWorkingDir . 'vendor' . DIRECTORY_SEPARATOR . $element;
+            return new ComposerPackage( $dir );
+        }, $projectComposerPackage->getRequiresNames() );
 
         $workingDir = $this->testsWorkingDir;
         $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
@@ -62,28 +62,28 @@ 
 //        $config = $this->createStub(StraussConfig::class);
 //        $config->method('getTargetDirectory')->willReturn('strauss' . DIRECTORY_SEPARATOR);
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+        $fileEnumerator = new FileEnumerator( $dependencies, $workingDir, $config );
         $fileEnumerator->compileFileList();
         $fileList = $fileEnumerator->getAllFilesAndDependencyList();
         $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
 
-        $copier = new Copier($fileList, $workingDir, $relativeTargetDir);
+        $copier = new Copier( $fileList, $workingDir, $relativeTargetDir );
         $copier->prepareTarget();
         $copier->copy();
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList);
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->findInFiles( $absoluteTargetDir, $phpFileList );
 
         $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
         $classes = $changeEnumerator->getDiscoveredClasses();
 
-        $replacer = new Prefixer($config, $workingDir);
+        $replacer = new Prefixer( $config, $workingDir );
 
-        $replacer->replaceInFiles($namespaces, $classes, $phpFileList);
+        $replacer->replaceInFiles( $namespaces, $classes, $phpFileList );
 
-        $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php');
+        $updatedFile = file_get_contents( $absoluteTargetDir . 'google/apiclient/src/Client.php' );
 
-        $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile);
+        $this->assertStringContainsString( 'use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile );
     }
 
 
@@ -106,18 +106,18 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
         $mozartCompose = new Compose();
 
-        $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
+        $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock );
 
 
 //        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
@@ -150,8 +150,8 @@ 
 //
 //        $replacer->replaceInFiles($namespaces, $classes, $phpFileList);
 
-        $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php');
+        $updatedFile = file_get_contents( $this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php' );
 
-        $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile);
+        $this->assertStringContainsString( 'class BrianHenryIE_Strauss_FPDF', $updatedFile );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -20,11 +20,9 @@ 
  * @package BrianHenryIE\Strauss\Tests\Integration
  * @coversNothing
  */
-class ReplacerIntegrationTest extends IntegrationTestCase
-{
+class ReplacerIntegrationTest extends IntegrationTestCase {
 
-    public function testReplaceNamespace()
-    {
+    public function testReplaceNamespace() {
 
         $composerJsonString = <<<'EOD'
 {
@@ -87,8 +85,7 @@ 
     }
 
 
-    public function testReplaceClass()
-    {
+    public function testReplaceClass() {
 
         $composerJsonString = <<<'EOD'
 {

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
@@ -94,7 +94,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 

vendor/brianhenryie/strauss/tests/Integration/CleanupIntegrationTest.php

Braces +1 added lines, -2 removed lines

@@ -6,7 +6,6 @@
  * @package BrianHenryIE\Strauss\Tests\Integration
  * @coversNothing
  */
-class CleanupIntegrationTest
-{
+class CleanupIntegrationTest {
 
 }

vendor/brianhenryie/strauss/tests/Integration/FileEnumeratorIntegrationTest.php

Indentation +27 added lines, -27 removed lines

@@ -19,10 +19,10 @@ 
 class FileEnumeratorIntegrationTest extends IntegrationTestCase
 {
 
-    public function testBuildFileList()
-    {
+	public function testBuildFileList()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -38,41 +38,41 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+		$projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
 
-        // Only one because we haven't run "flat dependency list".
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+		// Only one because we haven't run "flat dependency list".
+		$dependencies = array_map(function ($element) {
+			$dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
+			return new ComposerPackage($dir);
+		}, $projectComposerPackage->getRequiresNames());
 
-        $workingDir = $this->testsWorkingDir;
-        $config = $this->createStub(StraussConfig::class);
+		$workingDir = $this->testsWorkingDir;
+		$config = $this->createStub(StraussConfig::class);
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+		$fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
 
-        $fileEnumerator->compileFileList();
+		$fileEnumerator->compileFileList();
 
-        $list = array_keys($fileEnumerator->getAllFilesAndDependencyList());
+		$list = array_keys($fileEnumerator->getAllFilesAndDependencyList());
 
-        $this->assertContains('google/apiclient/src/aliases.php', $list);
-    }
+		$this->assertContains('google/apiclient/src/aliases.php', $list);
+	}
 
 
-    public function testClassmapAutoloader()
-    {
-        $this->markTestIncomplete();
-    }
+	public function testClassmapAutoloader()
+	{
+		$this->markTestIncomplete();
+	}
 
 
-    public function testFilesAutoloader()
-    {
-        $this->markTestIncomplete();
-    }
+	public function testFilesAutoloader()
+	{
+		$this->markTestIncomplete();
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -38,30 +38,30 @@
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+        $projectComposerPackage = new ProjectComposerPackage( $this->testsWorkingDir );
 
         // Only one because we haven't run "flat dependency list".
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+        $dependencies = array_map( function( $element ) {
+            $dir = $this->testsWorkingDir . 'vendor' . DIRECTORY_SEPARATOR . $element;
+            return new ComposerPackage( $dir );
+        }, $projectComposerPackage->getRequiresNames() );
 
         $workingDir = $this->testsWorkingDir;
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+        $fileEnumerator = new FileEnumerator( $dependencies, $workingDir, $config );
 
         $fileEnumerator->compileFileList();
 
-        $list = array_keys($fileEnumerator->getAllFilesAndDependencyList());
+        $list = array_keys( $fileEnumerator->getAllFilesAndDependencyList() );
 
-        $this->assertContains('google/apiclient/src/aliases.php', $list);
+        $this->assertContains( 'google/apiclient/src/aliases.php', $list );
     }
 
 

Braces +4 added lines, -8 removed lines

@@ -16,11 +16,9 @@ 
  * @package BrianHenryIE\Strauss
  * @coversNothing
  */
-class FileEnumeratorIntegrationTest extends IntegrationTestCase
-{
+class FileEnumeratorIntegrationTest extends IntegrationTestCase {
 
-    public function testBuildFileList()
-    {
+    public function testBuildFileList() {
 
         $composerJsonString = <<<'EOD'
 {
@@ -65,14 +63,12 @@ 
     }
 
 
-    public function testClassmapAutoloader()
-    {
+    public function testClassmapAutoloader() {
         $this->markTestIncomplete();
     }
 
 
-    public function testFilesAutoloader()
-    {
+    public function testFilesAutoloader() {
         $this->markTestIncomplete();
     }
 }

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 

vendor/brianhenryie/strauss/tests/Integration/ChangeEnumeratorIntegrationTest.php

Indentation +34 added lines, -34 removed lines

@@ -18,13 +18,13 @@ 
 class ChangeEnumeratorIntegrationTest extends IntegrationTestCase
 {
 
-    /**
-     * Given a list of files, find all the global classes and the namespaces.
-     */
-    public function testOne()
-    {
+	/**
+	 * Given a list of files, find all the global classes and the namespaces.
+	 */
+	public function testOne()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss",
   "require": {
@@ -40,53 +40,53 @@ 
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+		file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
 
-        chdir($this->testsWorkingDir);
+		chdir($this->testsWorkingDir);
 
-        exec('composer install');
+		exec('composer install');
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+		$projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+		$dependencies = array_map(function ($element) {
+			$dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
+			return new ComposerPackage($dir);
+		}, $projectComposerPackage->getRequiresNames());
 
-        $workingDir = $this->testsWorkingDir;
-        $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
+		$workingDir = $this->testsWorkingDir;
+		$relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
 
-        $config = $this->createStub(StraussConfig::class);
+		$config = $this->createStub(StraussConfig::class);
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+		$fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
 
-        $fileEnumerator->compileFileList();
+		$fileEnumerator->compileFileList();
 
-        $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir);
+		$copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir);
 
-        $copier->prepareTarget();
+		$copier->prepareTarget();
 
-        $copier->copy();
+		$copier->copy();
 
-        $config = $this->createStub(StraussConfig::class);
+		$config = $this->createStub(StraussConfig::class);
 
-        $config->method('getExcludePackagesFromPrefixing')->willReturn(array());
-        $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array());
+		$config->method('getExcludePackagesFromPrefixing')->willReturn(array());
+		$config->method('getExcludeNamespacesFromPrefixing')->willReturn(array());
 
-        $changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator = new ChangeEnumerator($config);
 
-        $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
+		$phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
 
-        $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList);
+		$changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList);
 
 
-        $classes = $changeEnumerator->getDiscoveredClasses();
+		$classes = $changeEnumerator->getDiscoveredClasses();
 
-        $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
+		$namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
 
-        $this->assertNotEmpty($classes);
-        $this->assertNotEmpty($namespaces);
+		$this->assertNotEmpty($classes);
+		$this->assertNotEmpty($namespaces);
 
-        $this->assertContains('Google_Task_Composer', $classes);
-    }
+		$this->assertContains('Google_Task_Composer', $classes);
+	}
 }

Spacing +19 added lines, -19 removed lines

@@ -40,53 +40,53 @@
 }
 EOD;
 
-        file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
+        file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString );
 
-        chdir($this->testsWorkingDir);
+        chdir( $this->testsWorkingDir );
 
-        exec('composer install');
+        exec( 'composer install' );
 
-        $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir);
+        $projectComposerPackage = new ProjectComposerPackage( $this->testsWorkingDir );
 
-        $dependencies = array_map(function ($element) {
-            $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element;
-            return new ComposerPackage($dir);
-        }, $projectComposerPackage->getRequiresNames());
+        $dependencies = array_map( function( $element ) {
+            $dir = $this->testsWorkingDir . 'vendor' . DIRECTORY_SEPARATOR . $element;
+            return new ComposerPackage( $dir );
+        }, $projectComposerPackage->getRequiresNames() );
 
         $workingDir = $this->testsWorkingDir;
         $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR;
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
 
-        $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config);
+        $fileEnumerator = new FileEnumerator( $dependencies, $workingDir, $config );
 
         $fileEnumerator->compileFileList();
 
-        $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir);
+        $copier = new Copier( $fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir );
 
         $copier->prepareTarget();
 
         $copier->copy();
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
 
-        $config->method('getExcludePackagesFromPrefixing')->willReturn(array());
-        $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array());
+        $config->method( 'getExcludePackagesFromPrefixing' )->willReturn( array() );
+        $config->method( 'getExcludeNamespacesFromPrefixing' )->willReturn( array() );
 
-        $changeEnumerator = new ChangeEnumerator($config);
+        $changeEnumerator = new ChangeEnumerator( $config );
 
         $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList();
 
-        $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList);
+        $changeEnumerator->findInFiles( $workingDir . $relativeTargetDir, $phpFileList );
 
 
         $classes = $changeEnumerator->getDiscoveredClasses();
 
         $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements();
 
-        $this->assertNotEmpty($classes);
-        $this->assertNotEmpty($namespaces);
+        $this->assertNotEmpty( $classes );
+        $this->assertNotEmpty( $namespaces );
 
-        $this->assertContains('Google_Task_Composer', $classes);
+        $this->assertContains( 'Google_Task_Composer', $classes );
     }
 }

Braces +2 added lines, -4 removed lines

@@ -15,14 +15,12 @@
  * @package BrianHenryIE\Strauss
  * @coversNothing
  */
-class ChangeEnumeratorIntegrationTest extends IntegrationTestCase
-{
+class ChangeEnumeratorIntegrationTest extends IntegrationTestCase {
 
     /**
      * Given a list of files, find all the global classes and the namespaces.
      */
-    public function testOne()
-    {
+    public function testOne() {
 
         $composerJsonString = <<<'EOD'
 {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     }
   }
 }
-EOD;
+eod;
 
         file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);