Use getenv() to retrieve test testup factory (ezsystems#2397)

ezecosystem · Jul 17, 2018 · 67603a3 · 67603a3
1 parent b0c9201
commit 67603a3
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/eZ/Publish/API/Repository/Tests/BaseTest.php b/eZ/Publish/API/Repository/Tests/BaseTest.php
@@ -160,17 +160,18 @@ protected function getRepository($initialInitializeFromScratch = true)
     protected function getSetupFactory()
     {
         if (null === $this->setupFactory) {
-            if (false === isset($_ENV['setupFactory'])) {
+            $setupClass = getenv('setupFactory');
+
+            if (false === $setupClass) {
                 throw new \ErrorException(
-                    'Missing mandatory setting $_ENV["setupFactory"], this should normally be set in the relevant phpunit-integration-*.xml file and refer to a setupFactory for the given StorageEngine/SearchEngine in use'
+                    'Missing mandatory environment variable "setupFactory", this should normally be set in the relevant phpunit-integration-*.xml file and refer to a setupFactory for the given StorageEngine/SearchEngine in use'
                 );
             }
 
-            $setupClass = $_ENV['setupFactory'];
             if (false === class_exists($setupClass)) {
                 throw new \ErrorException(
                     sprintf(
-                        '$_ENV["setupFactory"] does not reference an existing class: %s. Did you forget to install an package dependency?',
+                        'Environment variable "setupFactory" does not reference an existing class: %s. Did you forget to install an package dependency?',
                         $setupClass
                     )
                 );