]> git.mxchange.org Git - friendica.git/commitdiff
Clean static private properties of LoggerManager after test
authorArt4 <art4@wlabs.de>
Mon, 13 Jan 2025 16:41:35 +0000 (16:41 +0000)
committerArt4 <art4@wlabs.de>
Mon, 13 Jan 2025 16:41:35 +0000 (16:41 +0000)
tests/Unit/Core/Logger/LoggerManagerTest.php

index 14db1c267644170835897d2fb72861d2c8f41889..0dae2fcc6c8f650707cf4a719ff253e7ee9bb86e 100644 (file)
@@ -23,30 +23,30 @@ class LoggerManagerTest extends TestCase
 {
        public function testGetLoggerReturnsPsrLogger(): void
        {
-               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
-               $reflectionProperty->setAccessible(true);
-               $reflectionProperty->setValue(null, null);
-
                $factory = new LoggerManager(
                        $this->createStub(IManageConfigValues::class),
                        $this->createStub(LoggerFactory::class)
                );
 
                $this->assertInstanceOf(LoggerInterface::class, $factory->getLogger());
-       }
 
-       public function testGetLoggerReturnsSameObject(): void
-       {
                $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
                $reflectionProperty->setAccessible(true);
                $reflectionProperty->setValue(null, null);
+       }
 
+       public function testGetLoggerReturnsSameObject(): void
+       {
                $factory = new LoggerManager(
                        $this->createStub(IManageConfigValues::class),
                        $this->createStub(LoggerFactory::class)
                );
 
                $this->assertSame($factory->getLogger(), $factory->getLogger());
+
+               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
+               $reflectionProperty->setAccessible(true);
+               $reflectionProperty->setValue(null, null);
        }
 
        public function testGetLoggerWithDebugDisabledReturnsNullLogger(): void
@@ -56,16 +56,16 @@ class LoggerManagerTest extends TestCase
                        ['system', 'debugging', null, false],
                ]);
 
-               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
-               $reflectionProperty->setAccessible(true);
-               $reflectionProperty->setValue(null, null);
-
                $factory = new LoggerManager(
                        $config,
                        $this->createStub(LoggerFactory::class)
                );
 
                $this->assertInstanceOf(NullLogger::class, $factory->getLogger());
+
+               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
+               $reflectionProperty->setAccessible(true);
+               $reflectionProperty->setValue(null, null);
        }
 
        public function testGetLoggerWithProfilerEnabledReturnsProfilerLogger(): void
@@ -76,16 +76,16 @@ class LoggerManagerTest extends TestCase
                        ['system', 'profiling', null, true],
                ]);
 
-               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
-               $reflectionProperty->setAccessible(true);
-               $reflectionProperty->setValue(null, null);
-
                $factory = new LoggerManager(
                        $config,
                        $this->createStub(LoggerFactory::class)
                );
 
                $this->assertInstanceOf(ProfilerLogger::class, $factory->getLogger());
+
+               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
+               $reflectionProperty->setAccessible(true);
+               $reflectionProperty->setValue(null, null);
        }
 
        public function testChangeLogChannelReturnsDifferentLogger(): void
@@ -96,10 +96,6 @@ class LoggerManagerTest extends TestCase
                        ['system', 'profiling', null, true],
                ]);
 
-               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
-               $reflectionProperty->setAccessible(true);
-               $reflectionProperty->setValue(null, null);
-
                $factory = new LoggerManager(
                        $config,
                        $this->createStub(LoggerFactory::class)
@@ -110,6 +106,10 @@ class LoggerManagerTest extends TestCase
                $factory->changeLogChannel(LogChannel::CONSOLE);
 
                $this->assertNotSame($logger1, $factory->getLogger());
+
+               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
+               $reflectionProperty->setAccessible(true);
+               $reflectionProperty->setValue(null, null);
        }
 
        public function testChangeLogChannelToWorkerReturnsWorkerLogger(): void
@@ -120,10 +120,6 @@ class LoggerManagerTest extends TestCase
                        ['system', 'profiling', null, true],
                ]);
 
-               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
-               $reflectionProperty->setAccessible(true);
-               $reflectionProperty->setValue(null, null);
-
                $factory = new LoggerManager(
                        $config,
                        $this->createStub(LoggerFactory::class)
@@ -132,5 +128,9 @@ class LoggerManagerTest extends TestCase
                $factory->changeLogChannel(LogChannel::WORKER);
 
                $this->assertInstanceOf(WorkerLogger::class, $factory->getLogger());
+
+               $reflectionProperty = new \ReflectionProperty(LoggerManager::class, 'logger');
+               $reflectionProperty->setAccessible(true);
+               $reflectionProperty->setValue(null, null);
        }
 }