winp

Author: Pratiksha

.gitignore

@@ -11,7 +11,7 @@ test
 .phpunit.result.cache
 
 #PHP-CS-FIXER
-.php-cs-fixer.php
+
 .php-cs-fixer.cache
 
 

src/BasicAuthentication.php

@@ -11,7 +11,6 @@ class BasicAuthentication implements AuthenticateInterface
 
     public function __construct(?string $username = null, ?string $password = null)
     {
-        // Use provided values or fallback to environment variables
         $this->username = $username ?? getenv("NEO4J_USERNAME") ?: '';
         $this->password = $password ?? getenv("NEO4J_PASSWORD") ?: '';
     }

src/Objects/Person.php

@@ -15,7 +15,7 @@ class Person extends Node
      */
     public function __construct(array $properties)
     {
-        // Pass the label 'Person' along with the properties to the parent Node constructor.
+
         parent::__construct(['Person'], $properties);
     }
 }

tests/Integration/Neo4jOGMTest.php

@@ -385,10 +385,6 @@ public function testWithPath()
         $this->assertEquals($expectedResult, $actual);
     }
 
-
-
-
-
     #[DataProvider('nullDataProvider')]
     public function testWithNull(string $query, array $parameters, ?string $expectedResult): void
     {

tests/Integration/Neo4jQueryAPIIntegrationTest.php

@@ -182,23 +182,17 @@ public function testProfileCreateKnowsBidirectionalRelationshipsMock(): void
 
         $handler = HandlerStack::create($mockSack);
         $client = new Client(['handler' => $handler]);
-
-        // Use environment variables for authentication
         $auth = Authentication::fromEnvironment();
 
-        // Pass both client and authentication to Neo4jQueryAPI
         $api = new Neo4jQueryAPI($client, $auth);
 
-        // Execute the query
         $result = $api->run($query);
 
         $plan = $result->getProfiledQueryPlan();
         $this->assertNotNull($plan, "The result of the query should not be null.");
 
-        // Load expected data
         $expected = require __DIR__ . '/../resources/expected/complex-query-profile.php';
 
-        // Assert the profiled query plan matches the expected result
         $this->assertEquals($expected->getProfiledQueryPlan(), $plan, "Profiled query plan does not match the expected value.");
     }
 

tests/Integration/Neo4jTransactionIntegrationTest.php

@@ -53,27 +53,19 @@ private function populateTestData(): void
     }
     public function testTransactionCommit(): void
     {
-        // Begin a new transaction
+
         $tsx = $this->api->beginTransaction();
 
-        // Generate a random name for the node
         $name = (string)mt_rand(1, 100000);
-
-        // Create a node within the transaction
         $tsx->run("CREATE (x:Human {name: \$name})", ['name' => $name]);
 
-        // Validate that the node does not exist in the database before the transaction is committed
         $results = $this->api->run("MATCH (x:Human {name: \$name}) RETURN x", ['name' => $name]);
         $this->assertCount(0, $results);
 
-        // Validate that the node exists within the transaction
         $results = $tsx->run("MATCH (x:Human {name: \$name}) RETURN x", ['name' => $name]);
         $this->assertCount(1, $results);
 
-        // Commit the transaction
         $tsx->commit();
-
-        // Validate that the node now exists in the database
         $results = $this->api->run("MATCH (x:Human {name: \$name}) RETURN x", ['name' => $name]);
         $this->assertCount(1, $results); // Updated to expect 1 result
     }