- Fixed improper passing of collection registration from a subquery t…

…o its parent. - Correct tests namespacing
LaravelFreelancerNL · Sep 4, 2021 · 5a2c4b6 · 5a2c4b6
1 parent 5be1f57
commit 5a2c4b6
Show file tree

Hide file tree

Showing 22 changed files with 81 additions and 35 deletions.
diff --git a/composer.json b/composer.json
@@ -33,7 +33,8 @@
   },
   "autoload": {
     "psr-4": {
-      "LaravelFreelancerNL\\FluentAQL\\": "src"
+      "LaravelFreelancerNL\\FluentAQL\\": "src",
+      "Tests\\":  "tests"
     }
   },
   "autoload-dev": {

diff --git a/src/Expressions/QueryExpression.php b/src/Expressions/QueryExpression.php
@@ -17,18 +17,18 @@ public function __construct(QueryBuilder $expression)
         parent::__construct($expression);
     }
 
-    public function compile(QueryBuilder $parentQueryBuilder): string
+    public function compile(QueryBuilder $queryBuilder): string
     {
-        $this->expression->registerVariable($parentQueryBuilder->getVariables());
+        $this->expression->registerVariable($queryBuilder->getVariables());
 
-        $this->expression = $this->expression->compile($parentQueryBuilder);
+        $this->expression = $this->expression->compile($queryBuilder);
 
-        $parentQueryBuilder->binds = array_unique(array_merge($parentQueryBuilder->binds, $this->expression->binds));
+        $queryBuilder->binds = array_unique(array_merge($queryBuilder->binds, $this->expression->binds));
 
         // Extract collections
         if (isset($this->expression->collections)) {
-            foreach ($this->expression->collections as $collection => $mode) {
-                $parentQueryBuilder->registerCollections($this->expression->collections[$collection], $mode);
+            foreach ($this->expression->collections as $mode => $collections) {
+                $queryBuilder->registerCollections($this->expression->collections[$mode], $mode);
             }
         }
 

diff --git a/tests/Unit/AQL/ArrayFunctionsTest.php b/tests/Unit/AQL/ArrayFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/DateFunctionsTest.php b/tests/Unit/AQL/DateFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/DocumentFunctionsTest.php b/tests/Unit/AQL/DocumentFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/Clauses/FilterClauseTest.php → tests/Unit/AQL/FilterClauseTest.php b/tests/Unit/Clauses/FilterClauseTest.php → tests/Unit/AQL/FilterClauseTest.php
@@ -1,8 +1,7 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Expressions;
+namespace Tests\Unit\AQL;
 
-use LaravelFreelancerNL\FluentAQL\Expressions\NullExpression;
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
 
@@ -15,7 +14,6 @@ class FilterClauseTest extends TestCase
 {
     public function testGetPredicates()
     {
-        $qb = new QueryBuilder();
         $result = (new QueryBuilder())
             ->for('u', 'Users')
             ->filter('u.active', '==', 'true')

diff --git a/tests/Unit/AQL/GeoFunctionsTest.php b/tests/Unit/AQL/GeoFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/GraphClausesTest.php b/tests/Unit/AQL/GraphClausesTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Clauses;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/MiscellaneousFunctionsTest.php b/tests/Unit/AQL/MiscellaneousFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/NumericFunctionsTest.php b/tests/Unit/AQL/NumericFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/OperatorExpressionsTest.php b/tests/Unit/AQL/OperatorExpressionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/QueryClausesTest.php b/tests/Unit/AQL/QueryClausesTest.php
@@ -1,9 +1,10 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Clauses;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
+use stdClass;
 
 /**
  * Class StructureTest.
@@ -373,7 +374,7 @@ public function testReturnClause()
 
     public function testOptionsClause()
     {
-        $options = new \stdClass();
+        $options = new stdClass();
         $options->method = 'sorted';
         $result = (new QueryBuilder())
             ->options($options)

diff --git a/tests/Unit/AQL/StatementClausesTest.php b/tests/Unit/AQL/StatementClausesTest.php
@@ -1,9 +1,10 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Clauses;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
+use stdClass;
 
 /**
  * Class StructureTest.
@@ -35,7 +36,7 @@ public function testLetStatement()
         self::assertEquals('LET x = @' . $result->getQueryId() . '_1', $result->query);
 
         $qb = new QueryBuilder();
-        $object = new \stdClass();
+        $object = new stdClass();
         $object->name = 'Catelyn';
         $object->surname = 'Stark';
         $object->alive = false;
@@ -89,7 +90,7 @@ public function testUpdateClause()
 
     public function testUpdateMaintainsNullValue()
     {
-        $data = new \stdClass();
+        $data = new stdClass();
         $data->name['first_name'] = null;
         $data->name['last_name'] = null;
         $result = (new QueryBuilder())

diff --git a/tests/Unit/AQL/StringFunctionsTest.php b/tests/Unit/AQL/StringFunctionsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Functions;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/AQL/SupportCommandsTest.php b/tests/Unit/AQL/SupportCommandsTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit;
+namespace Tests\Unit\AQL;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;

diff --git a/tests/Unit/Expressions/LiteralExpressionTest.php b/tests/Unit/Expressions/LiteralExpressionTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Expressions;
+namespace Tests\Unit\Expressions;
 
 use LaravelFreelancerNL\FluentAQL\Expressions\AttributeExpression;
 use LaravelFreelancerNL\FluentAQL\Expressions\BindExpression;

diff --git a/tests/Unit/Expressions/NullExpressionTest.php b/tests/Unit/Expressions/NullExpressionTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Expressions;
+namespace Tests\Unit\Expressions;
 
 use LaravelFreelancerNL\FluentAQL\Expressions\NullExpression;
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;

diff --git a/tests/Unit/Expressions/PredicateExpressionTest.php b/tests/Unit/Expressions/PredicateExpressionTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Expressions;
+namespace Tests\Unit\Expressions;
 
 use LaravelFreelancerNL\FluentAQL\Expressions\LiteralExpression;
 use LaravelFreelancerNL\FluentAQL\Expressions\PredicateExpression;
@@ -27,7 +27,7 @@ public function testPredicateExpressionWithLogicalOperatorAnd()
     {
         $qb = new QueryBuilder();
         $expression = new PredicateExpression((new LiteralExpression('x')), '==', (new LiteralExpression(10)), "AND");
-        $result = $expression->compile($qb);
+        $expression->compile($qb);
 
         self::assertEquals('AND', $expression->logicalOperator);
     }
@@ -36,7 +36,7 @@ public function testPredicateExpressionWithLogicalOperatorOr()
     {
         $qb = new QueryBuilder();
         $expression = new PredicateExpression((new LiteralExpression('x')), '==', (new LiteralExpression(10)), "OR");
-        $result = $expression->compile($qb);
+        $expression->compile($qb);
 
         self::assertEquals('OR', $expression->logicalOperator);
     }

diff --git a/tests/Unit/Expressions/QueryExpressionTest.php b/tests/Unit/Expressions/QueryExpressionTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit\Expressions;
+namespace Tests\Unit\Expressions;
 
 use LaravelFreelancerNL\FluentAQL\Expressions\QueryExpression;
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;

diff --git a/tests/Unit/GrammarTest.php b/tests/Unit/GrammarTest.php
@@ -1,10 +1,11 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit;
+namespace Tests\Unit;
 
 use LaravelFreelancerNL\FluentAQL\Grammar;
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
+use stdClass;
 
 /**
  * Class StructureTest.
@@ -255,7 +256,7 @@ public function testIsReference()
 
     public function testIsDocument()
     {
-        $doc = new \stdClass();
+        $doc = new stdClass();
         $doc->attribute1 = 'test';
         $result = $this->grammar->isObject($doc);
         self::assertTrue($result);

diff --git a/tests/Unit/QueryBuilderTest.php b/tests/Unit/QueryBuilderTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit;
+namespace Tests\Unit;
 
 use LaravelFreelancerNL\FluentAQL\Clauses\ForClause;
 use LaravelFreelancerNL\FluentAQL\Expressions\BindExpression;

diff --git a/tests/Unit/SubqueryTest.php b/tests/Unit/SubqueryTest.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace LaravelFreelancerNL\FluentAQL\Tests\Unit;
+namespace Tests\Unit;
 
 use LaravelFreelancerNL\FluentAQL\QueryBuilder;
 use LaravelFreelancerNL\FluentAQL\Tests\TestCase;
@@ -14,6 +14,7 @@ public function testSimpleSubQuery()
             ->filter('u.active', '==', 'true')
             ->return('u._key')
             ->get();
+
         self::assertEquals(
             'FOR u IN users FILTER u.active == true RETURN u._key',
             $subQuery->query
@@ -107,4 +108,47 @@ public function testSubQueryWithFunction()
             $result->query
         );
     }
+
+    public function testSubqueryWithLet()
+    {
+        $vertexGetQuery = (new QueryBuilder())
+            ->for('v', '0..99')
+            ->traverse('persons/123')
+            ->edgeCollections('relations')
+            ->return('v._key');
+
+        $vertexRemovalQuery = (new QueryBuilder())
+            ->for('vertexKey', 'verteces')
+            ->remove('vertexKey', 'persons')
+            ->get();
+
+        $edgeGetQuery = (new QueryBuilder())
+            ->for(['v', 'e'], '1..99')
+            ->traverse('persons/123')
+            ->edgeCollections('relations')
+            ->return('e._key');
+
+        $edgeRemovalQuery = (new QueryBuilder())
+            ->for('edgeKey', 'edges')
+            ->remove('edgeKey', 'relations');
+
+        $deleteQuery = (new QueryBuilder())
+            ->with('persons')
+            ->let('verteces', $vertexGetQuery)
+            ->let('edges', $edgeGetQuery)
+            ->let('vertexRemovals', $vertexRemovalQuery)
+            ->let('edgeRemovals', $edgeRemovalQuery)
+            ->return(['vertexRemovals', 'edgeRemovals'])
+            ->get();
+
+        self::assertEquals(
+            'WITH persons'
+            . ' LET verteces = (FOR v IN 0..99 OUTBOUND "persons/123" relations RETURN v._key)'
+            . ' LET edges = (FOR v, e IN 1..99 OUTBOUND "persons/123" relations RETURN e._key)'
+            . ' LET vertexRemovals = (FOR vertexKey IN verteces REMOVE vertexKey IN persons)'
+            . ' LET edgeRemovals = (FOR edgeKey IN edges REMOVE edgeKey IN relations)'
+            . ' RETURN [vertexRemovals,edgeRemovals]',
+            $deleteQuery->query
+        );
+    }
 }