Refactor: Add 'final' to classes in the 'tests' folder and other minor improvements

walmir-silva · walmir-silva · commit 39235ab2f62a · 2024-09-19T14:22:55.000-03:00
diff --git a/tests/Decorator/AsyncLoggerTest.php b/tests/Decorator/AsyncLoggerTest.php
@@ -10,7 +10,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class AsyncLoggerTest extends TestCase
+final class AsyncLoggerTest extends TestCase
 {
     private AsyncLogger $asyncLogger;
     private Logger|MockObject $logger;
diff --git a/tests/Decorator/BaseLoggerDecoratorTest.php b/tests/Decorator/BaseLoggerDecoratorTest.php
@@ -10,7 +10,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class BaseLoggerDecoratorTest extends TestCase
+final class BaseLoggerDecoratorTest extends TestCase
 {
     private BaseLoggerDecorator $baseLoggerDecorator;
     private Logger|MockObject $logger;
diff --git a/tests/Decorator/ContextualLoggerTest.php b/tests/Decorator/ContextualLoggerTest.php
@@ -10,7 +10,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class ContextualLoggerTest extends TestCase
+final class ContextualLoggerTest extends TestCase
 {
     private ContextualLogger $contextualLogger;
     private Logger|MockObject $logger;
diff --git a/tests/Exception/LoggingExceptionTest.php b/tests/Exception/LoggingExceptionTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Exception\LoggingException;
 use PHPUnit\Framework\TestCase;
 
-class LoggingExceptionTest extends TestCase
+final class LoggingExceptionTest extends TestCase
 {
     public function testExceptionMessage(): void
     {
diff --git a/tests/Formatter/AbstractFormatterTest.php b/tests/Formatter/AbstractFormatterTest.php
@@ -8,7 +8,7 @@
 use KaririCode\Logging\Formatter\AbstractFormatter;
 use PHPUnit\Framework\TestCase;
 
-class AbstractFormatterTest extends TestCase
+final class AbstractFormatterTest extends TestCase
 {
     public function testGetFormatter(): void
     {
diff --git a/tests/Formatter/ConsoleColorFormatterTest.php b/tests/Formatter/ConsoleColorFormatterTest.php
@@ -8,7 +8,7 @@
 use KaririCode\Logging\LogLevel;
 use PHPUnit\Framework\TestCase;
 
-class ConsoleColorFormatterTest extends TestCase
+final class ConsoleColorFormatterTest extends TestCase
 {
     private ConsoleColorFormatter $formatter;
 
diff --git a/tests/Formatter/ElasticFormatterTest.php b/tests/Formatter/ElasticFormatterTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class ElasticFormatterTest extends TestCase
+final class ElasticFormatterTest extends TestCase
 {
     private ElasticFormatter $formatter;
 
diff --git a/tests/Formatter/JsonFormatterTest.php b/tests/Formatter/JsonFormatterTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class JsonFormatterTest extends TestCase
+final class JsonFormatterTest extends TestCase
 {
     private JsonFormatter $jsonFormatter;
 
diff --git a/tests/Formatter/LineFormatterTest.php b/tests/Formatter/LineFormatterTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class LineFormatterTest extends TestCase
+final class LineFormatterTest extends TestCase
 {
     private LineFormatter $formatter;
 
diff --git a/tests/Handler/AbstractHandlerTest.php b/tests/Handler/AbstractHandlerTest.php
@@ -12,15 +12,15 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class ConcreteHandler extends AbstractHandler
+final class ConcreteHandler extends AbstractHandler
 {
     public function handle(ImmutableValue $record): void
     {
         // Implementação de exemplo para o método abstrato
     }
 }
 
-class AbstractHandlerTest extends TestCase
+final class AbstractHandlerTest extends TestCase
 {
     private ConcreteHandler $handler;
 
diff --git a/tests/Handler/ConsoleHandlerTest.php b/tests/Handler/ConsoleHandlerTest.php
@@ -13,7 +13,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class ConsoleHandlerTest extends TestCase
+final class ConsoleHandlerTest extends TestCase
 {
     private mixed $outputMock;
     private LogFormatter|MockObject $formatterMock;
diff --git a/tests/Handler/FileHandlerTest.php b/tests/Handler/FileHandlerTest.php
@@ -11,7 +11,7 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class FileHandlerTest extends TestCase
+final class FileHandlerTest extends TestCase
 {
     private string $testLogDir;
     private string $testLogFile;
diff --git a/tests/Handler/NullHandlerTest.php b/tests/Handler/NullHandlerTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\LogRecord;
 use PHPUnit\Framework\TestCase;
 
-class NullHandlerTest extends TestCase
+final class NullHandlerTest extends TestCase
 {
     private NullHandler $nullHandler;
 
diff --git a/tests/Handler/SlackHandlerTest.php b/tests/Handler/SlackHandlerTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class SlackHandlerTest extends TestCase
+final class SlackHandlerTest extends TestCase
 {
     private SlackClient|MockObject $slackClient;
     private LogFormatter|MockObject $formatter;
diff --git a/tests/Handler/SyslogUdpHandlerTest.php b/tests/Handler/SyslogUdpHandlerTest.php
@@ -13,7 +13,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class SyslogUdpHandlerTest extends TestCase
+final class SyslogUdpHandlerTest extends TestCase
 {
     private string $host = '127.0.0.1';
     private int $port = 514;
diff --git a/tests/Logger/LoggerBuilderTest.php b/tests/Logger/LoggerBuilderTest.php
@@ -11,7 +11,7 @@
 use KaririCode\Logging\LoggerBuilder;
 use PHPUnit\Framework\TestCase;
 
-class LoggerBuilderTest extends TestCase
+final class LoggerBuilderTest extends TestCase
 {
     public function testBuildLogger(): void
     {
diff --git a/tests/Logger/LoggerConfigurationTest.php b/tests/Logger/LoggerConfigurationTest.php
@@ -8,7 +8,7 @@
 use KaririCode\Logging\LoggerConfiguration;
 use PHPUnit\Framework\TestCase;
 
-class LoggerConfigurationTest extends TestCase
+final class LoggerConfigurationTest extends TestCase
 {
     private LoggerConfiguration $config;
 
diff --git a/tests/Logger/LoggerFactoryTest.php b/tests/Logger/LoggerFactoryTest.php
@@ -15,7 +15,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class LoggerFactoryTest extends TestCase
+final class LoggerFactoryTest extends TestCase
 {
     private LoggerConfiguration|MockObject $config;
     private LoggerHandlerFactory|MockObject $handlerFactory;
diff --git a/tests/Logger/LoggerManagerTest.php b/tests/Logger/LoggerManagerTest.php
@@ -10,14 +10,13 @@
 use KaririCode\Contract\Logging\Structural\ProcessorAware;
 use KaririCode\Logging\Formatter\LineFormatter;
 use KaririCode\Logging\Handler\AbstractHandler;
-use KaririCode\Logging\Handler\ConsoleHandler;
 use KaririCode\Logging\LoggerManager;
 use KaririCode\Logging\LogLevel;
 use KaririCode\Logging\LogRecord;
 use KaririCode\Logging\Processor\AbstractProcessor;
 use PHPUnit\Framework\TestCase;
 
-class LoggerManagerTest extends TestCase
+final class LoggerManagerTest extends TestCase
 {
     private LoggerManager $loggerManager;
 
diff --git a/tests/Metric/MetricsCollectorTest.php b/tests/Metric/MetricsCollectorTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Metric\MetricsCollector;
 use PHPUnit\Framework\TestCase;
 
-class MetricsCollectorTest extends TestCase
+final class MetricsCollectorTest extends TestCase
 {
     private MetricsCollector $metricsCollector;
 
diff --git a/tests/Metric/MonitorTest.php b/tests/Metric/MonitorTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Metric\Monitor;
 use PHPUnit\Framework\TestCase;
 
-class MonitorTest extends TestCase
+final class MonitorTest extends TestCase
 {
     private Monitor $monitor;
 
diff --git a/tests/Processor/AbstractProcessorTest.php b/tests/Processor/AbstractProcessorTest.php
@@ -10,23 +10,7 @@
 use KaririCode\Logging\Processor\AbstractProcessor;
 use PHPUnit\Framework\TestCase;
 
-// Classe concreta para testar AbstractProcessor
-class ConcreteTestProcessor extends AbstractProcessor
-{
-    public function process(ImmutableValue $record): ImmutableValue
-    {
-        // Implementação simples para teste
-        return $record;
-    }
-
-    // Expõe o método protegido para teste
-    public function testHasValidContext(array $context): bool
-    {
-        return $this->hasValidContext($context);
-    }
-}
-
-class AbstractProcessorTest extends TestCase
+final class AbstractProcessorTest extends TestCase
 {
     private ConcreteTestProcessor $processor;
 
@@ -96,3 +80,20 @@ public function testProcessMethod(): void
         $this->assertSame($mockRecord, $result);
     }
 }
+
+
+
+final class ConcreteTestProcessor extends AbstractProcessor
+{
+    public function process(ImmutableValue $record): ImmutableValue
+    {
+        // Implementação simples para teste
+        return $record;
+    }
+
+    // Expõe o método protegido para teste
+    public function testHasValidContext(array $context): bool
+    {
+        return $this->hasValidContext($context);
+    }
+}
diff --git a/tests/Processor/AsyncLogProcessorTest.php b/tests/Processor/AsyncLogProcessorTest.php
@@ -11,7 +11,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class AsyncLogProcessorTest extends TestCase
+final class AsyncLogProcessorTest extends TestCase
 {
     private AsyncLogProcessor $asyncLogProcessor;
     private Logger|MockObject $logger;
diff --git a/tests/Processor/BulkProcessorTest.php b/tests/Processor/BulkProcessorTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\Processor\BulkProcessor;
 use PHPUnit\Framework\TestCase;
 
-class BulkProcessorTest extends TestCase
+final class BulkProcessorTest extends TestCase
 {
     private BulkProcessor $bulkProcessor;
     private array $flushedRecords;
diff --git a/tests/Processor/MemoryUsageProcessorTest.php b/tests/Processor/MemoryUsageProcessorTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\Processor\Metric\MemoryUsageProcessor;
 use PHPUnit\Framework\TestCase;
 
-class MemoryUsageProcessorTest extends TestCase
+final class MemoryUsageProcessorTest extends TestCase
 {
     private MemoryUsageProcessor $memoryUsageProcessor;
 
diff --git a/tests/Processor/WebProcessorTest.php b/tests/Processor/WebProcessorTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\Processor\WebProcessor;
 use PHPUnit\Framework\TestCase;
 
-class WebProcessorTest extends TestCase
+final class WebProcessorTest extends TestCase
 {
     private WebProcessor $webProcessor;
 
diff --git a/tests/Resilience/CircuitBreakerTest.php b/tests/Resilience/CircuitBreakerTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Resilience\CircuitBreaker;
 use PHPUnit\Framework\TestCase;
 
-class CircuitBreakerTest extends TestCase
+final class CircuitBreakerTest extends TestCase
 {
     private CircuitBreaker $circuitBreaker;
 
diff --git a/tests/Resilience/FallbackTest.php b/tests/Resilience/FallbackTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Resilience\Fallback;
 use PHPUnit\Framework\TestCase;
 
-class FallbackTest extends TestCase
+final class FallbackTest extends TestCase
 {
     public function testPrimaryOperationSucceeds(): void
     {
diff --git a/tests/Resilience/RateLimiterTest.php b/tests/Resilience/RateLimiterTest.php
@@ -9,7 +9,7 @@
 use KaririCode\Logging\Resilience\RateLimiter;
 use PHPUnit\Framework\TestCase;
 
-class RateLimiterTest extends TestCase
+final class RateLimiterTest extends TestCase
 {
     private RateLimiter $rateLimiter;
 
diff --git a/tests/Resilience/RetryTest.php b/tests/Resilience/RetryTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Resilience\Retry;
 use PHPUnit\Framework\TestCase;
 
-class RetryTest extends TestCase
+final class RetryTest extends TestCase
 {
     public function testExecuteHappyPath(): void
     {
diff --git a/tests/Rotetor/SizeBasedRotatorTest.php b/tests/Rotetor/SizeBasedRotatorTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Rotator\SizeBasedRotator;
 use PHPUnit\Framework\TestCase;
 
-class SizeBasedRotatorTest extends TestCase
+final class SizeBasedRotatorTest extends TestCase
 {
     private string $tempDir;
     private string $logFile;
diff --git a/tests/Security/AnonymizerTest.php b/tests/Security/AnonymizerTest.php
@@ -8,7 +8,7 @@
 use KaririCode\Logging\Security\Anonymizer;
 use PHPUnit\Framework\TestCase;
 
-class AnonymizerTest extends TestCase
+final class AnonymizerTest extends TestCase
 {
     private Anonymizer $anonymizer;
 
diff --git a/tests/Security/EncryptorTest.php b/tests/Security/EncryptorTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Security\Encryptor;
 use PHPUnit\Framework\TestCase;
 
-class EncryptorTest extends TestCase
+final class EncryptorTest extends TestCase
 {
     private Encryptor $encryptor;
 
diff --git a/tests/Security/MutexTest.php b/tests/Security/MutexTest.php
@@ -8,7 +8,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class MutexTest extends TestCase
+final class MutexTest extends TestCase
 {
     private Mutex|MockObject $mutex;
 
diff --git a/tests/Security/SimpleMutexTest.php b/tests/Security/SimpleMutexTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Security\SimpleMutex;
 use PHPUnit\Framework\TestCase;
 
-class SimpleMutexTest extends TestCase
+final class SimpleMutexTest extends TestCase
 {
     private SimpleMutex $simpleMutex;
 
diff --git a/tests/Security/ThreadSafeTest.php b/tests/Security/ThreadSafeTest.php
@@ -9,7 +9,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class ThreadSafeTest extends TestCase
+final class ThreadSafeTest extends TestCase
 {
     private ThreadSafe $threadSafe;
     private Mutex|MockObject $mutex;
diff --git a/tests/Service/LoggerServiceProviderTest.php b/tests/Service/LoggerServiceProviderTest.php
@@ -13,7 +13,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
-class LoggerServiceProviderTest extends TestCase
+final class LoggerServiceProviderTest extends TestCase
 {
     private LoggerConfiguration|MockObject $config;
     private LoggerFactory|MockObject $loggerFactory;
diff --git a/tests/Tracing/DistributedTracingTest.php b/tests/Tracing/DistributedTracingTest.php
@@ -7,7 +7,7 @@
 use KaririCode\Logging\Tracing\DistributedTracing;
 use PHPUnit\Framework\TestCase;
 
-class DistributedTracingTest extends TestCase
+final class DistributedTracingTest extends TestCase
 {
     private DistributedTracing $distributedTracing;
 
diff --git a/tests/Util/ContextPropagatorTest.php b/tests/Util/ContextPropagatorTest.php
diff --git a/tests/Util/CurlClientTest.php b/tests/Util/CurlClientTest.php
diff --git a/tests/Util/SamplerTest.php b/tests/Util/SamplerTest.php
diff --git a/tests/Util/SerializerTest.php b/tests/Util/SerializerTest.php

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`use PHPUnit\Framework\MockObject\MockObject;`
`11`	`11`	`use PHPUnit\Framework\TestCase;`
`12`	`12`
`13`		`-class AsyncLoggerTest extends TestCase`
	`13`	`+final class AsyncLoggerTest extends TestCase`
`14`	`14`	`{`
`15`	`15`	`private AsyncLogger $asyncLogger;`
`16`	`16`	`private Logger\|MockObject $logger;`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`use KaririCode\Logging\Exception\LoggingException;`
`8`	`8`	`use PHPUnit\Framework\TestCase;`
`9`	`9`
`10`		`-class LoggingExceptionTest extends TestCase`
	`10`	`+final class LoggingExceptionTest extends TestCase`
`11`	`11`	`{`
`12`	`12`	`public function testExceptionMessage(): void`
`13`	`13`	`{`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`use KaririCode\Logging\Formatter\AbstractFormatter;`
`9`	`9`	`use PHPUnit\Framework\TestCase;`
`10`	`10`
`11`		`-class AbstractFormatterTest extends TestCase`
	`11`	`+final class AbstractFormatterTest extends TestCase`
`12`	`12`	`{`
`13`	`13`	`public function testGetFormatter(): void`
`14`	`14`	`{`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`use KaririCode\Logging\LogLevel;`
`9`	`9`	`use PHPUnit\Framework\TestCase;`
`10`	`10`
`11`		`-class ConsoleColorFormatterTest extends TestCase`
	`11`	`+final class ConsoleColorFormatterTest extends TestCase`
`12`	`12`	`{`
`13`	`13`	`private ConsoleColorFormatter $formatter;`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`use KaririCode\Logging\LogRecord;`
`10`	`10`	`use PHPUnit\Framework\TestCase;`
`11`	`11`
`12`		`-class ElasticFormatterTest extends TestCase`
	`12`	`+final class ElasticFormatterTest extends TestCase`
`13`	`13`	`{`
`14`	`14`	`private ElasticFormatter $formatter;`
`15`	`15`
Original file line number	Diff line number	Diff line change
`@@ -12,15 +12,15 @@`
`12`	`12`	`use KaririCode\Logging\LogRecord;`
`13`	`13`	`use PHPUnit\Framework\TestCase;`
`14`	`14`
`15`		`-class ConcreteHandler extends AbstractHandler`
	`15`	`+final class ConcreteHandler extends AbstractHandler`
`16`	`16`	`{`
`17`	`17`	`public function handle(ImmutableValue $record): void`
`18`	`18`	`{`
`19`	`19`	`// Implementação de exemplo para o método abstrato`
`20`	`20`	`}`
`21`	`21`	`}`
`22`	`22`
`23`		`-class AbstractHandlerTest extends TestCase`
	`23`	`+final class AbstractHandlerTest extends TestCase`
`24`	`24`	`{`
`25`	`25`	`private ConcreteHandler $handler;`
`26`	`26`