Split and refactor Multiple Filters Test

Power-Components · May 9, 2024 · 05556c9 · 05556c9
1 parent 7ac36f0
commit 05556c9
Show file tree

Hide file tree

Showing 6 changed files with 182 additions and 52 deletions.
diff --git a/tests/Concerns/Components/ComponentsForFilterTest.php b/tests/Concerns/Components/ComponentsForFilterTest.php
@@ -0,0 +1,45 @@
+<?php
+
+use PowerComponents\LivewirePowerGrid\Facades\Filter;
+
+use PowerComponents\LivewirePowerGrid\Tests\{Concerns\Components\DishesQueryBuilderTable,
+    Concerns\Components\DishesTable,
+    Concerns\Components\DishesTableWithJoin};
+
+$component = new class () extends DishesTable {
+    public function filters(): array
+    {
+        return [
+            Filter::number('price_BRL')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::number('price') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::inputText('name')->placeholder('dish_name_xyz_placeholder')->operators(),
+            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::boolean('in_stock'),
+        ];
+    }
+};
+
+$componentQueryBuilder = new class () extends DishesQueryBuilderTable {
+    public function filters(): array
+    {
+        return [
+            Filter::number('price_BRL')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::number('price') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::inputText('name')->placeholder('dish_name_xyz_placeholder')->operators(),
+            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::boolean('in_stock'),
+        ];
+    }
+};
+
+$componentJoin = new class () extends DishesTableWithJoin {
+    public function filters(): array
+    {
+        return [
+            Filter::number('price_BRL') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::inputText('dish_name')->placeholder('dish_name_xyz_placeholder')->operators(),
+            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
+            Filter::boolean('in_stock'),
+        ];
+    }
+};
diff --git a/tests/Datasets/FilterComponent.php b/tests/Datasets/FilterComponent.php
@@ -0,0 +1,10 @@
+<?php
+
+require(__DIR__ . '/../Concerns/Components/ComponentsForFilterTest.php');
+
+dataset('filterComponent', [
+    'tailwind -> id'         => [$component::class, (object) ['theme' => 'tailwind', 'field' => 'name']],
+    'bootstrap -> id'        => [$component::class, (object) ['theme' => 'bootstrap', 'field' => 'name']],
+    'tailwind -> dishes.id'  => [$componentJoin::class, (object) ['theme' => 'tailwind', 'field' => 'dishes.name']],
+    'bootstrap -> dishes.id' => [$componentJoin::class, (object) ['theme' => 'bootstrap', 'field' => 'dishes.name']],
+]);
diff --git a/tests/Feature/Filters/FilterInputTextTest.php b/tests/Feature/Filters/FilterInputTextTest.php
@@ -0,0 +1,65 @@
+<?php
+
+use Illuminate\Support\Str;
+use PowerComponents\LivewirePowerGrid\PowerGridComponent;
+
+use function PowerComponents\LivewirePowerGrid\Tests\Plugins\livewire;
+
+require(__DIR__ . '/../../Concerns/Components/ComponentsForFilterTest.php');
+
+it('properly filters by inputText', function (string $component, object $params) {
+    $component = livewire($component)
+        ->call($params->theme);
+
+    /** @var PowerGridComponent $component */
+    expect($component->filters)
+        ->toMatchArray([]);
+
+    $component->set('filters', filterInputText('ba', 'contains', $params->field));
+
+    if (str_contains($params->field, '.')) {
+        $data  = Str::of($params->field)->explode('.');
+        $table = $data->get(0);
+        $field = $data->get(1);
+
+        expect($component->filters)
+            ->toMatchArray([
+                'input_text' => [
+                    $table => [
+                        $field => 'ba',
+                    ],
+                ],
+                'input_text_options' => [
+                    $table => [
+                        $field => 'contains',
+                    ],
+                ],
+            ]);
+    } else {
+        expect($component->filters)
+            ->toMatchArray([
+                'input_text' => [
+                    $params->field => 'ba',
+                ],
+                'input_text_options' => [
+                    $params->field => 'contains',
+                ],
+            ]);
+    }
+
+    $component->assertSee('Barco-Sushi da Sueli')
+        ->assertSeeHtml('dish_name_xyz_placeholder');
+
+    $filters = array_merge($component->filters, filterNumber('price', min: '1\'500.20', max: '3\'000.00'));
+
+    $component->set('filters', $filters)
+        ->assertSeeHtml('placeholder="min_xyz_placeholder"')
+        ->assertSeeHtml('placeholder="max_xyz_placeholder"')
+        ->assertSee('Barco-Sushi Simples')
+        ->assertDontSee('Barco-Sushi da Sueli')
+        ->assertDontSee('Polpetone Filé Mignon')
+        ->assertDontSee('борщ');
+
+    expect($component->filters)->toBe($filters);
+})->group('filters')
+->with('filterComponent');
diff --git a/tests/Feature/Filters/FilterNumberTest.php b/tests/Feature/Filters/FilterNumberTest.php
@@ -0,0 +1,42 @@
+<?php
+
+use function PowerComponents\LivewirePowerGrid\Tests\Plugins\livewire;
+
+require(__DIR__ . '/../../Concerns/Components/ComponentsForFilterTest.php');
+
+it('properly filters by filter Number', function (string $component, object $params) {
+    $component = livewire($component)
+        ->call($params->theme);
+
+    $filters = array_merge($component->filters, filterNumber('price', min: '1\'500.20', max: '3\'000.00'));
+
+    $component->set('filters', $filters)
+        ->assertSeeHtml('placeholder="min_xyz_placeholder"')
+        ->assertSeeHtml('placeholder="max_xyz_placeholder"')
+        ->assertSee('Barco-Sushi Simples')
+        ->assertDontSee('Barco-Sushi da Sueli')
+        ->assertDontSee('Polpetone Filé Mignon')
+        ->assertDontSee('борщ');
+
+    expect($component->filters)->toBe($filters);
+
+    // Use wrong separators
+    $filters = array_merge($component->filters, filterNumber('price', min: '1@500#20', max: '3@000#00'));
+
+    $component->set('filters', $filters)
+        ->assertSee('No records found');
+})->group('filters')
+->with('filterComponent');
+
+it('properly filters by filter Number with wrong separators', function (string $component, object $params) {
+    $component = livewire($component)
+        ->call($params->theme);
+
+    // Use wrong separators
+    $filters = array_merge($component->filters, filterNumber('price', min: '1@500#20', max: '3@000#00'));
+
+    $component->set('filters', $filters)
+        ->assertSee('No records found');
+})->skipOnPostgreSQL()
+->group('filters')
+->with('filterComponent');
diff --git a/tests/Feature/Filters/FilterMultipleTest.php → ...s/Feature/Filters/MultipleFiltersTest.php b/tests/Feature/Filters/FilterMultipleTest.php → ...s/Feature/Filters/MultipleFiltersTest.php
@@ -1,52 +1,11 @@
 <?php
 
 use Illuminate\Support\Str;
-use PowerComponents\LivewirePowerGrid\Facades\Filter;
 use PowerComponents\LivewirePowerGrid\PowerGridComponent;
 
 use function PowerComponents\LivewirePowerGrid\Tests\Plugins\livewire;
 
-use PowerComponents\LivewirePowerGrid\Tests\{Concerns\Components\DishesQueryBuilderTable,
-    Concerns\Components\DishesTable,
-    Concerns\Components\DishesTableWithJoin};
-
-$component = new class () extends DishesTable {
-    public function filters(): array
-    {
-        return [
-            Filter::number('price_BRL')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::number('price') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::inputText('name')->placeholder('dish_name_xyz_placeholder')->operators(),
-            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::boolean('in_stock'),
-        ];
-    }
-};
-
-$componentQueryBuilder = new class () extends DishesQueryBuilderTable {
-    public function filters(): array
-    {
-        return [
-            Filter::number('price_BRL')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::number('price') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::inputText('name')->placeholder('dish_name_xyz_placeholder')->operators(),
-            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::boolean('in_stock'),
-        ];
-    }
-};
-
-$componentJoin = new class () extends DishesTableWithJoin {
-    public function filters(): array
-    {
-        return [
-            Filter::number('price_BRL') ->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::inputText('dish_name')->placeholder('dish_name_xyz_placeholder')->operators(),
-            Filter::number('price')->placeholder('min_xyz_placeholder', 'max_xyz_placeholder')->thousands("'")->decimal(','),
-            Filter::boolean('in_stock'),
-        ];
-    }
-};
+require(__DIR__ . '/../../Concerns/Components/ComponentsForFilterTest.php');
 
 it('properly filters by inputText, number, boolean filter and clearAll', function (string $component, object $params) {
     $component = livewire($component)
@@ -103,9 +62,6 @@ public function filters(): array
 
     expect($component->filters)->toBe($filters);
 
-    // Use wrong separators
-    $filters = array_merge($component->filters, filterNumber('price', min: '1@500#20', max: '3@000#00'));
-
     $component->set('filters', $filters)
         ->assertSee('No records found');
 
@@ -130,9 +86,4 @@ public function filters(): array
     expect($component->filters)
         ->toMatchArray([]);
 })->group('filters')
-    ->with([
-        'tailwind -> id'         => [$component::class, (object) ['theme' => 'tailwind', 'field' => 'name']],
-        'bootstrap -> id'        => [$component::class, (object) ['theme' => 'bootstrap', 'field' => 'name']],
-        'tailwind -> dishes.id'  => [$componentJoin::class, (object) ['theme' => 'tailwind', 'field' => 'dishes.name']],
-        'bootstrap -> dishes.id' => [$componentJoin::class, (object) ['theme' => 'bootstrap', 'field' => 'dishes.name']],
-    ]);
+->with('filterComponent');
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -38,6 +38,14 @@ function requiresMySQL()
 
     return test();
 }
+function skipOnMySQL()
+{
+    if (DB::getDriverName() === 'mysql') {
+        test()->markTestSkipped('This test does not run on MySQL');
+    }
+
+    return test();
+}
 
 function requiresSQLite()
 {
@@ -51,7 +59,7 @@ function requiresSQLite()
 function skipOnSQLite()
 {
     if (DB::getDriverName() === 'sqlite') {
-        test()->markTestSkipped('This test requires MYSQL/PGSQL database');
+        test()->markTestSkipped('This test does not run on SQLite');
     }
 
     return test();
@@ -66,6 +74,15 @@ function requiresPostgreSQL()
     return test();
 }
 
+function skipOnPostgreSQL()
+{
+    if (DB::getDriverName() === 'pgsql') {
+        test()->markTestSkipped('This test does not run on PostgreSQL');
+    }
+
+    return test();
+}
+
 function requiresOpenSpout()
 {
     $isInstalled = \Composer\InstalledVersions::isInstalled('openspout/openspout');