feat(log): configure log paths through env by default (#820)

Co-authored-by: Brent Roose <[email protected]>

Author: kevinfrom

.env.example

@@ -18,4 +18,8 @@ CONFIG_CACHE=false
 VIEW_CACHE=false
 
 # Enable or disable project cache (allround cache)
-PROJECT_CACHE=false
+PROJECT_CACHE=false
+
+# Overwrite default log paths (null = default)
+DEBUG_LOG_PATH=null
+SERVER_LOG_PATH=null

.gitignore

@@ -13,3 +13,4 @@ debug.log
 tempest.log
 public/static
 tests/Unit/Log
+log/

phpunit.xml.dist

@@ -33,6 +33,8 @@
     <env name="BASE_URI" value="" />
     <env name="CACHE" value="null" />
     <env name="DISCOVERY_CACHE" value="true" />
+    <env name="DEBUG_LOG_PATH" value="log/debug.test.log" />
+    <env name="SERVER_LOG_PATH" value="log/server.test.log" />
     <ini name="memory_limit" value="256M" />
   </php>
 </phpunit>

src/Tempest/Log/src/Config/logs.config.php

@@ -5,5 +5,9 @@
 namespace Tempest\Config;
 
 use Tempest\Log\LogConfig;
+use function Tempest\env;
 
-return new LogConfig();
+return new LogConfig(
+    debugLogPath: env('DEBUG_LOG_PATH'),
+    serverLogPath: env('SERVER_LOG_PATH'),
+);

tests/Integration/Log/LogConfigTest.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Log;
+
+use Tempest\Log\LogConfig;
+use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
+
+/**
+ * @internal
+ */
+final class LogConfigTest extends FrameworkIntegrationTestCase
+{
+    public function test_log_path_by_env(): void
+    {
+        $expectedDebugLogPath = 'log/debug.test.log';
+        $expectedServerLogPath = 'log/server.test.log';
+
+        $this->kernel->loadConfig();
+
+        $logConfig = $this->container->get(LogConfig::class);
+
+        $this->assertSame($expectedDebugLogPath, $logConfig->debugLogPath);
+        $this->assertSame($expectedServerLogPath, $logConfig->serverLogPath);
+    }
+}