Merge pull request #41 from tattersoftware/config

Test Config

Author: MGatner

src/Asset.php

@@ -2,6 +2,7 @@
 
 namespace Tatter\Assets;
 
+use CodeIgniter\Config\Factories;
 use CodeIgniter\Files\File;
 use Tatter\Assets\Config\Assets as AssetsConfig;
 use Tatter\Assets\Exceptions\AssetsException;
@@ -76,6 +77,9 @@ public static function config(): AssetsConfig
     public static function useConfig(?AssetsConfig $config)
     {
         self::$config = $config;
+
+        // If a new config was supplied then use it with Factories, otherwise reset to the "vanilla" version
+        Factories::injectMock('config', 'Assets', $config ?? new AssetsConfig());
     }
 
     //--------------------------------------------------------------------

src/Test/AssetsTestTrait.php

@@ -45,7 +45,7 @@ protected function setUpAssetsTestTrait(): void
         }
 
         // Create the config
-        $this->config                = config('Assets');
+        $this->config                = new AssetsConfig();
         $this->config->directory     = $this->root->url() . DIRECTORY_SEPARATOR;
         $this->config->useTimestamps = false; // These make testing much harder
 
@@ -64,6 +64,8 @@ protected function tearDownAssetsTestTrait(): void
             $this->root      = null;
             $this->published = false;
         }
+
+        Asset::useConfig(null);
     }
 
     /**

tests/AssetTest.php

@@ -34,7 +34,7 @@ public function testUsesConfig()
         $result = Asset::config();
 
         $this->assertSame($config, $result);
-        $this->assertNotSame(config(AssetsConfig::class), $result);
+        $this->assertSame(config(AssetsConfig::class), $result);
     }
 
     public function testLoadsConfig()