Fixed incorrect testing vars and reset env vars in config test

diff --git a/phpunit.xml b/phpunit.xml
index 0e51f6af16b375b92fd612353a29306a4e06feef..53722a71b4e193a9a1949322c3581bfacc293c08 100644 (file)
--- a/phpunit.xml
+++ b/phpunit.xml
@@ -34,8 +34,8 @@
         <env name="AVATAR_URL" value=""/>
         <env name="LDAP_VERSION" value="3"/>
         <env name="STORAGE_TYPE" value="local"/>
-        <env name="ATTACHMENT_STORAGE_TYPE" value="local"/>
-        <env name="IMAGE_STORAGE_TYPE" value="local"/>
+        <env name="STORAGE_ATTACHMENT_TYPE" value="local"/>
+        <env name="STORAGE_IMAGE_TYPE" value="local"/>
         <env name="GITHUB_APP_ID" value="aaaaaaaaaaaaaa"/>
         <env name="GITHUB_APP_SECRET" value="aaaaaaaaaaaaaa"/>
         <env name="GITHUB_AUTO_REGISTER" value=""/>

diff --git a/tests/Unit/ConfigTest.php b/tests/Unit/ConfigTest.php
index 45f7b079d92c2af132ecfe9b21cef38bba462dde..967915af97a8560ef7310fd86853aec01afbe2cf 100644 (file)
--- a/tests/Unit/ConfigTest.php
+++ b/tests/Unit/ConfigTest.php
@@ -16,6 +16,8 @@ class ConfigTest extends TestCase
 
         $this->checkEnvConfigResult('STORAGE_IMAGE_TYPE', 's3', 'filesystems.images', 's3');
         $this->checkEnvConfigResult('STORAGE_IMAGE_TYPE', null, 'filesystems.images', 'local_secure');
+
+        putenv('STORAGE_TYPE=local');
     }
 
     public function test_filesystem_attachments_falls_back_to_storage_type_var()
@@ -24,6 +26,8 @@ class ConfigTest extends TestCase
 
         $this->checkEnvConfigResult('STORAGE_ATTACHMENT_TYPE', 's3', 'filesystems.attachments', 's3');
         $this->checkEnvConfigResult('STORAGE_ATTACHMENT_TYPE', null, 'filesystems.attachments', 'local_secure');
+
+        putenv('STORAGE_TYPE=local');
     }
 
     public function test_app_url_blank_if_old_default_value()
@@ -45,10 +49,12 @@ class ConfigTest extends TestCase
      */
     protected function checkEnvConfigResult(string $envName, $envVal, string $configKey, string $expectedResult)
     {
+        $originalVal = getenv($envName);
         $envString = $envName . (is_null($envVal) ? '' : '=') . ($envVal ?? '');
         putenv($envString);
         $this->refreshApplication();
         $this->assertEquals($expectedResult, config($configKey));
+        putenv($envString = $envName . (empty($originalVal) ? '' : '=') . ($originalVal ?? ''));
     }
 
 }
\ No newline at end of file