Merge pull request #286 from infinum/change-test-variable

Change test variable
infinum · May 5, 2022 · f1ca5bf · f1ca5bf
2 parents d06ef94 + 9c9593a
commit f1ca5bf
Show file tree

Hide file tree

Showing 13 changed files with 23 additions and 23 deletions.
diff --git a/phpunit.xml b/phpunit.xml
@@ -10,7 +10,7 @@
 		</testsuite>
 	</testsuites>
 	<php>
-        <env name="TEST" value="true" force="true" />
+        <env name="ES_TEST" value="true" force="true" />
     </php>
 	<coverage processUncoveredFiles="true">
 		<include>

diff --git a/src/Blocks/AbstractBlocks.php b/src/Blocks/AbstractBlocks.php
@@ -396,7 +396,7 @@ private function getBlocksFolderPath(): string
 	{
 		$blocksPath = \dirname(__DIR__, 5) . self::PATH_BLOCKS_PARENT;
 
-		if (\getenv('TEST')) {
+		if (\getenv('ES_TEST')) {
 			$blocksPath = \dirname(__DIR__, 2) . self::PATH_BLOCKS_PARENT_TESTS;
 		}
 

diff --git a/src/Blocks/AbstractBlocksCli.php b/src/Blocks/AbstractBlocksCli.php
@@ -56,7 +56,7 @@ protected function blocksMove(array $assocArgs, string $outputDir, bool $isCompo
 			$path = "{$outputDir}/{$block}";
 			$sourcePath = "{$sourcePathFolder}{$block}";
 
-			if (!\getenv('TEST')) {
+			if (!\getenv('ES_TEST')) {
 				$destinationPath = $root . \DIRECTORY_SEPARATOR . $path;
 			} else {
 				$destinationPath = $this->getProjectRootPath(true) . '/cliOutput';

diff --git a/src/Blocks/BlockVariationCli.php b/src/Blocks/BlockVariationCli.php
@@ -86,7 +86,7 @@ public function __invoke(array $args, array $assocArgs)
 		$sourcePathFolder = $rootNode . $ds . static::OUTPUT_DIR . $ds;
 		$sourcePath = "{$sourcePathFolder}{$name}";
 
-		if (!\getenv('TEST')) {
+		if (!\getenv('ES_TEST')) {
 			$destinationPath = $root . $ds . $path;
 		} else {
 			$destinationPath = $this->getProjectRootPath(true) . '/cliOutput';

diff --git a/src/Blocks/BlockWrapperCli.php b/src/Blocks/BlockWrapperCli.php
@@ -65,7 +65,7 @@ public function __invoke(array $args, array $assocArgs)
 		$sourcePathFolder = $rootNode . $ds . static::OUTPUT_DIR . $ds;
 		$sourcePath = "{$sourcePathFolder}";
 
-		if (!\getenv('TEST')) {
+		if (!\getenv('ES_TEST')) {
 			$destinationPath = $root . $ds . $path;
 		} else {
 			$destinationPath = $this->getProjectRootPath(true) . '/cliOutput';

diff --git a/src/Cli/AbstractCli.php b/src/Cli/AbstractCli.php
@@ -345,7 +345,7 @@ public function outputWrite(string $outputDir, string $outputFile, array $args =
 	 */
 	public function getOutputDir(string $path = ''): string
 	{
-		if (\function_exists('\add_action') && !\getenv('TEST')) {
+		if (\function_exists('\add_action') && !\getenv('ES_TEST')) {
 			$root = $this->getProjectRootPath();
 		} else {
 			$root = $this->getProjectRootPath(true) . '/cliOutput';
@@ -396,7 +396,7 @@ public function renameNamespace(array $args = []): self
 		$namespace = $this->getNamespace($args);
 		$vendorPrefix = $this->getVendorPrefix($args);
 
-		if (\function_exists('\add_action') && !\getenv('TEST')) {
+		if (\function_exists('\add_action') && !\getenv('ES_TEST')) {
 			$output = \str_replace(
 				"namespace {$vendorPrefix}\EightshiftBoilerplate\\",
 				"namespace {$namespace}\\",
@@ -554,7 +554,7 @@ public function renameProjectName(array $args = []): self
 	{
 		$projectName = 'eightshift-boilerplate';
 
-		if (\function_exists('\add_action') && !\getenv('TEST')) {
+		if (\function_exists('\add_action') && !\getenv('ES_TEST')) {
 			$projectName = \basename(\dirname(__DIR__, 5));
 		}
 
@@ -796,7 +796,7 @@ public function getProjectRootPath(bool $isDev = false): string
 	{
 		$output = \dirname(__DIR__, 5);
 
-		if ($isDev || \getenv('TEST') !== false) {
+		if ($isDev || \getenv('ES_TEST') !== false) {
 			$output = \dirname(__DIR__, 2);
 		}
 
@@ -818,7 +818,7 @@ public function getProjectConfigRootPath(bool $isDev = false): string
 			$output = \dirname(__DIR__, 2);
 		}
 
-		if (\getenv('TEST')) {
+		if (\getenv('ES_TEST')) {
 			$output = \dirname(__DIR__, 2);
 		}
 
@@ -846,7 +846,7 @@ public function getFrontendLibsPath(string $path = ''): string
 	 */
 	public function getLibsPath(string $path = ''): string
 	{
-		if (\getenv('TEST')) {
+		if (\getenv('ES_TEST')) {
 			return "{$this->getProjectRootPath()}/{$path}";
 		}
 

diff --git a/src/Cli/CliInitAll.php b/src/Cli/CliInitAll.php
@@ -65,7 +65,7 @@ public function __invoke(array $args, array $assocArgs) // phpcs:ignore
 
 		WP_CLI::log('--------------------------------------------------');
 
-		if (!\getenv('TEST')) {
+		if (!\getenv('ES_TEST')) {
 			WP_CLI::log((string)shell_exec('npm run start')); // phpcs:ignore
 		}
 

diff --git a/src/Cli/CliInitProject.php b/src/Cli/CliInitProject.php
@@ -102,7 +102,7 @@ public function __invoke(array $args, array $assocArgs)
 
 		WP_CLI::log('--------------------------------------------------');
 
-		if (!\getenv('TEST')) {
+		if (!\getenv('ES_TEST')) {
 			WP_CLI::log((string)shell_exec('npm run start')); // phpcs:ignore
 		}
 

diff --git a/src/Cli/CliInitTheme.php b/src/Cli/CliInitTheme.php
@@ -95,7 +95,7 @@ public function __invoke(array $args, array $assocArgs)
 
 		WP_CLI::log('--------------------------------------------------');
 
-		if (!\getenv('TEST')) {
+		if (!\getenv('ES_TEST')) {
 			WP_CLI::log((string)shell_exec('npm run start')); // phpcs:ignore
 		}
 

diff --git a/src/Helpers/Components.php b/src/Helpers/Components.php
@@ -82,7 +82,7 @@ public static function render(string $component, array $attributes = [], string
 		if (empty($parentPath)) {
 			$parentPath = \dirname(__DIR__, 5);
 
-			if (\getenv('TEST')) {
+			if (\getenv('ES_TEST')) {
 				$parentPath = \dirname(__DIR__, 2);
 			}
 		}
@@ -107,7 +107,7 @@ public static function render(string $component, array $attributes = [], string
 		} else {
 			$blocksPath = AbstractBlocks::PATH_BLOCKS_PARENT;
 
-			if (\getenv('TEST')) {
+			if (\getenv('ES_TEST')) {
 				$blocksPath = AbstractBlocks::PATH_BLOCKS_PARENT_TESTS;
 			}
 

diff --git a/src/Manifest/AbstractManifest.php b/src/Manifest/AbstractManifest.php
@@ -37,7 +37,7 @@ abstract class AbstractManifest implements ServiceInterface, ManifestInterface
 	 */
 	public function setAssetsManifestRaw(): void
 	{
-		if (\defined('WP_CLI') && !\getenv('TEST')) {
+		if (\defined('WP_CLI') && !\getenv('ES_TEST')) {
 			return;
 		}
 
@@ -73,7 +73,7 @@ function ($manifestItem) {
 	 */
 	public function getAssetsManifestItem(string $key): string
 	{
-		if (\defined('WP_CLI') && !\getenv('TEST')) {
+		if (\defined('WP_CLI') && !\getenv('ES_TEST')) {
 			return '';
 		}
 

diff --git a/src/Setup/UpdateCli.php b/src/Setup/UpdateCli.php
@@ -82,7 +82,7 @@ public function __invoke(array $args, array $assocArgs)
 
 		$setupFilename = 'setup.json';
 
-		if (\getenv('TEST') !== false) {
+		if (\getenv('ES_TEST') !== false) {
 			$setupFilename = $this->getProjectConfigRootPath() . '/cliOutput/setup.json';
 		}
 

diff --git a/tests/Cli/AbstractCliTest.php b/tests/Cli/AbstractCliTest.php
@@ -193,13 +193,13 @@ public function getDoc(): array
 	$abstractMock = new AbstractTest('test');
 
 	// The test one is covered. Let's see if we can get the one that will be used
-	putenv('TEST');
+	putenv('ES_TEST');
 	$output = $abstractMock->getLibsPath('test');
 
 	$this->assertIsString($output);
 	$this->assertStringContainsString('/vendor/infinum/eightshift-libs/test', $output);
 
-	putenv('TEST=1');
+	putenv('ES_TEST=1');
 });
 
 
@@ -213,15 +213,15 @@ public function getDoc(): array
 
 
 test('Project config root path works if dev is false', function() {
-	putenv('TEST');
+	putenv('ES_TEST');
 	$abstractMock = new AbstractTest('test');
 
 	$outputProd = $abstractMock->getProjectConfigRootPath(false);
 
 	$this->assertIsString($outputProd);
 	$this->assertSame('/', $outputProd);
 
-	putenv('TEST=1');
+	putenv('ES_TEST=1');
 });