tbianchini/shopit-back
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(menu): update contact menu to dynamic content type and preserve legacy data for rollback

Browse Source
This commit is contained in:
ncoronel
2026-09-30 13:29:41 -03:00
parent c40f145b60
commit 0ca2bc8d1b
7 changed files with 149 additions and 116 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
tests/Feature/Migrations/DynamicContactMenuTest.php Normal file
View File

@@ -0,0 +1,48 @@
<?php
namespace Tests\Feature\Migrations;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Schema;
use Tests\TestCase;
class DynamicContactMenuTest extends TestCase
{
public function test_it_converts_only_contact_and_preserves_legacy_data_for_rollback(): void
{
Schema::create('menues', function (Blueprint $table): void {
$table->string('code')->primary();
$table->string('content_type');
$table->json('static_content_schema')->nullable();
});
Schema::create('tenants_menues', function (Blueprint $table): void {
$table->string('tenant_code');
$table->string('menu_code');
$table->json('static_content')->nullable();
});
DB::table('menues')->insert([
['code' => 'help.contact', 'content_type' => 'static', 'static_content_schema' => '{}'],
['code' => 'help.faq', 'content_type' => 'static', 'static_content_schema' => '{}'],
]);
$legacy = json_encode(['whatsapp' => ['whatsapp_url' => 'https://wa.me/123']], JSON_THROW_ON_ERROR);
DB::table('tenants_menues')->insert([
'tenant_code' => 'test', 'menu_code' => 'help.contact', 'static_content' => $legacy,
]);
$migration = require database_path('migrations/2026_09_30_000600_make_contact_menu_dynamic.php');
$migration->up();
$migration->up();
$this->assertDatabaseHas('menues', [
'code' => 'help.contact', 'content_type' => 'dynamic', 'static_content_schema' => null,
]);
$this->assertDatabaseHas('menues', ['code' => 'help.faq', 'content_type' => 'static', 'static_content_schema' => '{}']);
$this->assertDatabaseHas('tenants_menues', ['menu_code' => 'help.contact', 'static_content' => $legacy]);
$migration->down();
$this->assertDatabaseHas('menues', ['code' => 'help.contact', 'content_type' => 'static']);
$schema = json_decode(DB::table('menues')->where('code', 'help.contact')->value('static_content_schema'), true, flags: JSON_THROW_ON_ERROR);
$this->assertSame('required|url', $schema['whatsapp.whatsapp_url']);
$this->assertDatabaseHas('tenants_menues', ['menu_code' => 'help.contact', 'static_content' => $legacy]);
}
}

51
tests/Feature/Seeders/MenuSeederTest.php
View File

@@ -338,26 +338,8 @@ class MenuSeederTest extends TestCase
], $faq->static_content_schema);
$contact = Menu::query()->where('code', 'help.contact')->firstOrFail();
$this->assertSame(Menu::CONTENT_TYPE_STATIC, $contact->content_type);
$this->assertSame('required|url', $contact->static_content_schema['whatsapp.whatsapp_url']);
$this->assertSame('required|string', $contact->static_content_schema['phone']);
$this->assertSame(
'required|array|min:1',
$contact->static_content_schema['locations.*.addresses']
);
$this->assertSame(
'required|array|size:2',
$contact->static_content_schema['locations.*.addresses.*.coordinates']
);
$this->assertSame(
'required|string',
$contact->static_content_schema['locations.*.addresses.*.address']
);
$this->assertSame(
'required|numeric|between:-90,90',
$contact->static_content_schema['locations.*.addresses.*.coordinates.0']
);
$this->assertSame('prohibited', $contact->static_content_schema['map_locations']);
$this->assertSame(Menu::CONTENT_TYPE_DYNAMIC, $contact->content_type);
$this->assertNull($contact->static_content_schema);
}
public function test_it_assigns_help_to_the_supported_tenants_with_faq_content(): void
@@ -385,30 +367,6 @@ class MenuSeederTest extends TestCase
'is_active' => false,
],
];
$expectedContact = [
'whatsapp' => [
'whatsapp_url' => 'https://wa.me/543412602222',
'whatsapp_label' => 'Chatea con nosotros',
],
'phone' => '+54 9 (0341) 6658247',
'locations' => [
'rosario' => [
'label' => 'Rosario',
'addresses' => [
[
'label' => 'Gigante de Arroyito',
'address' => 'Av. Génova 640, Rosario',
'coordinates' => [-32.913997, -60.674567],
],
[
'label' => 'Telepagos',
'address' => 'Rioja 1150, piso 12, dpto. 3, Rosario',
'coordinates' => [-32.946820, -60.639320],
],
],
],
],
];
foreach ([$sonder, $fiesta] as $tenant) {
$helpMenus = $tenant->menues()
@@ -422,10 +380,7 @@ class MenuSeederTest extends TestCase
$expectedQuestions,
$helpMenus->firstWhere('code', 'help.faq')->pivot->static_content
);
$this->assertEquals(
$expectedContact,
$helpMenus->firstWhere('code', 'help.contact')->pivot->static_content
);
$this->assertNull($helpMenus->firstWhere('code', 'help.contact')->pivot->static_content);
}
$this->assertFalse(

2
tests/Feature/Tenant/BootstrapTenantControllerTest.php
View File

@@ -510,7 +510,7 @@ class BootstrapTenantControllerTest extends TestCase
$tenant->menues()->sync([
$parent->code,
$staticChild->code => ['static_content' => $staticContent],
$dynamicChild->code,
$dynamicChild->code => ['static_content' => ['legacy' => 'must not be returned']],
]);
$userRole->menus()->sync([
$parent->code,

54
tests/Unit/Bootstrap/TenantContactResourceTest.php Normal file
View File

@@ -0,0 +1,54 @@
<?php
namespace Tests\Unit\Bootstrap;
use App\Domains\Core\Menu\Models\Menu;
use App\Domains\Core\Menu\Models\TenantMenu;
use App\Domains\Core\Tenant\Models\SocialMedia;
use App\Domains\Core\Tenant\Models\Tenant;
use App\Domains\Core\Tenant\Resources\TenantResource;
use App\Domains\Ticketing\Event\Models\Event;
use Illuminate\Database\Eloquent\Relations\Pivot;
use Tests\TestCase;
class TenantContactResourceTest extends TestCase
{
public function test_dynamic_contact_omits_legacy_content_and_prioritizes_active_event_social_media(): void
{
$tenant = new Tenant(['codigo' => 'test']);
foreach (['favicon', 'headerLogo', 'footerLogo', 'headerBackgroundImage', 'footerBackgroundImage', 'storefrontWebsiteType'] as $relation) {
$tenant->setRelation($relation, null);
}
$contact = new Menu([
'code' => 'help.contact', 'label' => 'Contacto', 'content_type' => 'dynamic', 'route' => '/ayuda/contacto',
]);
$contact->setRelation('pivot', new TenantMenu(['static_content' => ['whatsapp' => 'legacy']]));
$faq = new Menu(['code' => 'help.faq', 'label' => 'FAQ', 'content_type' => 'static']);
$faq->setRelation('pivot', new TenantMenu(['static_content' => ['question' => 'Example']]));
$tenant->setRelation('menues', collect([$contact, $faq]));
$tenantSocial = new SocialMedia(['code' => 'whatsapp', 'name' => 'WhatsApp', 'icon' => 'fa-brands fa-whatsapp']);
$tenantSocial->setRelation('pivot', new Pivot(['url' => 'https://wa.me/111']));
$eventSocial = new SocialMedia(['code' => 'whatsapp', 'name' => 'WhatsApp', 'icon' => 'fa-brands fa-whatsapp']);
$eventSocial->setRelation('pivot', new Pivot(['url' => 'https://wa.me/222']));
$tenant->setRelation('socialMedia', collect([$tenantSocial]));
$event = new Event;
$event->setRelation('socialMedia', collect([$eventSocial]));
$event->setRelation('address', null);
$event->setRelation('dates', collect());
$tenant->setRelation('activeEvent', $event);
$data = TenantResource::make($tenant)->resolve(request());
$this->assertSame('dynamic', $data['menues'][0]['content_type']);
$this->assertArrayNotHasKey('static_content', $data['menues'][0]);
$this->assertSame(['question' => 'Example'], $data['menues'][1]['static_content']);
$this->assertSame('https://wa.me/222', $data['social_media'][0]['url']);
$event->setRelation('socialMedia', collect());
$data = TenantResource::make($tenant)->resolve(request());
$this->assertCount(0, $data['social_media']);
$tenant->setRelation('activeEvent', null);
$data = TenantResource::make($tenant)->resolve(request());
$this->assertSame('https://wa.me/111', $data['social_media'][0]['url']);
}
}