feat(catalog): enhance featured groups with paginated and carousel layouts, update seeder and tests

2026-07-23 13:24:39 -03:00
parent 1438ff66c7
commit e778d862ba
5 changed files with 157 additions and 39 deletions
--- a/app/Domains/Catalog/Controllers/CatalogController.php
+++ b/app/Domains/Catalog/Controllers/CatalogController.php
@@ -2,8 +2,10 @@

 namespace App\Domains\Catalog\Controllers;

+use App\Domains\Catalog\Enums\GroupLayout;
 use App\Domains\Catalog\Models\CatalogItem;
 use App\Domains\Catalog\Models\FeaturedGroup;
+use App\Domains\Catalog\Models\FeaturedItem;
 use App\Domains\Catalog\Requests\CatalogItemDetailRequest;
 use App\Domains\Catalog\Requests\FeaturedGroupPageRequest;
 use App\Domains\Catalog\Requests\StoreCatalogItemRequest;
@@ -14,6 +16,7 @@ use App\Domains\Catalog\Resources\CatalogItemResource;
 use App\Domains\Catalog\Services\CatalogService;
 use App\Domains\Tenant\Models\Tenant;
 use App\Http\Controllers\Controller;
+use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Pagination\LengthAwarePaginator;

@@ -81,9 +84,19 @@ class CatalogController extends Controller
            ->setStatusCode(201);
    }

-    /** @return array<string, mixed> */
+    /** @return array<array-key, mixed> */
    private function featuredItemsResponse(FeaturedGroup $featuredGroup, int $page): array
    {
+        if ($featuredGroup->group_layout !== GroupLayout::Paginated) {
+            $featuredItems = $this->featuredItemsQuery($featuredGroup)->get();
+
+            $featuredItems->each(
+                fn ($featuredItem) => $featuredItem->setRelation('featuredGroup', $featuredGroup)
+            );
+
+            return CatalogFeaturedItemResource::collection($featuredItems)->resolve();
+        }
+
        $paginator = $this->paginateFeaturedItems($featuredGroup, $page);

        $paginator->getCollection()->each(
@@ -99,17 +112,7 @@ class CatalogController extends Controller
        FeaturedGroup $featuredGroup,
        int $page,
    ): LengthAwarePaginator {
-        $paginator = $featuredGroup->featuredItems()
-            ->with([
-                'catalogItem.inventory',
-                'catalogItem.attachments',
-                'catalogItem.variants.inventory',
-                'catalogItem.variants.attachments',
-                'catalogItem.variants.definitions.itemAttribute.attribute',
-                'catalogItem.bundleComponents.catalogItem',
-                'catalogItem.bundleComponents.variant.catalogItem',
-            ])
-            ->paginate(
+        $paginator = $this->featuredItemsQuery($featuredGroup)->paginate(
            perPage: self::ITEMS_PER_PAGE,
            pageName: 'page',
            page: $page,
@@ -120,4 +123,18 @@ class CatalogController extends Controller
            'featuredGroup' => $featuredGroup->id,
        ]));
    }
+
+    /** @return HasMany<FeaturedItem, FeaturedGroup> */
+    private function featuredItemsQuery(FeaturedGroup $featuredGroup): HasMany
+    {
+        return $featuredGroup->featuredItems()->with([
+            'catalogItem.inventory',
+            'catalogItem.attachments',
+            'catalogItem.variants.inventory',
+            'catalogItem.variants.attachments',
+            'catalogItem.variants.definitions.itemAttribute.attribute',
+            'catalogItem.bundleComponents.catalogItem',
+            'catalogItem.bundleComponents.variant.catalogItem',
+        ]);
+    }
 }
--- a/app/Domains/Catalog/Resources/CatalogFeaturedGroupResource.php
+++ b/app/Domains/Catalog/Resources/CatalogFeaturedGroupResource.php
@@ -9,8 +9,8 @@ use Illuminate\Http\Resources\Json\JsonResource;
 /** @mixin FeaturedGroup */
 class CatalogFeaturedGroupResource extends JsonResource
 {
-    /** @param array<string, mixed> $itemsPage */
-    public function __construct($resource, private readonly array $itemsPage)
+    /** @param array<array-key, mixed> $items */
+    public function __construct($resource, private readonly array $items)
    {
        parent::__construct($resource);
    }
@@ -24,7 +24,7 @@ class CatalogFeaturedGroupResource extends JsonResource
            'layout' => $this->product_layout->value,
            'group_layout' => $this->group_layout->value,
            'group_order' => $this->group_order,
-            'items' => $this->itemsPage,
+            'items' => $this->items,
        ];
    }
 }
--- a/database/seeders/ProductCatalogFromImagesSeeder.php
+++ b/database/seeders/ProductCatalogFromImagesSeeder.php
@@ -168,12 +168,12 @@ class ProductCatalogFromImagesSeeder extends Seeder
    {
        FeaturedGroup::query()->where('tenant_code', $tenant->codigo)->delete();

-        $group = FeaturedGroup::query()->create([
+        $paginatedGroup = FeaturedGroup::query()->create([
            'tenant_code' => $tenant->codigo,
            'product_layout' => ProductLayout::ColumnWithImage,
            'group_layout' => GroupLayout::Paginated,
            'group_name' => 'Productos',
-            'group_order' => 0,
+            'group_order' => 2,
        ]);

        $items = CatalogItem::query()
@@ -181,7 +181,7 @@ class ProductCatalogFromImagesSeeder extends Seeder
            ->orderBy('id')
            ->get('id');

-        $group->featuredItems()->createMany(
+        $paginatedGroup->featuredItems()->createMany(
            $items->values()->map(
                fn (CatalogItem $item, int $order): array => [
                    'catalog_item_id' => $item->id,
@@ -189,6 +189,29 @@ class ProductCatalogFromImagesSeeder extends Seeder
                ]
            )->all()
        );
+
+        $carouselGroup = FeaturedGroup::query()->create([
+            'tenant_code' => $tenant->codigo,
+            'product_layout' => ProductLayout::ColumnWithImage,
+            'group_layout' => GroupLayout::Carousel,
+            'group_name' => 'Productos destacados',
+            'group_order' => 1,
+        ]);
+
+        $randomItems = CatalogItem::query()
+            ->where('tenant_code', $tenant->codigo)
+            ->inRandomOrder()
+            ->limit(5)
+            ->get('id');
+
+        $carouselGroup->featuredItems()->createMany(
+            $randomItems->values()->map(
+                fn (CatalogItem $item, int $order): array => [
+                    'catalog_item_id' => $item->id,
+                    'order' => $order,
+                ]
+            )->all()
+        );
    }

    /**
--- a/tests/Feature/Catalog/CatalogControllerTest.php
+++ b/tests/Feature/Catalog/CatalogControllerTest.php
@@ -58,14 +58,13 @@ class CatalogControllerTest extends TestCase
            ->assertJsonPath('0.title', 'Cart')
            ->assertJsonPath('0.layout', ProductLayout::ColumnWithCart->value)
            ->assertJsonPath('0.group_layout', GroupLayout::SimpleVertical->value)
-            ->assertJsonPath('0.items.meta.current_page', 1)
-            ->assertJsonPath('0.items.data.0.nombre', 'Variants')
-            ->assertJsonPath('0.items.data.0.descripcion', 'Variants description')
-            ->assertJsonPath('0.items.data.0.precio', '100.00')
-            ->assertJsonPath('0.items.data.0.stock_tecnico', 7)
-            ->assertJsonCount(2, '0.items.data.0.variants')
-            ->assertJsonPath('0.items.data.0.variants.0.stock_tecnico', 4)
-            ->assertJsonPath('0.items.data.0.variants.1.stock_tecnico', 3)
+            ->assertJsonPath('0.items.0.nombre', 'Variants')
+            ->assertJsonPath('0.items.0.descripcion', 'Variants description')
+            ->assertJsonPath('0.items.0.precio', '100.00')
+            ->assertJsonPath('0.items.0.stock_tecnico', 7)
+            ->assertJsonCount(2, '0.items.0.variants')
+            ->assertJsonPath('0.items.0.variants.0.stock_tecnico', 4)
+            ->assertJsonPath('0.items.0.variants.1.stock_tecnico', 3)
            ->assertJsonPath('1.title', 'Row')
            ->assertJsonPath('1.items.data.0.stock_tecnico', 8)
            ->assertJsonCount(0, '1.items.data.0.variants');
@@ -145,6 +144,60 @@ class CatalogControllerTest extends TestCase
            ->assertJsonPath('data.0.nombre', 'Item 13');
    }

+    public function test_non_paginated_group_layouts_return_all_items_as_a_plain_array(): void
+    {
+        $tenant = $this->createTenant('catalog-simple-layouts');
+        $layouts = [
+            GroupLayout::Simple,
+            GroupLayout::SimpleVertical,
+            GroupLayout::Carousel,
+        ];
+
+        foreach ($layouts as $order => $groupLayout) {
+            $group = $this->createGroup(
+                $tenant,
+                ProductLayout::Row,
+                $groupLayout->value,
+                $order,
+                $groupLayout,
+            );
+
+            foreach (range(1, 13) as $number) {
+                $item = $this->createItem($tenant, "{$groupLayout->value} Item {$number}");
+                $group->featuredItems()->create([
+                    'catalog_item_id' => $item->id,
+                    'order' => $number,
+                ]);
+            }
+        }
+
+        $response = $this->getJson("/api/tenants/{$tenant->codigo}/catalog")->assertOk();
+        $groups = $response->json();
+
+        foreach ($layouts as $index => $groupLayout) {
+            $this->assertSame($groupLayout->value, $groups[$index]['group_layout']);
+            $this->assertCount(13, $groups[$index]['items']);
+            $this->assertSame(
+                "{$groupLayout->value} Item 1",
+                $groups[$index]['items'][0]['nombre'],
+            );
+            $this->assertArrayNotHasKey('data', $groups[$index]['items']);
+            $this->assertArrayNotHasKey('meta', $groups[$index]['items']);
+        }
+
+        $carouselGroup = FeaturedGroup::query()
+            ->where('tenant_code', $tenant->codigo)
+            ->where('group_layout', GroupLayout::Carousel)
+            ->sole();
+
+        $this->getJson(
+            "/api/tenants/{$tenant->codigo}/catalog/featured-groups/{$carouselGroup->id}/items?page=2"
+        )
+            ->assertOk()
+            ->assertJsonCount(13)
+            ->assertJsonPath('0.nombre', 'carousel Item 1');
+    }
+
    private function createGroup(
        Tenant $tenant,
        ProductLayout $layout,
--- a/tests/Feature/Seeders/ProductCatalogFromImagesSeederTest.php
+++ b/tests/Feature/Seeders/ProductCatalogFromImagesSeederTest.php
@@ -21,7 +21,7 @@ class ProductCatalogFromImagesSeederTest extends TestCase
 {
    use RefreshDatabase;

-    public function test_it_features_every_sonder_product_in_a_single_image_column(): void
+    public function test_it_creates_paginated_and_random_carousel_groups_for_sonder(): void
    {
        Storage::fake('s3');

@@ -52,19 +52,44 @@ class ProductCatalogFromImagesSeederTest extends TestCase
            ProductCatalogFromImagesSeeder::class,
        ]);

-        $group = FeaturedGroup::query()
+        $groups = FeaturedGroup::query()
            ->where('tenant_code', $tenant->codigo)
            ->with('featuredItems.catalogItem')
-            ->sole();
+            ->orderBy('group_order')
+            ->get()
+            ->keyBy('group_name');

-        $this->assertSame('Productos', $group->group_name);
-        $this->assertSame(ProductLayout::ColumnWithImage, $group->product_layout);
-        $this->assertSame(GroupLayout::Paginated, $group->group_layout);
-        $this->assertSame(0, $group->group_order);
+        $paginatedGroup = $groups->get('Productos');
+        $carouselGroup = $groups->get('Productos destacados');
+        $catalogItemIds = CatalogItem::query()
+            ->where('tenant_code', $tenant->codigo)
+            ->orderBy('id')
+            ->pluck('id');
+
+        $this->assertNotNull($paginatedGroup);
+        $this->assertSame(ProductLayout::ColumnWithImage, $paginatedGroup->product_layout);
+        $this->assertSame(GroupLayout::Paginated, $paginatedGroup->group_layout);
+        $this->assertSame(0, $paginatedGroup->group_order);
        $this->assertSame(
            CatalogItem::query()->where('tenant_code', $tenant->codigo)->orderBy('id')->pluck('slug')->all(),
-            $group->featuredItems->pluck('catalogItem.slug')->all(),
+            $paginatedGroup->featuredItems->pluck('catalogItem.slug')->all(),
+        );
+        $this->assertCount(10, $paginatedGroup->featuredItems);
+
+        $this->assertNotNull($carouselGroup);
+        $this->assertSame(ProductLayout::ColumnWithImage, $carouselGroup->product_layout);
+        $this->assertSame(GroupLayout::Carousel, $carouselGroup->group_layout);
+        $this->assertSame(1, $carouselGroup->group_order);
+        $this->assertCount(5, $carouselGroup->featuredItems);
+        $this->assertCount(
+            5,
+            $carouselGroup->featuredItems->pluck('catalog_item_id')->unique(),
+        );
+        $this->assertCount(
+            0,
+            $carouselGroup->featuredItems
+                ->pluck('catalog_item_id')
+                ->diff($catalogItemIds),
        );
-        $this->assertCount(10, $group->featuredItems);
    }
 }