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);
+	}
 }

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);
+			}
+		};
+	}
 }

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);
+	}
 }

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);
+	}
 }

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');
+	}
 }

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);
+	}
 }

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

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);
+	}
 }

vendor/brianhenryie/strauss/tests/Issues/MozartIssue66Test.php

Indentation +25 added lines, -25 removed lines

@@ -22,23 +22,23 @@ 
 class MozartIssue66Test extends IntegrationTestCase
 {
 
-    /**
-     *
-     * php-di's composer.json's autoload key:
-     *
-     * "autoload": {
-     *    "psr-4": {
-     *      "DI\\": "src/"
-     *     },
-     *     "files": [
-     *        "src/functions.php"
-     *    ]
-     * },
-     */
-    public function testFilesAutoloaderIsUsed()
-    {
+	/**
+	 *
+	 * php-di's composer.json's autoload key:
+	 *
+	 * "autoload": {
+	 *    "psr-4": {
+	 *      "DI\\": "src/"
+	 *     },
+	 *     "files": [
+	 *        "src/functions.php"
+	 *    ]
+	 * },
+	 */
+	public function testFilesAutoloaderIsUsed()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "markjaquith/mozart-bug-example",
   "require": {
@@ -64,19 +64,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');
 
-        $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();
 
-        $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
+		$mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
 
-        $this->assertFileExists($this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php');
-    }
+		$this->assertFileExists($this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php');
+	}
 }