Replace internal deleteLeafNodes query to use chunking instead of pag…

…ination

Closes #324

src/Models/Model.php

@@ -1174,16 +1174,15 @@ public function delete($recursive = false)
      *
      * @throws \LdapRecord\LdapRecordException
      *
-     * @return Collection
+     * @return void
      */
     protected function deleteLeafNodes()
     {
-        return $this->newQueryWithoutScopes()
+        $this->newQueryWithoutScopes()
             ->in($this->dn)
             ->listing()
-            ->paginate()
-            ->each(function (self $model) {
-                $model->delete($recursive = true);
+            ->chunk(250, function ($models) {
+                $models->each->delete($recursive = true);
             });
     }
 

tests/Models/ModelQueryTest.php

@@ -2,6 +2,7 @@
 
 namespace LdapRecord\Tests\Models;
 
+use Closure;
 use LdapRecord\Connection;
 use LdapRecord\Container;
 use LdapRecord\ContainerException;
@@ -338,13 +339,17 @@ public function test_delete_leaf_nodes()
         $query = m::mock(Builder::class);
         $query->shouldReceive('listing')->once()->andReturnSelf();
         $query->shouldReceive('in')->once()->with('foo')->andReturnSelf();
-        $query->shouldReceive('paginate')->once()->andReturn($shouldBeDeleted);
+        $query->shouldReceive('chunk')->once()->with(250, m::on(function ($callback) use ($shouldBeDeleted) {
+            $callback($shouldBeDeleted);
+
+            return $callback instanceof Closure;
+        }));
 
         $model = m::mock(Entry::class)->makePartial();
         $model->shouldReceive('newQueryWithoutScopes')->once()->andReturn($query);
 
         $model->setRawAttributes(['dn' => 'foo', 'cn' => 'bar']);
-        $this->assertEquals($shouldBeDeleted, $model->deleteLeafNodes());
+        $this->assertNull($model->deleteLeafNodes());
         $this->assertFalse($shouldBeDeleted->first()->exists);
     }