add test

Author: soyuka

src/Laravel/ApiPlatformProvider.php

@@ -81,6 +81,7 @@
 use ApiPlatform\Laravel\Controller\EntrypointController;
 use ApiPlatform\Laravel\Eloquent\Extension\FilterQueryExtension;
 use ApiPlatform\Laravel\Eloquent\Extension\QueryExtensionInterface;
+use ApiPlatform\Laravel\Eloquent\Filter\BooleanFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\DateFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\EqualsFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\FilterInterface as EloquentFilterInterface;
@@ -422,7 +423,7 @@ public function register(): void
 
         $this->app->bind(OperationMetadataFactoryInterface::class, OperationMetadataFactory::class);
 
-        $this->app->tag([EqualsFilter::class, PartialSearchFilter::class, DateFilter::class, OrderFilter::class, RangeFilter::class], EloquentFilterInterface::class);
+        $this->app->tag([BooleanFilter::class, EqualsFilter::class, PartialSearchFilter::class, DateFilter::class, OrderFilter::class, RangeFilter::class], EloquentFilterInterface::class);
 
         $this->app->bind(FilterQueryExtension::class, function (Application $app) {
             $tagged = iterator_to_array($app->tagged(EloquentFilterInterface::class));

src/Laravel/Eloquent/Extension/FilterQueryExtension.php

@@ -40,7 +40,7 @@ public function apply(Builder $builder, array $uriVariables, Operation $operatio
         $context['operation'] = $operation;
 
         foreach ($operation->getParameters() ?? [] as $parameter) {
-            if (!($values = $parameter->getValue()) || $values instanceof ParameterNotFound) {
+            if (null === ($values = $parameter->getValue()) || $values instanceof ParameterNotFound) {
                 continue;
             }
 

src/Laravel/Eloquent/Filter/BooleanFilter.php

@@ -13,11 +13,12 @@
 
 namespace ApiPlatform\Laravel\Eloquent\Filter;
 
+use ApiPlatform\Metadata\JsonSchemaFilterInterface;
 use ApiPlatform\Metadata\Parameter;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
 
-final class BooleanFilter implements FilterInterface
+final class BooleanFilter implements FilterInterface, JsonSchemaFilterInterface
 {
     use QueryPropertyTrait;
 
@@ -40,4 +41,9 @@ public function apply(Builder $builder, mixed $values, Parameter $parameter, arr
 
         return $builder->{$context['whereClause'] ?? 'where'}($this->getQueryProperty($parameter), $values);
     }
+
+    public function getSchema(Parameter $parameter): array
+    {
+        return ['type' => 'boolean'];
+    }
 }

src/Laravel/Metadata/ParameterValidationResourceMetadataCollectionFactory.php

@@ -144,6 +144,10 @@ private function addSchemaValidation(Parameter $parameter): Parameter
             $assertions[] = 'array';
         }
 
+        if (isset($schema['type']) && 'boolean' === $schema['type']) {
+            $assertions[] = 'boolean';
+        }
+
         if (!$assertions) {
             return $parameter;
         }

src/Laravel/Tests/EloquentTest.php

@@ -406,4 +406,15 @@ public function testWithAccessor(): void
         $res = $this->get('/api/with_accessors/1', ['Accept' => ['application/ld+json']]);
         $this->assertArraySubset(['name' => 'test'], $res->json());
     }
+
+    public function testBooleanFilter(): void
+    {
+        BookFactory::new()->has(AuthorFactory::new())->count(10)->create();
+        $res = $this->get('/api/books?published=notabool', ['Accept' => ['application/ld+json']]);
+        $this->assertEquals($res->getStatusCode(), 422);
+
+        $res = $this->get('/api/books?published=0', ['Accept' => ['application/ld+json']]);
+        $this->assertEquals($res->getStatusCode(), 200);
+        $this->assertEquals($res->json()['totalItems'], 0);
+    }
 }

src/Laravel/workbench/app/Models/Book.php

@@ -13,6 +13,7 @@
 
 namespace Workbench\App\Models;
 
+use ApiPlatform\Laravel\Eloquent\Filter\BooleanFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\DateFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\EqualsFilter;
 use ApiPlatform\Laravel\Eloquent\Filter\OrderFilter;
@@ -71,12 +72,13 @@
     property: 'name'
 )]
 #[QueryParameter(key: 'properties', filter: PropertyFilter::class)]
+#[QueryParameter(key: 'published', filter: BooleanFilter::class)]
 class Book extends Model
 {
     use HasFactory;
     use HasUlids;
 
-    protected $visible = ['name', 'author', 'isbn', 'publication_date', 'is_available'];
+    protected $visible = ['name', 'author', 'isbn', 'publication_date', 'published', 'is_available'];
     protected $fillable = ['name', 'is_available'];
     protected $casts = [
         'is_available' => 'boolean',

src/Laravel/workbench/database/factories/BookFactory.php

@@ -39,6 +39,7 @@ public function definition(): array
             'publication_date' => fake()->optional()->date(),
             'is_available' => 1 === random_int(0, 1),
             'internal_note' => fake()->text(),
+            'published' => fake()->boolean(100),
         ];
     }
 }

src/Laravel/workbench/database/migrations/2023_07_15_231244_create_book_table.php

@@ -34,6 +34,7 @@ public function up(): void
             $table->date('publication_date')->nullable();
             $table->boolean('is_available')->default(true);
             $table->text('internal_note')->nullable();
+            $table->boolean('published')->nullable();
             $table->integer('author_id')->unsigned();
             $table->foreign('author_id')->references('id')->on('authors');
             $table->timestamps();