Inspection of "Fix script and styles not enquing when manually re..." - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#1656)

unknown

created 2021-10-27 22:19 UTC

Status

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

@@ -18,15 +18,15 @@  discard block
 block discarded – undo
 class MozartIssue106Test extends IntegrationTestCase
 {
 
-    /**
-     * Issue #106, multiple classmap prefixing.
-     *
-     * @see https://github.com/coenjacobs/mozart/issues/106
-     */
-    public function test_only_prefix_classmap_classes_once()
-    {
+	/**
+	 * Issue #106, multiple classmap prefixing.
+	 *
+	 * @see https://github.com/coenjacobs/mozart/issues/106
+	 */
+	public function test_only_prefix_classmap_classes_once()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-106",
 	"require": {
@@ -42,25 +42,25 @@  discard block
 block discarded – undo
 }
 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);
 
-        $php_string = file_get_contents($this->testsWorkingDir .'/strauss/symfony/polyfill-intl-normalizer/Resources/stubs/Normalizer.php');
+		$php_string = file_get_contents($this->testsWorkingDir .'/strauss/symfony/polyfill-intl-normalizer/Resources/stubs/Normalizer.php');
 
-        // Confirm problem is gone.
-        $this->assertStringNotContainsString('class BrianHenryIE_Strauss_BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Double prefixing problem still present.');
+		// Confirm problem is gone.
+		$this->assertStringNotContainsString('class BrianHenryIE_Strauss_BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Double prefixing problem still present.');
 
-        // Confirm solution is correct.
-        $this->assertStringContainsString('class BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Class name not properly prefixed.');
-    }
+		// Confirm solution is correct.
+		$this->assertStringContainsString('class BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Class name not properly prefixed.');
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue108Test.php 1 patch

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

@@ -28,13 +28,13 @@  discard block
 block discarded – undo
 class MozartIssue108Test extends IntegrationTestCase
 {
 
-    /**
-     * WooCommerce Action Scheduler ... has no autoload key. But also needs some Mozart patches to work correctly.
-     */
-    public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
-    {
+	/**
+	 * WooCommerce Action Scheduler ... has no autoload key. But also needs some Mozart patches to work correctly.
+	 */
+	public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "require": {
     "woocommerce/action-scheduler": "dev-master",
@@ -58,31 +58,31 @@  discard block
 block discarded – undo
 }
 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);
 
-        $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/deliciousbrains/wp-background-processing/classes/wp-async-request.php');
-        $this->assertStringContainsString('abstract class Strauss_WP_Async_Request', $pdf_contents);
+		$pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/deliciousbrains/wp-background-processing/classes/wp-async-request.php');
+		$this->assertStringContainsString('abstract class Strauss_WP_Async_Request', $pdf_contents);
 
 //        $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/mtdowling/cron-expression/src/Cron/CronExpression.php');
 //        $this->assertStringContainsString('namespace Strauss\\CronExpression', $pdf_contents);
 
-        $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/lib/cron-expression/CronExpression.php');
-        $this->assertStringContainsString('class Strauss_CronExpression', $pdf_contents);
+		$pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/lib/cron-expression/CronExpression.php');
+		$this->assertStringContainsString('class Strauss_CronExpression', $pdf_contents);
 
-        $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php');
-        $this->assertStringContainsString('if ( ! is_a( $recurrence, \'Strauss_CronExpression\' ) ) {', $pdf_contents);
+		$pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php');
+		$this->assertStringContainsString('if ( ! is_a( $recurrence, \'Strauss_CronExpression\' ) ) {', $pdf_contents);
 
-        $this->markTestIncomplete('Shouldnt use dev-master, should use a published package.');
-    }
+		$this->markTestIncomplete('Shouldnt use dev-master, should use a published package.');
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue90Test.php 1 patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -18,15 +18,15 @@  discard block
 block discarded – undo
 class MozartIssue90Test extends IntegrationTestCase
 {
 
-    /**
-     * Issue 90. Needs "iio/libmergepdf".
-     *
-     * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php".
-     */
-    public function testLibpdfmergeSucceeds()
-    {
+	/**
+	 * Issue 90. Needs "iio/libmergepdf".
+	 *
+	 * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php".
+	 */
+	public function testLibpdfmergeSucceeds()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-90",
 	"require": {
@@ -41,24 +41,24 @@  discard block
 block discarded – undo
 }
 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);
 
-        $this->assertEquals(0, $result);
+		$this->assertEquals(0, $result);
 
-        // This test would only fail on Windows?
-        $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi');
+		// This test would only fail on Windows?
+		$this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi');
 
-        $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php');
-    }
+		$this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php');
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue86Test.php 1 patch

Indentation +43 added lines, -43 removed lines patch added patch discarded remove patch

@@ -19,15 +19,15 @@  discard block
 block discarded – undo
 class MozartIssue86Test extends IntegrationTestCase
 {
 
-    /**
-     * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed!
-     *
-     * Solved by https://github.com/ziodave
-     */
-    public function test_do_not_parse_comments_to_classnames()
-    {
-
-        $composerJsonString = <<<'EOD'
+	/**
+	 * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed!
+	 *
+	 * Solved by https://github.com/ziodave
+	 */
+	public function test_do_not_parse_comments_to_classnames()
+	{
+
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-86",
 	"require": {
@@ -50,38 +50,38 @@  discard block
 block discarded – undo
 }
 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);
 
-        $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php');
+		$php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php');
 
-        // Confirm problem is gone.
-        $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string);
+		// Confirm problem is gone.
+		$this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string);
 
-        // Confirm solution is correct.
-        $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string);
-    }
+		// Confirm solution is correct.
+		$this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string);
+	}
 
 
-    /**
-     * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were
-     *
-     * @see https://github.com/afragen/wp-dependency-installer/
-     */
-    public function test_do_not_parse_comments_to_classnames_wp_dependency_installer()
-    {
+	/**
+	 * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were
+	 *
+	 * @see https://github.com/afragen/wp-dependency-installer/
+	 */
+	public function test_do_not_parse_comments_to_classnames_wp_dependency_installer()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-86-2",
 	"require": {
@@ -96,25 +96,25 @@  discard block
 block discarded – undo
 }
 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);
 
-        $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php');
+		$php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php');
 
-        // Confirm problem is gone.
-        $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.');
+		// Confirm problem is gone.
+		$this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.');
 
-        // Confirm solution is correct.
-        $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.');
-    }
+		// Confirm solution is correct.
+		$this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.');
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/StraussIssue19Test.php 1 patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -17,15 +17,15 @@  discard block
 block discarded – undo
 class StraussIssue19Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase
 {
 
-    /**
-     * Need to make the class finder in change enumerator stricter.
-     *
-     * @author BrianHenryIE
-     */
-    public function testObjectIsNotPrefixed()
-    {
-
-        $composerJsonString = <<<'EOD'
+	/**
+	 * Need to make the class finder in change enumerator stricter.
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function testObjectIsNotPrefixed()
+	{
+
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/strauss-issue-19",
   "require": {
@@ -40,23 +40,23 @@  discard block
 block discarded – undo
 }
 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);
 
-        $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php');
+		$php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php');
 
-        $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string);
+		$this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string);
         
-        $this->assertStringContainsString('* Creates a copy of a class object', $php_string);
-    }
+		$this->assertStringContainsString('* Creates a copy of a class object', $php_string);
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue97Test.php 1 patch

Indentation +19 added lines, -19 removed lines patch added patch discarded remove patch

@@ -15,16 +15,16 @@  discard block
 block discarded – undo
 class MozartIssue97Test extends IntegrationTestCase
 {
 
-    /**
-     * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json
-     * has the package name as "unsplash/unsplash".
-     *
-     * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist."
-     */
-    public function testCrewlabsUnsplashSucceeds()
-    {
-
-        $composerJsonString = <<<'EOD'
+	/**
+	 * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json
+	 * has the package name as "unsplash/unsplash".
+	 *
+	 * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist."
+	 */
+	public function testCrewlabsUnsplashSucceeds()
+	{
+
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-97",
 	"require": {
@@ -39,19 +39,19 @@  discard block
 block discarded – undo
 }
 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);
 
-        $this->assertEquals(0, $result);
-    }
+		$this->assertEquals(0, $result);
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue48Test.php 1 patch

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

@@ -20,52 +20,52 @@
 block discarded – undo
 class MozartIssue48Test extends IntegrationTestCase
 {
 
-    /**
-     * rubix/tensor
-     *
-     * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others)
-     * `EigenvalueDecomposition.php` at all. Test for its presence.
-     */
-    public function testRubixTensorBothPathsPersist()
-    {
+	/**
+	 * rubix/tensor
+	 *
+	 * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others)
+	 * `EigenvalueDecomposition.php` at all. Test for its presence.
+	 */
+	public function testRubixTensorBothPathsPersist()
+	{
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
     "name": "brianhenryie/mozart-issue-48",
     "require": { "rubix/tensor": "*" }
 }
 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);
         
-        // EigenvalueDecomposition.php
-        // assert file exists somewhere in the tree
+		// EigenvalueDecomposition.php
+		// assert file exists somewhere in the tree
 
-        // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file
-        $rsearch = function ($folder, $pattern) {
-            $dir = new \RecursiveDirectoryIterator($folder);
-            $ite = new \RecursiveIteratorIterator($dir);
-            $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH);
-            $fileList = array();
-            foreach ($files as $file) {
-                $fileList = array_merge($fileList, $file);
-            }
-            return $fileList;
-        };
+		// https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file
+		$rsearch = function ($folder, $pattern) {
+			$dir = new \RecursiveDirectoryIterator($folder);
+			$ite = new \RecursiveIteratorIterator($dir);
+			$files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH);
+			$fileList = array();
+			foreach ($files as $file) {
+				$fileList = array_merge($fileList, $file);
+			}
+			return $fileList;
+		};
 
-        $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~');
+		$found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~');
 
-        $this->assertNotEmpty($found);
-    }
+		$this->assertNotEmpty($found);
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue62Test.php 1 patch

Indentation +17 added lines, -17 removed lines patch added patch discarded remove patch

@@ -32,14 +32,14 @@  discard block
 block discarded – undo
 class MozartIssue62Test extends IntegrationTestCase
 {
 
-    /**
-     * Just confirms `use Guzzle\Common\Collection;` is prefixed.
-     */
-    public function testGuzzleNamespaceIsPrefixedInS3Client()
-    {
-        $this->markTestSkipped('Very slow to run');
+	/**
+	 * Just confirms `use Guzzle\Common\Collection;` is prefixed.
+	 */
+	public function testGuzzleNamespaceIsPrefixedInS3Client()
+	{
+		$this->markTestSkipped('Very slow to run');
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
   "name": "brianhenryie/mozart-issue-62",
   "require": {
@@ -54,21 +54,21 @@  discard block
 block discarded – undo
 }
 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);
 
-        $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php');
+		$phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php');
 
-        $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString);
-    }
+		$this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString);
+	}
 }

Please login to merge, or discard this patch.

vendor/brianhenryie/strauss/tests/Issues/MozartIssue93Test.php 1 patch

Indentation +26 added lines, -26 removed lines patch added patch discarded remove patch

@@ -17,21 +17,21 @@  discard block
 block discarded – undo
  */
 class MozartIssue93Test extends IntegrationTestCase
 {
-    /**
-     * Issue #93 shows a classname being updated inside a class whose namespace has also been updated
-     * by Mozart.
-     *
-     * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader.
-     * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
-    {
+	/**
+	 * Issue #93 shows a classname being updated inside a class whose namespace has also been updated
+	 * by Mozart.
+	 *
+	 * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader.
+	 * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
+	{
 
-        $this->markTestSkipped('Not respecting the pinned commit.');
+		$this->markTestSkipped('Not respecting the pinned commit.');
 
-        $composerJsonString = <<<'EOD'
+		$composerJsonString = <<<'EOD'
 {
 	"name": "brianhenryie/mozart-issue-93",
 	"repositories": [{
@@ -51,25 +51,25 @@  discard block
 block discarded – undo
 }
 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);
 
-        $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php');
+		$php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php');
 
-        // Confirm problem is gone.
-        $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string);
+		// Confirm problem is gone.
+		$this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string);
 
-        // Confirm solution is correct.
-        $this->assertStringContainsString('class Logger extends', $php_string);
-    }
+		// Confirm solution is correct.
+		$this->assertStringContainsString('class Logger extends', $php_string);
+	}
 }

Please login to merge, or discard this patch.

		@@ -18,15 +18,15 @@ discard block
		block discarded – undo
18	18	class MozartIssue106Test extends IntegrationTestCase
19	19	{
20	20
21		- /**
22		- * Issue #106, multiple classmap prefixing.
23		- *
24		- * @see https://github.com/coenjacobs/mozart/issues/106
25		- */
26		- public function test_only_prefix_classmap_classes_once()
27		- {
	21	+ /**
	22	+ * Issue #106, multiple classmap prefixing.
	23	+ *
	24	+ * @see https://github.com/coenjacobs/mozart/issues/106
	25	+ */
	26	+ public function test_only_prefix_classmap_classes_once()
	27	+ {
28	28
29		- $composerJsonString = <<<'EOD'
	29	+ $composerJsonString = <<<'EOD'
30	30	{
31	31	"name": "brianhenryie/mozart-issue-106",
32	32	"require": {
		@@ -42,25 +42,25 @@ discard block
		block discarded – undo
42	42	}
43	43	EOD;
44	44
45		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	45	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
46	46
47		- chdir($this->testsWorkingDir);
	47	+ chdir($this->testsWorkingDir);
48	48
49		- exec('composer install');
	49	+ exec('composer install');
50	50
51		- $inputInterfaceMock = $this->createMock(InputInterface::class);
52		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	51	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	52	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
53	53
54		- $mozartCompose = new Compose();
	54	+ $mozartCompose = new Compose();
55	55
56		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	56	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
57	57
58		- $php_string = file_get_contents($this->testsWorkingDir .'/strauss/symfony/polyfill-intl-normalizer/Resources/stubs/Normalizer.php');
	58	+ $php_string = file_get_contents($this->testsWorkingDir .'/strauss/symfony/polyfill-intl-normalizer/Resources/stubs/Normalizer.php');
59	59
60		- // Confirm problem is gone.
61		- $this->assertStringNotContainsString('class BrianHenryIE_Strauss_BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Double prefixing problem still present.');
	60	+ // Confirm problem is gone.
	61	+ $this->assertStringNotContainsString('class BrianHenryIE_Strauss_BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Double prefixing problem still present.');
62	62
63		- // Confirm solution is correct.
64		- $this->assertStringContainsString('class BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Class name not properly prefixed.');
65		- }
	63	+ // Confirm solution is correct.
	64	+ $this->assertStringContainsString('class BrianHenryIE_Strauss_Normalizer extends', $php_string, 'Class name not properly prefixed.');
	65	+ }
66	66	}

		@@ -28,13 +28,13 @@ discard block
		block discarded – undo
28	28	class MozartIssue108Test extends IntegrationTestCase
29	29	{
30	30
31		- /**
32		- * WooCommerce Action Scheduler ... has no autoload key. But also needs some Mozart patches to work correctly.
33		- */
34		- public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
35		- {
	31	+ /**
	32	+ * WooCommerce Action Scheduler ... has no autoload key. But also needs some Mozart patches to work correctly.
	33	+ */
	34	+ public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
	35	+ {
36	36
37		- $composerJsonString = <<<'EOD'
	37	+ $composerJsonString = <<<'EOD'
38	38	{
39	39	"require": {
40	40	"woocommerce/action-scheduler": "dev-master",
		@@ -58,31 +58,31 @@ discard block
		block discarded – undo
58	58	}
59	59	EOD;
60	60
61		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	61	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
62	62
63		- chdir($this->testsWorkingDir);
	63	+ chdir($this->testsWorkingDir);
64	64
65		- exec('composer install');
	65	+ exec('composer install');
66	66
67		- $inputInterfaceMock = $this->createMock(InputInterface::class);
68		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	67	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	68	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
69	69
70		- $mozartCompose = new Compose();
	70	+ $mozartCompose = new Compose();
71	71
72		- $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	72	+ $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
73	73
74		- $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/deliciousbrains/wp-background-processing/classes/wp-async-request.php');
75		- $this->assertStringContainsString('abstract class Strauss_WP_Async_Request', $pdf_contents);
	74	+ $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/deliciousbrains/wp-background-processing/classes/wp-async-request.php');
	75	+ $this->assertStringContainsString('abstract class Strauss_WP_Async_Request', $pdf_contents);
76	76
77	77	// $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/mtdowling/cron-expression/src/Cron/CronExpression.php');
78	78	// $this->assertStringContainsString('namespace Strauss\\CronExpression', $pdf_contents);
79	79
80		- $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/lib/cron-expression/CronExpression.php');
81		- $this->assertStringContainsString('class Strauss_CronExpression', $pdf_contents);
	80	+ $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/lib/cron-expression/CronExpression.php');
	81	+ $this->assertStringContainsString('class Strauss_CronExpression', $pdf_contents);
82	82
83		- $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php');
84		- $this->assertStringContainsString('if ( ! is_a( $recurrence, \'Strauss_CronExpression\' ) ) {', $pdf_contents);
	83	+ $pdf_contents = file_get_contents($this->testsWorkingDir .'strauss/woocommerce/action-scheduler/classes/schedules/ActionScheduler_CronSchedule.php');
	84	+ $this->assertStringContainsString('if ( ! is_a( $recurrence, \'Strauss_CronExpression\' ) ) {', $pdf_contents);
85	85
86		- $this->markTestIncomplete('Shouldnt use dev-master, should use a published package.');
87		- }
	86	+ $this->markTestIncomplete('Shouldnt use dev-master, should use a published package.');
	87	+ }
88	88	}

		@@ -18,15 +18,15 @@ discard block
		block discarded – undo
18	18	class MozartIssue90Test extends IntegrationTestCase
19	19	{
20	20
21		- /**
22		- * Issue 90. Needs "iio/libmergepdf".
23		- *
24		- * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php".
25		- */
26		- public function testLibpdfmergeSucceeds()
27		- {
	21	+ /**
	22	+ * Issue 90. Needs "iio/libmergepdf".
	23	+ *
	24	+ * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php".
	25	+ */
	26	+ public function testLibpdfmergeSucceeds()
	27	+ {
28	28
29		- $composerJsonString = <<<'EOD'
	29	+ $composerJsonString = <<<'EOD'
30	30	{
31	31	"name": "brianhenryie/mozart-issue-90",
32	32	"require": {
		@@ -41,24 +41,24 @@ discard block
		block discarded – undo
41	41	}
42	42	EOD;
43	43
44		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	44	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
45	45
46		- chdir($this->testsWorkingDir);
	46	+ chdir($this->testsWorkingDir);
47	47
48		- exec('composer install');
	48	+ exec('composer install');
49	49
50		- $inputInterfaceMock = $this->createMock(InputInterface::class);
51		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	50	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	51	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
52	52
53		- $mozartCompose = new Compose();
	53	+ $mozartCompose = new Compose();
54	54
55		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	55	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
56	56
57		- $this->assertEquals(0, $result);
	57	+ $this->assertEquals(0, $result);
58	58
59		- // This test would only fail on Windows?
60		- $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi');
	59	+ // This test would only fail on Windows?
	60	+ $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi');
61	61
62		- $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php');
63		- }
	62	+ $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php');
	63	+ }
64	64	}

		@@ -19,15 +19,15 @@ discard block
		block discarded – undo
19	19	class MozartIssue86Test extends IntegrationTestCase
20	20	{
21	21
22		- /**
23		- * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed!
24		- *
25		- * Solved by https://github.com/ziodave
26		- */
27		- public function test_do_not_parse_comments_to_classnames()
28		- {
29		-
30		- $composerJsonString = <<<'EOD'
	22	+ /**
	23	+ * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed!
	24	+ *
	25	+ * Solved by https://github.com/ziodave
	26	+ */
	27	+ public function test_do_not_parse_comments_to_classnames()
	28	+ {
	29	+
	30	+ $composerJsonString = <<<'EOD'
31	31	{
32	32	"name": "brianhenryie/mozart-issue-86",
33	33	"require": {
		@@ -50,38 +50,38 @@ discard block
		block discarded – undo
50	50	}
51	51	EOD;
52	52
53		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	53	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
54	54
55		- chdir($this->testsWorkingDir);
	55	+ chdir($this->testsWorkingDir);
56	56
57		- exec('composer install');
	57	+ exec('composer install');
58	58
59		- $inputInterfaceMock = $this->createMock(InputInterface::class);
60		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	59	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	60	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
61	61
62		- $mozartCompose = new Compose();
	62	+ $mozartCompose = new Compose();
63	63
64		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	64	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
65	65
66		- $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php');
	66	+ $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php');
67	67
68		- // Confirm problem is gone.
69		- $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string);
	68	+ // Confirm problem is gone.
	69	+ $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string);
70	70
71		- // Confirm solution is correct.
72		- $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string);
73		- }
	71	+ // Confirm solution is correct.
	72	+ $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string);
	73	+ }
74	74
75	75
76		- /**
77		- * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were
78		- *
79		- * @see https://github.com/afragen/wp-dependency-installer/
80		- */
81		- public function test_do_not_parse_comments_to_classnames_wp_dependency_installer()
82		- {
	76	+ /**
	77	+ * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were
	78	+ *
	79	+ * @see https://github.com/afragen/wp-dependency-installer/
	80	+ */
	81	+ public function test_do_not_parse_comments_to_classnames_wp_dependency_installer()
	82	+ {
83	83
84		- $composerJsonString = <<<'EOD'
	84	+ $composerJsonString = <<<'EOD'
85	85	{
86	86	"name": "brianhenryie/mozart-issue-86-2",
87	87	"require": {
		@@ -96,25 +96,25 @@ discard block
		block discarded – undo
96	96	}
97	97	EOD;
98	98
99		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	99	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
100	100
101		- chdir($this->testsWorkingDir);
	101	+ chdir($this->testsWorkingDir);
102	102
103		- exec('composer install');
	103	+ exec('composer install');
104	104
105		- $inputInterfaceMock = $this->createMock(InputInterface::class);
106		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	105	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	106	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
107	107
108		- $mozartCompose = new Compose();
	108	+ $mozartCompose = new Compose();
109	109
110		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	110	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
111	111
112		- $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php');
	112	+ $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php');
113	113
114		- // Confirm problem is gone.
115		- $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.');
	114	+ // Confirm problem is gone.
	115	+ $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.');
116	116
117		- // Confirm solution is correct.
118		- $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.');
119		- }
	117	+ // Confirm solution is correct.
	118	+ $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.');
	119	+ }
120	120	}

		@@ -17,15 +17,15 @@ discard block
		block discarded – undo
17	17	class StraussIssue19Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase
18	18	{
19	19
20		- /**
21		- * Need to make the class finder in change enumerator stricter.
22		- *
23		- * @author BrianHenryIE
24		- */
25		- public function testObjectIsNotPrefixed()
26		- {
27		-
28		- $composerJsonString = <<<'EOD'
	20	+ /**
	21	+ * Need to make the class finder in change enumerator stricter.
	22	+ *
	23	+ * @author BrianHenryIE
	24	+ */
	25	+ public function testObjectIsNotPrefixed()
	26	+ {
	27	+
	28	+ $composerJsonString = <<<'EOD'
29	29	{
30	30	"name": "brianhenryie/strauss-issue-19",
31	31	"require": {
		@@ -40,23 +40,23 @@ discard block
		block discarded – undo
40	40	}
41	41	EOD;
42	42
43		- file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString);
	43	+ file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString);
44	44
45		- chdir($this->testsWorkingDir);
	45	+ chdir($this->testsWorkingDir);
46	46
47		- exec('composer install');
	47	+ exec('composer install');
48	48
49		- $inputInterfaceMock = $this->createMock(InputInterface::class);
50		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	49	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	50	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
51	51
52		- $mozartCompose = new Compose();
	52	+ $mozartCompose = new Compose();
53	53
54		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	54	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
55	55
56		- $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php');
	56	+ $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php');
57	57
58		- $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string);
	58	+ $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string);
59	59
60		- $this->assertStringContainsString('* Creates a copy of a class object', $php_string);
61		- }
	60	+ $this->assertStringContainsString('* Creates a copy of a class object', $php_string);
	61	+ }
62	62	}

		@@ -15,16 +15,16 @@ discard block
		block discarded – undo
15	15	class MozartIssue97Test extends IntegrationTestCase
16	16	{
17	17
18		- /**
19		- * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json
20		- * has the package name as "unsplash/unsplash".
21		- *
22		- * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist."
23		- */
24		- public function testCrewlabsUnsplashSucceeds()
25		- {
26		-
27		- $composerJsonString = <<<'EOD'
	18	+ /**
	19	+ * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json
	20	+ * has the package name as "unsplash/unsplash".
	21	+ *
	22	+ * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist."
	23	+ */
	24	+ public function testCrewlabsUnsplashSucceeds()
	25	+ {
	26	+
	27	+ $composerJsonString = <<<'EOD'
28	28	{
29	29	"name": "brianhenryie/mozart-issue-97",
30	30	"require": {
		@@ -39,19 +39,19 @@ discard block
		block discarded – undo
39	39	}
40	40	EOD;
41	41
42		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	42	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
43	43
44		- chdir($this->testsWorkingDir);
	44	+ chdir($this->testsWorkingDir);
45	45
46		- exec('composer install');
	46	+ exec('composer install');
47	47
48		- $inputInterfaceMock = $this->createMock(InputInterface::class);
49		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	48	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	49	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
50	50
51		- $mozartCompose = new Compose();
	51	+ $mozartCompose = new Compose();
52	52
53		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	53	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
54	54
55		- $this->assertEquals(0, $result);
56		- }
	55	+ $this->assertEquals(0, $result);
	56	+ }
57	57	}

		@@ -20,52 +20,52 @@
		block discarded – undo
20	20	class MozartIssue48Test extends IntegrationTestCase
21	21	{
22	22
23		- /**
24		- * rubix/tensor
25		- *
26		- * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others)
27		- * `EigenvalueDecomposition.php` at all. Test for its presence.
28		- */
29		- public function testRubixTensorBothPathsPersist()
30		- {
	23	+ /**
	24	+ * rubix/tensor
	25	+ *
	26	+ * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others)
	27	+ * `EigenvalueDecomposition.php` at all. Test for its presence.
	28	+ */
	29	+ public function testRubixTensorBothPathsPersist()
	30	+ {
31	31
32		- $composerJsonString = <<<'EOD'
	32	+ $composerJsonString = <<<'EOD'
33	33	{
34	34	"name": "brianhenryie/mozart-issue-48",
35	35	"require": { "rubix/tensor": "*" }
36	36	}
37	37	EOD;
38	38
39		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	39	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
40	40
41		- chdir($this->testsWorkingDir);
	41	+ chdir($this->testsWorkingDir);
42	42
43		- exec('composer install');
	43	+ exec('composer install');
44	44
45		- $inputInterfaceMock = $this->createMock(InputInterface::class);
46		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	45	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	46	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
47	47
48		- $mozartCompose = new Compose();
	48	+ $mozartCompose = new Compose();
49	49
50		- $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	50	+ $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
51	51
52		- // EigenvalueDecomposition.php
53		- // assert file exists somewhere in the tree
	52	+ // EigenvalueDecomposition.php
	53	+ // assert file exists somewhere in the tree
54	54
55		- // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file
56		- $rsearch = function ($folder, $pattern) {
57		- $dir = new \RecursiveDirectoryIterator($folder);
58		- $ite = new \RecursiveIteratorIterator($dir);
59		- $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH);
60		- $fileList = array();
61		- foreach ($files as $file) {
62		- $fileList = array_merge($fileList, $file);
63		- }
64		- return $fileList;
65		- };
	55	+ // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file
	56	+ $rsearch = function ($folder, $pattern) {
	57	+ $dir = new \RecursiveDirectoryIterator($folder);
	58	+ $ite = new \RecursiveIteratorIterator($dir);
	59	+ $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH);
	60	+ $fileList = array();
	61	+ foreach ($files as $file) {
	62	+ $fileList = array_merge($fileList, $file);
	63	+ }
	64	+ return $fileList;
	65	+ };
66	66
67		- $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~');
	67	+ $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~');
68	68
69		- $this->assertNotEmpty($found);
70		- }
	69	+ $this->assertNotEmpty($found);
	70	+ }
71	71	}

		@@ -32,14 +32,14 @@ discard block
		block discarded – undo
32	32	class MozartIssue62Test extends IntegrationTestCase
33	33	{
34	34
35		- /**
36		- * Just confirms `use Guzzle\Common\Collection;` is prefixed.
37		- */
38		- public function testGuzzleNamespaceIsPrefixedInS3Client()
39		- {
40		- $this->markTestSkipped('Very slow to run');
	35	+ /**
	36	+ * Just confirms `use Guzzle\Common\Collection;` is prefixed.
	37	+ */
	38	+ public function testGuzzleNamespaceIsPrefixedInS3Client()
	39	+ {
	40	+ $this->markTestSkipped('Very slow to run');
41	41
42		- $composerJsonString = <<<'EOD'
	42	+ $composerJsonString = <<<'EOD'
43	43	{
44	44	"name": "brianhenryie/mozart-issue-62",
45	45	"require": {
		@@ -54,21 +54,21 @@ discard block
		block discarded – undo
54	54	}
55	55	EOD;
56	56
57		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	57	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
58	58
59		- chdir($this->testsWorkingDir);
	59	+ chdir($this->testsWorkingDir);
60	60
61		- exec('composer install');
	61	+ exec('composer install');
62	62
63		- $inputInterfaceMock = $this->createMock(InputInterface::class);
64		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	63	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	64	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
65	65
66		- $mozartCompose = new Compose();
	66	+ $mozartCompose = new Compose();
67	67
68		- $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	68	+ $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
69	69
70		- $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php');
	70	+ $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php');
71	71
72		- $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString);
73		- }
	72	+ $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString);
	73	+ }
74	74	}

		@@ -17,21 +17,21 @@ discard block
		block discarded – undo
17	17	*/
18	18	class MozartIssue93Test extends IntegrationTestCase
19	19	{
20		- /**
21		- * Issue #93 shows a classname being updated inside a class whose namespace has also been updated
22		- * by Mozart.
23		- *
24		- * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader.
25		- * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29
26		- *
27		- * @author BrianHenryIE
28		- */
29		- public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
30		- {
	20	+ /**
	21	+ * Issue #93 shows a classname being updated inside a class whose namespace has also been updated
	22	+ * by Mozart.
	23	+ *
	24	+ * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader.
	25	+ * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29
	26	+ *
	27	+ * @author BrianHenryIE
	28	+ */
	29	+ public function test_it_does_not_make_classname_replacement_inside_namespaced_file()
	30	+ {
31	31
32		- $this->markTestSkipped('Not respecting the pinned commit.');
	32	+ $this->markTestSkipped('Not respecting the pinned commit.');
33	33
34		- $composerJsonString = <<<'EOD'
	34	+ $composerJsonString = <<<'EOD'
35	35	{
36	36	"name": "brianhenryie/mozart-issue-93",
37	37	"repositories": [{
		@@ -51,25 +51,25 @@ discard block
		block discarded – undo
51	51	}
52	52	EOD;
53	53
54		- file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
	54	+ file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString);
55	55
56		- chdir($this->testsWorkingDir);
	56	+ chdir($this->testsWorkingDir);
57	57
58		- exec('composer install');
	58	+ exec('composer install');
59	59
60		- $inputInterfaceMock = $this->createMock(InputInterface::class);
61		- $outputInterfaceMock = $this->createMock(OutputInterface::class);
	60	+ $inputInterfaceMock = $this->createMock(InputInterface::class);
	61	+ $outputInterfaceMock = $this->createMock(OutputInterface::class);
62	62
63		- $mozartCompose = new Compose();
	63	+ $mozartCompose = new Compose();
64	64
65		- $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
	65	+ $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock);
66	66
67		- $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php');
	67	+ $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php');
68	68
69		- // Confirm problem is gone.
70		- $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string);
	69	+ // Confirm problem is gone.
	70	+ $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string);
71	71
72		- // Confirm solution is correct.
73		- $this->assertStringContainsString('class Logger extends', $php_string);
74		- }
	72	+ // Confirm solution is correct.
	73	+ $this->assertStringContainsString('class Logger extends', $php_string);
	74	+ }
75	75	}

gravityview / GravityView

Pull Request — develop (#1656)

Status

Category

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Indentation +43 added lines, -43 removed lines patch added patch discarded remove patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Indentation +19 added lines, -19 removed lines patch added patch discarded remove patch

Indentation +32 added lines, -32 removed lines patch added patch discarded remove patch

Indentation +17 added lines, -17 removed lines patch added patch discarded remove patch

Indentation +26 added lines, -26 removed lines patch added patch discarded remove patch