Add and update tests

tests/BadConfigTest.php

@@ -0,0 +1,50 @@
+<?php
+/*
+ * This file is part of the OpxCore.
+ *
+ * Copyright (c) Lozovoy Vyacheslav <opxcore@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace OpxCore\Tests\Config;
+
+use OpxCore\Config\Config;
+use OpxCore\Config\Exceptions\ConfigException;
+use OpxCore\Tests\Config\Fixtures\BadRepo;
+use OpxCore\Tests\Config\Fixtures\CacheLoadException;
+use OpxCore\Tests\Config\Fixtures\CacheSaveException;
+use OpxCore\Tests\Config\Fixtures\Repo;
+use PHPUnit\Framework\TestCase;
+
+class BadConfigTest extends TestCase
+{
+    public function testBadRepo(): void
+    {
+        $config = new Config(new BadRepo());
+
+        $this->expectException(ConfigException::class);
+
+        $config->load();
+    }
+
+    public function testCacheReadError(): void
+    {
+        $repo = new Repo(['app' => ['name' => 'test']]);
+        $cache = new CacheLoadException();
+        $config = new Config($repo, $cache);
+
+        $this->expectException(ConfigException::class);
+        $config->load();
+    }
+    public function testCacheWriteError(): void
+    {
+        $repo = new Repo(['app' => ['name' => 'test']]);
+        $cache = new CacheSaveException();
+        $config = new Config($repo, $cache);
+
+        $this->expectException(ConfigException::class);
+        $config->load();
+    }
+}

tests/ConfigLoadTest.php

@@ -11,7 +11,6 @@
 namespace OpxCore\Tests\Config;
 
 use OpxCore\Config\Config;
-use OpxCore\Config\Exceptions\ConfigException;
 use OpxCore\Tests\Config\Fixtures\Cache;
 use OpxCore\Tests\Config\Fixtures\Repo;
 use OpxCore\Tests\Config\Fixtures\Env;
@@ -65,6 +64,22 @@ public function testCacheOkNoEnv(): void
         self::assertEquals('cached', $config->get('app.name'));
     }
 
+    /**
+     * Cache is ok, no env, load from cache
+     */
+    public function testCacheExpiredNoEnv(): void
+    {
+        $repo = new Repo(['app' => ['name' => 'test']]);
+        $cache = new Cache();
+        $cache->config = ['app' => ['name' => 'cached']];
+        $cache->expired = true;
+
+        $config = new Config($repo, $cache,);
+
+        self::assertTrue($config->load());
+        self::assertEquals('test', $config['app.name']);
+    }
+
     /**
      * Cache disabled by env, load from repo
      */

tests/ConfigTest.php

@@ -11,10 +11,7 @@
 namespace OpxCore\Tests\Config;
 
 use OpxCore\Config\Config;
-use OpxCore\Config\Exceptions\ConfigException;
-use OpxCore\Tests\Config\Fixtures\Cache;
 use OpxCore\Tests\Config\Fixtures\Repo;
-use OpxCore\Tests\Config\Fixtures\Env;
 use PHPUnit\Framework\TestCase;
 
 class ConfigTest extends TestCase

tests/Fixtures/BadRepo.php

@@ -0,0 +1,42 @@
+<?php
+/*
+ * This file is part of the OpxCore.
+ *
+ * Copyright (c) Lozovoy Vyacheslav <opxcore@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace OpxCore\Tests\Config\Fixtures;
+
+use Error;
+use Exception;
+use OpxCore\Config\Exceptions\ConfigRepositoryException;
+use OpxCore\Config\Interfaces\ConfigRepositoryInterface;
+
+class BadRepo implements ConfigRepositoryInterface
+{
+    /**
+     * @inheritDoc
+     * @throws ConfigRepositoryException
+     */
+    public function load(array &$config, $profile = null, $overrides = null): bool
+    {
+        try {
+            $config = require __DIR__ . DIRECTORY_SEPARATOR . 'bad.php';
+        } catch (Exception | Error $e) {
+            throw new ConfigRepositoryException("Error reading configuration file {$e->getFile()}:{$e->getLine()} {$e->getMessage()}");
+        }
+
+        return true;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function save(array $config, $profile = null): bool
+    {
+        return true;
+    }
+}

tests/Fixtures/CacheLoadException.php

@@ -0,0 +1,37 @@
+<?php
+/*
+ * This file is part of the OpxCore.
+ *
+ * Copyright (c) Lozovoy Vyacheslav <opxcore@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace OpxCore\Tests\Config\Fixtures;
+
+use OpxCore\Config\Exceptions\ConfigCacheException;
+use OpxCore\Config\Interfaces\ConfigCacheInterface;
+
+class CacheLoadException implements ConfigCacheInterface
+{
+    public array $config = [];
+
+    public bool $expired = false;
+    public ?string $profile = null;
+    public ?int $ttl = null;
+
+    public function load(array &$config, $profile = null): bool
+    {
+        throw new ConfigCacheException('Load rejected');
+    }
+
+    public function save(array $config, $profile = null, $ttl = null): bool
+    {
+        $this->config = $config;
+        $this->profile = $profile;
+        $this->ttl = $ttl;
+
+        return true;
+    }
+}

tests/Fixtures/CacheSaveException.php

@@ -0,0 +1,27 @@
+<?php
+/*
+ * This file is part of the OpxCore.
+ *
+ * Copyright (c) Lozovoy Vyacheslav <opxcore@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace OpxCore\Tests\Config\Fixtures;
+
+use OpxCore\Config\Exceptions\ConfigCacheException;
+use OpxCore\Config\Interfaces\ConfigCacheInterface;
+
+class CacheSaveException implements ConfigCacheInterface
+{
+    public function load(array &$config, $profile = null): bool
+    {
+        return false;
+    }
+
+    public function save(array $config, $profile = null, $ttl = null): bool
+    {
+        throw new ConfigCacheException('Save rejected');
+    }
+}

tests/Fixtures/bad.php

@@ -0,0 +1,14 @@
+<?php
+/*
+ * This file is part of the OpxCore.
+ *
+ * Copyright (c) Lozovoy Vyacheslav <opxcore@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+return [
+    'app' => 'bad',
+    e
+];