remove unnecessary comments and phpunit.xml

Author: pratikshazalte69

phpunit.xml

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Neo4j\Neo4jBundle\Tests\unit\Collector;
+namespace Neo4j\Neo4jBundle\Tests\Unit\Collector;
 
 use Neo4j\Neo4jBundle\Collector\Neo4jDataCollector;
 use Neo4j\Neo4jBundle\EventListener\Neo4jProfileListener;
@@ -21,8 +21,6 @@ protected function setUp(): void
         $this->collector = new Neo4jDataCollector($this->subscriber);
     }
 
-
-
     public function testGetName(): void
     {
         $this->assertSame('neo4j', $this->collector->getName());
@@ -41,7 +39,10 @@ public function testGetQueryCount(): void
     public function testRecursiveToArray(): void
     {
         $obj = new class {
-            public function toArray(): array { return ['key' => 'value']; }
+            public function toArray(): array
+            {
+                return ['key' => 'value'];
+            }
         };
         $reflection = new \ReflectionClass($this->collector);
         $method = $reflection->getMethod('recursiveToArray');

tests/unit/Collector/Neo4jDataCollectorTest.php renamed to tests/Unit/Collector/Neo4jDataCollectorTest.php

@@ -29,22 +29,18 @@ class SymfonyClientTest extends TestCase
 
     protected function setUp(): void
     {
-        // Mocks
         $this->driverSetupManagerMock = $this->createMock(DriverSetupManager::class);
         $this->driverFactoryMock = $this->createMock(SymfonyDriverFactory::class);
         $this->sessionMock = $this->createMock(SymfonySession::class);
         $this->transactionMock = $this->createMock(SymfonyTransaction::class);
 
-        // Use real instances for final classes
         $this->sessionConfig = new SessionConfiguration();
         $this->transactionConfig = new TransactionConfiguration();
 
-        // Setup default alias return value
         $this->driverSetupManagerMock
             ->method('getDefaultAlias')
             ->willReturn('default');
 
-        // Create SymfonyClient instance
         $this->client = new SymfonyClient(
             $this->driverSetupManagerMock,
             $this->sessionConfig,

tests/unit/SymfonyClientTest.php renamed to tests/Unit/Decorators/SymfonyClientTest.php

@@ -1,16 +1,14 @@
 <?php
 
-namespace Neo4j\Neo4jBundle\Tests\unit;
-use Neo4j\Neo4jBundle\Decorators\SymfonyTransaction;
+namespace Neo4j\Neo4jBundle\Tests\Unit\Decorators;
 
 use Laudis\Neo4j\Basic\Session;
 use Laudis\Neo4j\Databags\Bookmark;
 use Laudis\Neo4j\Databags\Statement;
-use Laudis\Neo4j\Databags\TransactionConfiguration;
 use Laudis\Neo4j\Databags\SummarizedResult;
 use Laudis\Neo4j\Types\CypherList;
-use Laudis\Neo4j\Types\CypherMap;
 use Neo4j\Neo4jBundle\Decorators\SymfonySession;
+use Neo4j\Neo4jBundle\Decorators\SymfonyTransaction;
 use Neo4j\Neo4jBundle\EventHandler;
 use Neo4j\Neo4jBundle\Factories\SymfonyDriverFactory;
 use PHPUnit\Framework\TestCase;
@@ -26,12 +24,12 @@ class SymfonySessionTest extends TestCase
 
     protected function setUp(): void
     {
-        // Create mock objects
+
         $this->sessionMock = $this->createMock(Session::class);
         $this->handlerMock = $this->createMock(EventHandler::class);
         $this->factoryMock = $this->createMock(SymfonyDriverFactory::class);
 
-        // Initialize SymfonySession with mocks
+
         $this->symfonySession = new SymfonySession(
             $this->sessionMock,
             $this->handlerMock,
@@ -46,12 +44,12 @@ public function testRunStatement()
         $statementMock = $this->createMock(Statement::class);
         $resultMock = $this->createMock(SummarizedResult::class);
 
-        // Mock the EventHandler handleQuery method
+
         $this->handlerMock
             ->expects($this->once())
             ->method('handleQuery')
             ->with(
-                $this->callback(fn ($callback) => is_callable($callback)),
+                $this->callback(fn($callback) => is_callable($callback)),
                 $statementMock,
                 $this->alias,
                 $this->schema,
@@ -70,7 +68,7 @@ public function testRunStatements()
         $resultMock1 = $this->createMock(SummarizedResult::class);
         $resultMock2 = $this->createMock(SummarizedResult::class);
 
-        // Mock the handler for both statements
+
         $this->handlerMock
             ->method('handleQuery')
             ->willReturnOnConsecutiveCalls($resultMock1, $resultMock2);
@@ -85,13 +83,13 @@ public function testBeginTransaction()
     {
         $transactionMock = $this->createMock(SymfonyTransaction::class);
 
-        // Mock the factory to return a SymfonyTransaction
+
         $this->factoryMock
             ->expects($this->once())
             ->method('createTransaction')
             ->with(
                 $this->sessionMock,
-                $this->anything(), // TransactionConfiguration (nullable)
+                $this->anything(),
                 $this->alias,
                 $this->schema
             )
@@ -106,7 +104,6 @@ public function testWriteTransaction()
     {
         $transactionMock = $this->createMock(SymfonyTransaction::class);
 
-        // Mock the factory to return a SymfonyTransaction when beginTransaction() is called
         $this->factoryMock
             ->expects($this->once())
             ->method('createTransaction')

tests/unit/SymfonyDriverTest.php renamed to tests/Unit/Decorators/SymfonyDriverTest.php

@@ -2,14 +2,13 @@
 
 declare(strict_types=1);
 
-namespace Tests\Neo4j\Neo4jBundle\Decorators;
+namespace Tests\Neo4j\Neo4jBundle\tests\unit\Decorators;
 
 use Laudis\Neo4j\Contracts\UnmanagedTransactionInterface;
 use Laudis\Neo4j\Databags\Statement;
 use Laudis\Neo4j\Databags\SummarizedResult;
 use Laudis\Neo4j\Enum\TransactionState;
 use Laudis\Neo4j\Types\CypherList;
-use Laudis\Neo4j\Types\CypherMap;
 use Neo4j\Neo4jBundle\Decorators\SymfonyTransaction;
 use Neo4j\Neo4jBundle\EventHandler;
 use PHPUnit\Framework\TestCase;

tests/unit/SymfonySessionTest.php renamed to tests/Unit/Decorators/SymfonySessionTest.php

@@ -5,19 +5,17 @@
 namespace Neo4j\Neo4jBundle\Tests\Unit\DependencyInjection;
 
 use Neo4j\Neo4jBundle\DependencyInjection\Neo4jExtension;
-use Neo4j\Neo4jBundle\EventHandler;
-use Neo4j\Neo4jBundle\EventListener\Neo4jProfileListener;
-use Neo4j\Neo4jBundle\Collector\Neo4jDataCollector;
+
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Reference;
+
 
 class Neo4jExtensionTest extends TestCase
 {
     public function testLoad(): void
     {
         $container = new ContainerBuilder();
-        $container->setParameter('kernel.debug', false); // Define missing parameter
+        $container->setParameter('kernel.debug', false);
 
         $extension = new Neo4jExtension();
         $configs = [

tests/unit/SymfonyTransactionTest.php renamed to tests/Unit/Decorators/SymfonyTransactionTest.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Neo4j\Neo4jBundle\Tests\unit\Event;
+namespace Neo4j\Neo4jBundle\Tests\Unit\Event;
 
 use Laudis\Neo4j\Databags\Statement;
 use Laudis\Neo4j\Exception\Neo4jException;

tests/unit/DependencyInjection/ConfigurationTest.php renamed to tests/Unit/DependencyInjection/ConfigurationTest.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Neo4j\Neo4jBundle\Tests\unit\EventListener;
+namespace Neo4j\Neo4jBundle\Tests\Unit\EventListener;
 
 use Laudis\Neo4j\Databags\ResultSummary;
 use Laudis\Neo4j\Databags\Statement;
@@ -21,8 +21,8 @@ public function testOnPostRun(): void
 
         $alias = 'default';
         $resultMock = $this->createMock(ResultSummary::class);
-        $resultMock->method('getResultAvailableAfter')->willReturn(10.0); // Ensure float type
-        $resultMock->method('getResultConsumedAfter')->willReturn(5.0);   // Ensure float type
+        $resultMock->method('getResultAvailableAfter')->willReturn(10.0);
+        $resultMock->method('getResultConsumedAfter')->willReturn(5.0);
 
         $time = new \DateTimeImmutable();
         $scheme = 'bolt';

tests/unit/DependencyInjection/Neo4jExtensionTest.php renamed to tests/Unit/DependencyInjection/Neo4jExtensionTest.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Neo4j\Neo4jBundle\Tests\unit;
+namespace Neo4j\Neo4jBundle\Tests\Unit;
 
 use Laudis\Neo4j\Databags\Statement;
 use Laudis\Neo4j\Databags\SummarizedResult;

tests/unit/Event/FailureEventTest.php renamed to tests/Unit/Event/FailureEventTest.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Neo4j\Neo4jBundle\Tests\unit\Factories;
+namespace Neo4j\Neo4jBundle\Tests\Unit\Factories;
 
 use Neo4j\Neo4jBundle\Factories\StopwatchEventNameFactory;
 use Laudis\Neo4j\Enum\TransactionState;

tests/unit/EventListener/Neo4jProfileListenerTest.php renamed to tests/Unit/EventListener/Neo4jProfileListenerTest.php

@@ -92,6 +92,9 @@ public function testGenerateTransactionIdWithUuidFactory(): void
         $this->assertSame('test-uuid', $id);
     }
 
+    /**
+     * @throws \ReflectionException
+     */
     public function testGenerateTransactionIdWithoutUuidFactory(): void
     {
         $factory = new SymfonyDriverFactory($this->eventHandler, null);